lw309637554 commented on a change in pull request #2275:
URL: https://github.com/apache/hudi/pull/2275#discussion_r548043722
##########
File path:
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieClusteringConfig.java
##########
@@ -73,9 +73,13 @@
public static final String CLUSTERING_TARGET_FILE_MAX_BYTES =
CLUSTERING_STRATEGY_PARAM_PREFIX + "target.file.max.bytes";
public static final String DEFAULT_CLUSTERING_TARGET_FILE_MAX_BYTES =
String.valueOf(1 * 1024 * 1024 * 1024L); // 1GB
- // constants related to clustering that may be used by more than 1 strategy.
+ // Constants related to clustering that may be used by more than 1 strategy.
public static final String CLUSTERING_SORT_COLUMNS_PROPERTY =
HoodieClusteringConfig.CLUSTERING_STRATEGY_PARAM_PREFIX + "sort.columns";
+ // When file groups is in clustering, need to handle the update to these
file groups. Default strategy just reject the update
+ public static final String CLUSTERING_UPDATES_STRATEGY_PROP =
"hoodie.clustering.updates.strategy";
+ public static final String DEFAULT_CLUSTERING_UPDATES_STRATEGY =
"org.apache.hudi.client.clustering.update.strategy.SparkRejectUpdateStrategy";
Review comment:
SparkRejectUpdateStrategy is in hudi-spark, hudi-client-common not
depend hudi-spark
----------------------------------------------------------------
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]
