JingGe commented on pull request #18911: URL: https://github.com/apache/flink/pull/18911#issuecomment-1050788912
> Thanks for the PR, @JingGe, I added some comments. In general, it looks fine to me, with the exception of two points: > > * What is the rationale behind marking the classes as Experimental? The API seems to be pretty tight in terms of input/output parameters, or do you expect potential changes? > * CI is currently failing, but the failures seem to be unrelated to your changes, so try a rebase. Thanks @afedulov for your effort. I have addressed your comments. The reason of using Experimental is that it is the first time we introduce `AvroParquetReaders`. I am not pretty sure if there will be any further change request coming from users. It will be upgraded to be PublishEvolving in 1.16 release. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
