feat(schema): Phase 18 - HoodieAvroUtils removal (Part 2) #17656
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal-p3
branch
2 times, most recently
from
2de4b54 to
e0a6621
Compare
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal-p3
branch
from
e0a6621 to
7b4aa7e
Compare
voonhous
changed the title
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal-p3
branch
from
7b4aa7e to
c98cc5c
Compare
Member
Author
|
Will need some suggestions on how to rename:
These will also need to be renamed:
I'll leave this as an exercise before actually performing the renames. |
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal-p3
branch
2 times, most recently
from
8843e78 to
d39084f
Compare
Member
Author
|
@hudi-bot run azure |
voonhous
requested review from
the-other-tim-brown
and removed request for
jonvex
3 tasks
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal-p3
branch
2 times, most recently
from
422899c to
2eda55e
Compare
voonhous
changed the base branch from
master
to
phase-18-HoodieAvroUtils-removal
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal-p3
branch
from
2eda55e to
f7ff4c5
Compare
Member
Author
|
@the-other-tim-brown This is ready for review too, I've changed the base branch to the PR that it is stacked on top. Only the NET diff are shown now. Should be easier for you to review now. |
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal
branch
from
24b1721 to
f93e264
Compare
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal-p3
branch
from
f7ff4c5 to
c393e06
Compare
Additional compilation error fixes - AvroOrcUtils - HoodieBootstrapSchemaProvider - HoodieSparkBootstrapSchemaProvider - TestOrcBootstrap
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal
branch
from
f93e264 to
a7fd9fa
Compare
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal-p3
branch
from
c393e06 to
2749bf0
Compare
voonhous
force-pushed
the
phase-18-HoodieAvroUtils-removal
branch
2 times, most recently
from
8224033 to
c94921b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe the issue this Pull Request addresses
Reference issue: #14283
Remove methods that were migrated to
HoodieSchemaUtils, consolidate remaining Avro-specific utilities, update documentation. The scope here only covers:hudi-hadoop-commonNOTE: Merge this after #17599
Key Changes:
HoodieAvroUtilsstatic functions have been fully qualified. This explicitly marks technical debt and makes these usages easily searchable for future refactoring.Summary and Changelog
Swap out HoodieAvroUtils to HoodieSchema equivalent.
Impact
None
Risk Level
Low
Documentation Update
Contributor's checklist