Splitting hudi-common classes into common and core
#19194
Closed
vinothchandar
started this conversation in
General Discussions
Replies: 5 comments
|
I have completed the clean up here. #19195 love to discuss the split here more, before I get started. |
0 replies
|
Opened a draft PR here. Will bring this up in the weekly sync as well |
0 replies
|
Impact: compile time deps on |
0 replies
|
+1 logically the new hierarchy makes sense. |
0 replies
|
Proceeding with the changes. Thanks for chiming in here and also the dev weekly. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
All , the hudi-common package has grown in scope. and I have spent sometime with #19193 introducing a new
coresubpackage here to house all the core Hudi timeline, filesystem view and the consistency/transactional stuff, as well as key I/O. I would likecommonto be mostly holding common models, common base infrastructure (functions, expressions, schema) used by everyother module.Eventually, I want to split off
hudi-commonandhudi-coreas separate modules where something likehudi-iocan depend onhudi-common. Today due to this mixing of classes with different scope, the dependency is other way around and we are keeping strictly I/O related classes in hudi-common without a choice.End state looks like this, with proper homing of classes.
Thoughts/suggestions welcome!
All reactions