[GitHub] [drill] arina-ielchiieva commented on issue #1836: DRILL-7156: empty parquet files support

[GitBox <gi...@apache.org>]

arina-ielchiieva commented on issue #1836: DRILL-7156: empty parquet files support
URL: https://github.com/apache/drill/pull/1836#issuecomment-518183790
 
 
   @oleg-zinovev thanks for making the changes, though the situation is little bit awkward, since I was working on similar changes and did not know you intend to do them as well (https://issues.apache.org/jira/browse/DRILL-4517). Though I was working on reading empty parquet files but not writing them. I suggest you separate out writing empty parquet files into separate PR as for reading it might be better if my changes will be used instead: first you change metadata cache files and this would affect backward compatibility as well as will have to store more information than needed, secondly your changes does not seem to optimize reading complex types. What do you think?
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services