Re: [VOTE] Add JSON encoding to Avro serialization
+1 Best, Alexander Fedulov On Thu, 26 Oct 2023 at 13:39, Jing Ge wrote: > Hi Ryan, > > +1 > > Thanks for driving it. I have left one comment. > > Best regards, > Jing > > > On Wed, Oct 25, 2023 at 10:05 PM David Radley > wrote: > > > Looks good to me +1 > > > > From: Ryan Skraba > > Date: Wednesday, 25 October 2023 at 17:19 > > To: dev@flink.apache.org > > Subject: [EXTERNAL] [VOTE] Add JSON encoding to Avro serialization > > Hello! > > > > I'm reviewing a new feature of another contributor (Dale Lane) on > > FLINK-33058 that adds JSON-encoding in addition to the binary Avro > > serialization format. He addressed my original objections that JSON > > encoding isn't _generally_ a best practice for Avro messages. > > > > The discussion is pretty well-captured in the JIRA and PR, but I > > wanted to give it a bit of visiblity and see if there were any strong > > opinions on the subject! Given the minor nature of this feature, I > > don't think it requires a FLIP. > > > > *TL;DR*: JSON-encoded Avro might not be ideal for production, but it > > has a place for small systems and especially setting up and testing > > before making the switch to binary-encoding. > > > > All my best, Ryan > > > > [Jira]: https://issues.apache.org/jira/browse/FLINK-33058 > > [PR]: https://github.com/apache/flink/pull/23395 > > > > Unless otherwise stated above: > > > > IBM United Kingdom Limited > > Registered in England and Wales with number 741598 > > Registered office: PO Box 41, North Harbour, Portsmouth, Hants. PO6 3AU > > >
Re: [VOTE] Add JSON encoding to Avro serialization
Hi Ryan, +1 Thanks for driving it. I have left one comment. Best regards, Jing On Wed, Oct 25, 2023 at 10:05 PM David Radley wrote: > Looks good to me +1 > > From: Ryan Skraba > Date: Wednesday, 25 October 2023 at 17:19 > To: dev@flink.apache.org > Subject: [EXTERNAL] [VOTE] Add JSON encoding to Avro serialization > Hello! > > I'm reviewing a new feature of another contributor (Dale Lane) on > FLINK-33058 that adds JSON-encoding in addition to the binary Avro > serialization format. He addressed my original objections that JSON > encoding isn't _generally_ a best practice for Avro messages. > > The discussion is pretty well-captured in the JIRA and PR, but I > wanted to give it a bit of visiblity and see if there were any strong > opinions on the subject! Given the minor nature of this feature, I > don't think it requires a FLIP. > > *TL;DR*: JSON-encoded Avro might not be ideal for production, but it > has a place for small systems and especially setting up and testing > before making the switch to binary-encoding. > > All my best, Ryan > > [Jira]: https://issues.apache.org/jira/browse/FLINK-33058 > [PR]: https://github.com/apache/flink/pull/23395 > > Unless otherwise stated above: > > IBM United Kingdom Limited > Registered in England and Wales with number 741598 > Registered office: PO Box 41, North Harbour, Portsmouth, Hants. PO6 3AU >
Re: [VOTE] Add JSON encoding to Avro serialization
Looks good to me +1 From: Ryan Skraba Date: Wednesday, 25 October 2023 at 17:19 To: dev@flink.apache.org Subject: [EXTERNAL] [VOTE] Add JSON encoding to Avro serialization Hello! I'm reviewing a new feature of another contributor (Dale Lane) on FLINK-33058 that adds JSON-encoding in addition to the binary Avro serialization format. He addressed my original objections that JSON encoding isn't _generally_ a best practice for Avro messages. The discussion is pretty well-captured in the JIRA and PR, but I wanted to give it a bit of visiblity and see if there were any strong opinions on the subject! Given the minor nature of this feature, I don't think it requires a FLIP. *TL;DR*: JSON-encoded Avro might not be ideal for production, but it has a place for small systems and especially setting up and testing before making the switch to binary-encoding. All my best, Ryan [Jira]: https://issues.apache.org/jira/browse/FLINK-33058 [PR]: https://github.com/apache/flink/pull/23395 Unless otherwise stated above: IBM United Kingdom Limited Registered in England and Wales with number 741598 Registered office: PO Box 41, North Harbour, Portsmouth, Hants. PO6 3AU
[VOTE] Add JSON encoding to Avro serialization
Hello! I'm reviewing a new feature of another contributor (Dale Lane) on FLINK-33058 that adds JSON-encoding in addition to the binary Avro serialization format. He addressed my original objections that JSON encoding isn't _generally_ a best practice for Avro messages. The discussion is pretty well-captured in the JIRA and PR, but I wanted to give it a bit of visiblity and see if there were any strong opinions on the subject! Given the minor nature of this feature, I don't think it requires a FLIP. *TL;DR*: JSON-encoded Avro might not be ideal for production, but it has a place for small systems and especially setting up and testing before making the switch to binary-encoding. All my best, Ryan [Jira]: https://issues.apache.org/jira/browse/FLINK-33058 [PR]: https://github.com/apache/flink/pull/23395