nsivabalan commented on code in PR #6581:
URL: https://github.com/apache/hudi/pull/6581#discussion_r1435369701
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieClusteringConfig.java:
##########
@@ -153,13 +156,26 @@ public class HoodieClusteringConfig extends HoodieConfig {
.key(PLAN_PARTITION_FILTER_MODE)
.defaultValue(ClusteringPlanPartitionFilterMode.NONE)
.sinceVersion("0.11.0")
- .withDocumentation("Partition filter mode used in the creation of
clustering plan. Available values are - "
- + "NONE: do not filter table partition and thus the clustering plan
will include all partitions that have clustering candidate."
- + "RECENT_DAYS: keep a continuous range of partitions, worked
together with configs '" + DAYBASED_LOOKBACK_PARTITIONS.key() + "' and '"
+ .withDocumentation("Partition Filter mode used in the creation of
clustering plan. Available values are - "
+ + "NONE: do not filter anything and thus the clustering plan will
include all file slices that have clustering candidate."
+ + "RECENT: keep a continuous range of partitions, worked together
with configs '" + DAYBASED_LOOKBACK_PARTITIONS.key() + "' and '"
+ PLAN_STRATEGY_SKIP_PARTITIONS_FROM_LATEST.key() + "."
+ "SELECTED_PARTITIONS: keep partitions that are in the specified
range ['" + PARTITION_FILTER_BEGIN_PARTITION.key() + "', '"
+ PARTITION_FILTER_END_PARTITION.key() + "'].");
+ public static final ConfigProperty<ClusteringPlanFilterMode>
PLAN_FILTER_MODE_NAME = ConfigProperty
+ .key(PLAN_FILTER_MODE)
+ .defaultValue(ClusteringPlanFilterMode.NONE)
+ .sinceVersion("0.13.0")
+ .withDocumentation("Filter mode used in the creation of clustering plan.
Available values are - "
+ + "NONE: do not filter table partition and thus the clustering plan will
include all partitions that have clustering candidate."
Review Comment:
Note to self: might need fixing.
--
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]