echauchot commented on pull request #15725:
URL: https://github.com/apache/flink/pull/15725#issuecomment-824843657


   @HuangZhenQiu you already agreed for this change in the jira ticket. Still 
there is a question (in TODO in the code and in the ticket) why is this needed 
to read the parquet schema from split in `open()` ? Shouldn't it be the same 
schema than in the complete file ? Can't we use the already extracted schema in 
the constructor ?
   
   @zentol as @HuangZhenQiu is not a committer, could you please take a look or 
point me to another committer ?
   
   
   


-- 
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]


Reply via email to