Skip to content

[HUDI-8910] Fix Unstable UT TestHoodieCompactionStrategy#12704

Merged
yihua merged 1 commit intomasterfrom
HUDI-8910
Jan 24, 2025
Merged

[HUDI-8910] Fix Unstable UT TestHoodieCompactionStrategy#12704
yihua merged 1 commit intomasterfrom
HUDI-8910

Conversation

@zhangyue19921010
Copy link
Contributor

Change Logs

TestHoodieCompactionStrategy.testBoundedIOSimple is Unstable because of random compactions based on .map(e -> Pair.of(e, partitionPaths[RANDOM.nextInt(partitionPaths.length - 1)]))

Impact

no

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

low

Documentation Update

Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

  • The config description must be updated if new configs are added or the default value of the configs are changed
  • Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
    ticket number here and follow the instruction to make
    changes to the website.

Contributor's checklist

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

@github-actions github-actions bot added the size:XS PR with lines of changes in <= 10 label Jan 24, 2025
@zhangyue19921010
Copy link
Contributor Author

Hi @codope Fix unstable ut. PTAL

Copy link
Contributor

@yihua yihua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@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

@yihua yihua merged commit d8b77c5 into master Jan 24, 2025
43 checks passed
@zhangyue19921010 zhangyue19921010 deleted the HUDI-8910 branch June 18, 2025 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS PR with lines of changes in <= 10

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants