Re: [VOTE] Add JSON encoding to Avro serialization

2023-10-26 Thread Alexander Fedulov
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 cont

Re: [VOTE] Add JSON encoding to Avro serialization

2023-10-26 Thread Jing Ge
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: [EXTERNA

Re: [VOTE] Add JSON encoding to Avro serialization

2023-10-25 Thread David Radley
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

[VOTE] Add JSON encoding to Avro serialization

2023-10-25 Thread Ryan Skraba
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