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
Make cortex, view, and layer idens to be unique #1402
Conversation
Note that this will change the iden of the view and the default layer.
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.
Looking really good! Few small tweaks and one big question ;)
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.
minor hive notes; need to add a regression test.
* Remove cortex _moveDict, * Add tests for hive rename and exists
* Remove a duplicate slab offset class * Add an additional delete method to slab offset class * Add unit tests for slab offset
This is for future expansion where the write layer might change.
…into uniq-iden
Codecov Report
@@ Coverage Diff @@
## master #1402 +/- ##
==========================================
- Coverage 95.18% 95.11% -0.07%
==========================================
Files 133 133
Lines 19598 19649 +51
==========================================
+ Hits 18654 18690 +36
- Misses 944 959 +15
Continue to review full report at Codecov.
|
Note that this will change the iden of the view and the default layer.