You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2020/05/14 16:42:11 UTC

[GitHub] [incubator-hudi] vinothchandar commented on pull request #1602: [HUDI-494] fix incorrect record size estimation

vinothchandar commented on pull request #1602:
URL: https://github.com/apache/incubator-hudi/pull/1602#issuecomment-628753707


   with dynamic bloom filter, we are going to bound the bloom filter size anyway.. And finally, parquet may have bloom filter support on its own.. I am not comfortable introducing complexity to solve this edge case..  
   
   It feels better to invest the energy to building a better statistically sound approach overall, than patch this,.. WDYT? 


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