You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2022/03/03 14:06:41 UTC

[GitHub] [hudi] huberylee commented on a change in pull request #4901: [HUDI-3445] Support Clustering Command Based on Call Procedure Command for Spark SQL

huberylee commented on a change in pull request #4901:
URL: https://github.com/apache/hudi/pull/4901#discussion_r818688760



##########
File path: hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/action/cluster/strategy/PartitionAwareClusteringPlanStrategy.java
##########
@@ -113,6 +114,15 @@ public PartitionAwareClusteringPlanStrategy(HoodieTable table, HoodieEngineConte
         .build());
   }
 
+  public List<String> getMatchedPartitions(HoodieWriteConfig config, List<String> partitionPaths) {
+    String partitionSelected = config.getClusteringPartitionSelected();
+    if (!StringUtils.isNullOrEmpty(partitionSelected)) {
+      return Arrays.asList(partitionSelected.split(","));

Review comment:
       Yes, the filtered partition is separated by comma when pruning partition, see the end of `org.apache.spark.sql.hudi.command.procedures.RunClusteringProcedure#prunePartition` for more detail.




-- 
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-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org