Skip to content

[HUDI-5097] Fix partition reading without partition fields table config #7069

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
alexeykudinkin merged 1 commit into from
Dec 15, 2022
Merged

[HUDI-5097] Fix partition reading without partition fields table config #7069

alexeykudinkin merged 1 commit into from
Dec 15, 2022

Conversation

@xushiyan
Copy link
Member

@xushiyan xushiyan commented Oct 26, 2022 •
edited
Loading

Change Logs

Revert to FSUtils.getAllPartitionPaths to load partitions properly. Details in #6016 (comment)

Only for 0.12.2 to keep behavior consistent over patch releases

Impact

Affects partition reading logic for partitioned tables.

Risk level

Low

Documentation Update

NA

Contributor's checklist

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

@xushiyan xushiyan added the priority:critical Production degraded; pipelines stalled label Oct 26, 2022
@xushiyan xushiyan marked this pull request as draft October 27, 2022 17:15
@xushiyan
Copy link
Member Author

xushiyan commented Oct 27, 2022

not landing this to master. only meant for 0.12.2 release patch

@alexeykudinkin
Copy link
Contributor

alexeykudinkin commented Nov 9, 2022 •
edited
Loading

not landing this to master. only meant for 0.12.2 release patch

LGTM

@nsivabalan nsivabalan added the release-0.12.2 Patches targetted for 0.12.2 label Dec 6, 2022
@xushiyan xushiyan changed the base branch from master to release-0.12.2-blockers-candidate December 15, 2022 12:20
@xushiyan xushiyan force-pushed the HUDI-5097-fix-partition-reading branch from 0fa2259 to a9bc4cc Compare December 15, 2022 12:22
@xushiyan xushiyan marked this pull request as ready for review December 15, 2022 12:24
@xushiyan xushiyan added priority:blocker Production down; release blocker and removed priority:critical Production degraded; pipelines stalled labels Dec 15, 2022
@hudi-bot
Copy link
Collaborator

hudi-bot commented Dec 15, 2022

CI report:

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

@alexeykudinkin alexeykudinkin merged commit 492f7d7 into apache:release-0.12.2-blockers-candidate Dec 15, 2022
@xushiyan xushiyan deleted the HUDI-5097-fix-partition-reading branch December 16, 2022 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexeykudinkin alexeykudinkin alexeykudinkin approved these changes

Assignees

@alexeykudinkin alexeykudinkin

Labels

priority:blocker Production down; release blocker release-0.12.2 Patches targetted for 0.12.2

Projects

Hudi PR Support
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@xushiyan @alexeykudinkin @hudi-bot @nsivabalan