You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/12/23 09:21:57 UTC

[GitHub] [flink] Tartarus0zm commented on pull request #21149: [FLINK-29527][formats/parquet] Make unknownFieldsIndices work for single ParquetReader

Tartarus0zm commented on PR #21149:
URL: https://github.com/apache/flink/pull/21149#issuecomment-1363774890

   @saLeox  Thanks for contribution.
   In our company, there are strict usage rules, so there is no such thing as multiple schema files in the same partition directory.
   The changes is fine, but I think it's best to have rule constraints, I'm not sure what the behavior in hive/spark is like, I think it's best to keep the engine behavior consistent, otherwise it will bring additional interpretation costs.
   What do you think? @saLeox @luoyuxia 


-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org