Skip to content

Conversation

rahil-c
Copy link
Contributor

@rahil-c rahil-c commented Oct 3, 2025

Describe the issue this Pull Request addresses

  • This is a follow up pr from test: add functional payload test cases for v6 (OverwriteWithLatestAvroPayload, DefaultHoodieRecordPayload) #14041, which seeks to add test coverage for a special case where a user created a MOR v6 table with an older 0.x writer, and with an ordering field explicitly set. In order to achieve this we needed to add a test in the payload fixtures approach, as the other pr had a limitation of using a 1.1.x writer when creating a v6 table (which has some inconsistency in the behavior around OverwriteWithLatestAvroPayload now not handling ordering values in 1.1.0)

Summary and Changelog

  • Modified the scala payload script, to simplify the batches of data preupgrade (only having higher ordering values)
  • Added specific mixed ordering values post upgrade for this case, as well as validation that data is following event time semantics.

Impact

None

Risk Level

None

Documentation Update

None

Contributor's checklist

  • Read through contributor's guide
  • Enough context is provided in the sections above
  • Adequate tests were added if applicable

@github-actions github-actions bot added the size:M PR with lines of changes in (100, 300] label Oct 3, 2025
private static Stream<Arguments> testArgsPayloadUpgradeDowngrade() {
String[] payloadTypes = {
"default", "overwrite", "partial", "postgres", "mysql",
"overwrite", "partial", "postgres", "mysql",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

need to add back default payload

@hudi-bot
Copy link

hudi-bot commented Oct 3, 2025

CI report:

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M PR with lines of changes in (100, 300]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants