[GitHub] [flink] slinkydeveloper commented on pull request #18657: [FLINK-26001][avro] Implement ProjectableDecodingFormat for avro BulkDecodingFormat

2022-02-10 Thread GitBox
slinkydeveloper commented on pull request #18657: URL: https://github.com/apache/flink/pull/18657#issuecomment-1034766454 Thanks for the explanation! As I said before, a comment which explains this would be nice to have there, for the rest it looks good and it sounds like a nice perf

[GitHub] [flink] slinkydeveloper commented on pull request #18657: [FLINK-26001][avro] Implement ProjectableDecodingFormat for avro BulkDecodingFormat

2022-02-10 Thread GitBox
slinkydeveloper commented on pull request #18657: URL: https://github.com/apache/flink/pull/18657#issuecomment-1034623498 Hi @JingsongLi @tsreaper, I have a question about this PR. I stumbled upon this and, without Avro knowledge, I thought it was wrong as I initially thought the sch