+1 Best, Alexander Fedulov
On Thu, 26 Oct 2023 at 13:39, Jing Ge <j...@ververica.com.invalid> 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 <david_rad...@uk.ibm.com> > wrote: > > > Looks good to me +1 > > > > From: Ryan Skraba <ryan.skr...@aiven.io.INVALID> > > Date: Wednesday, 25 October 2023 at 17:19 > > To: dev@flink.apache.org <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 > > >
