-
Notifications
You must be signed in to change notification settings - Fork 20
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.
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
Issue #398: Qbeast Spark Refactor -- Delta abstraction #446
Conversation
* Abstract the QbeastSnapshot from Delta Snapshot
…beast-io#421) * Abstract PreCommitHook * Abstract StagingDataManager * Propagate stats * Update MetadataManager * Update MetadataWriter * Update MetadataManager * Add static typing * Update QbeastFileUtils * Fix IndexFile stats * Update DeleteFile logic * Update qbeast files
…t-io#423) * Update rollup data writer * Abstract QbeastStats * Update QbeastStats to use Number * Update QbeastStats to use String * Update QbeastStats * Update QbeastStats Delta
* Code refactoring: Use Qbeast abstractions instead of Delta imports
* Separate modules between src, core and delta
…beast-io#436) * Fix QbeastStats serialization/deserialization
* Set dataChange to false on Table.optimize()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few comments left! Some of them can be solved here, others would be best to leave it for another discussion.
Let me know what do you think 🙏
The rest looks goooood
Just one small thing that we notice!! We should NOT update the README documentation. The only document that should be updated with developments is But for regular users, copying and pasting from the docs would not work with the new packages and the latest version 0.7.0. |
Description
Merge Refactoring Branch into Main #447
Final part of #398