vinothchandar commented on pull request #3330:
URL: https://github.com/apache/hudi/pull/3330#issuecomment-955637904


   >we can introduce a config in DataSourceReadOptions. WDYT?
   
   yes. we can introduce a new one there. But wondering if we need the data 
skipping flag in `HoodieWriteConfig` still? What purpose do you think it will 
solve. I am fine either ways. 
   
   Let me know when/if you have tested the latest code on your cluster and 
verified performance gains are still same (after removing the unsafe code etc). 
   
   Thanks for working through this with me 


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to