Splitting hudi-common classes into common and core
#19194
vinothchandar
started this conversation in
General Discussions
Replies: 4 comments
-
|
I have completed the clean up here. #19195 love to discuss the split here more, before I get started. |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Opened a draft PR here. Will bring this up in the weekly sync as well |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Impact: compile time deps on |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
+1 logically the new hierarchy makes sense. |
Beta Was this translation helpful? Give feedback.
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!
Beta Was this translation helpful? Give feedback.
All reactions