Skip to content

[HUDI-6000] Fix RunClusteringProcedure when no partition matched#8318

Merged
YannByron merged 1 commit intoapache:masterfrom
Zouxxyy:xinyu/fix-cluster
Apr 14, 2023
Merged

[HUDI-6000] Fix RunClusteringProcedure when no partition matched#8318
YannByron merged 1 commit intoapache:masterfrom
Zouxxyy:xinyu/fix-cluster

Conversation

@Zouxxyy
Copy link
Contributor

@Zouxxyy Zouxxyy commented Mar 29, 2023

Change Logs

There are two ways to specify the partitions to be clustered in run_clustering:

predicate => xxx and selected_partitions => xxx

But cluster is still scheduled when predicate does not match any partitions or selected_partitions = "" currently

Impact

Cluster will not be scheduled when no partition matched

Risk level (write none, low medium or high below)

low

Documentation Update

none

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@Zouxxyy
Copy link
Contributor Author

Zouxxyy commented Mar 29, 2023

@XuQianJin-Stars Can you help with a review~

@Zouxxyy Zouxxyy force-pushed the xinyu/fix-cluster branch from 58a156f to 85bd477 Compare March 30, 2023 01:25
@Zouxxyy Zouxxyy force-pushed the xinyu/fix-cluster branch from 00894dc to e182235 Compare March 30, 2023 12:43
@Zouxxyy Zouxxyy changed the title [HUDI-6000] Fix cluster when no partition matched with partition selected [HUDI-6000] Fix RunClusteringProcedure when no partition matched Mar 30, 2023
@hudi-bot
Copy link
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@Zouxxyy
Copy link
Contributor Author

Zouxxyy commented Apr 3, 2023

@leesf hi, CI is green

@YannByron YannByron merged commit 46c9bc1 into apache:master Apr 14, 2023
@Zouxxyy Zouxxyy deleted the xinyu/fix-cluster branch April 14, 2023 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants