lw309637554 commented on pull request #2275:
URL: https://github.com/apache/hudi/pull/2275#issuecomment-750892428


   @satishkotha @leesf  thanks, have resolved the comments. 
   1. add isClusteringEnable to open the update strategy  in 
BaseSparkCommitActionExecutor(not use the "'default' strategy"  for not open 
clustering, i think use the isClusteringEnable config more clearness)
   2. also use isClusteringEnable to check if we should exclude clustering file 
group in small files (not move the filter code to update strategy, i think 
partitioner handle for clustering make sense)


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