beyond1920 commented on code in PR #9030: URL: https://github.com/apache/hudi/pull/9030#discussion_r1239726999
########## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieClusteringConfig.java: ########## @@ -328,6 +330,20 @@ public class HoodieClusteringConfig extends HoodieConfig { + "Please exercise caution while setting this config, especially when clustering is done very frequently. This could lead to race condition in " + "rare scenarios, for example, when the clustering completes after instants are fetched but before rollback completed."); + public static final ConfigProperty<Boolean> BUCKET_CLUSTERING_MERGE_ENABLED = ConfigProperty + .key("hoodie.bucket.clustering.merge.enabled") + .defaultValue(true) + .markAdvanced() + .sinceVersion("0.14.0") Review Comment: Flink write pipelines could not handle `hoodie.bucket.clustering.merge.enabled` correctly, so the value is set to `false` in flink engine automatically, and would check the resize plan does not include merge operations. And Flink write pipelines could handle `hoodie.bucket.clustering.sort.enabled`. However it's time consuming, so the default value is set to `false` in flink engine automatically, and use could enable this if they need. -- 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: commits-unsubscr...@hudi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org