TsReaper commented on pull request #13636:
URL: https://github.com/apache/flink/pull/13636#issuecomment-714471424


   > Thanks for contribution, we should not rely on 
`StatsSetupConst.NUM_FILES`, because in Hive metastore, these stats information 
is just a hint, and they may not correct.
   > 
   > We can get file numbers from file system directly. You should make a 
performance testing for that.
   
   Indeed. I've tested the performance of listing the table file directory and 
its impact is neglectable (at least for TPCDS benchmark).


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