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

Reply via email to