echauchot commented on pull request #15156: URL: https://github.com/apache/flink/pull/15156#issuecomment-803949582
> > > Hey :) Thanks for CC. Looks good! > > > One thing to consider, is there is anything stopping us from going even futher and adding possibility to deserialize `SpecificRecord` instances too ? > > > > > > Nothing I guess, It is just that this PR relies on a already existing _AvroRowSerializationSchema_ that produces GenericRecords from internal flink Row. > > IMHO, we should try to incorporate that (if not too hard obviously), since having at least some type safety would be very beneficial. Sure, we can do this in next PR. I'd like that feature to be merged quite quickly so that my project could depend on a snapshot jar (and a released jar in the future) and not on my locally patched jar. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
