-
Notifications
You must be signed in to change notification settings - Fork 0
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
DM-42076: Add script for one-shot collections migration #29
Conversation
INI file specified only couple of useful options and those are actually fixed and are easier to setup when constructing configuration manually.
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #29 +/- ##
==========================================
- Coverage 56.02% 54.37% -1.66%
==========================================
Files 32 33 +1
Lines 1112 1144 +32
Branches 254 256 +2
==========================================
- Hits 623 622 -1
- Misses 455 488 +33
Partials 34 34 ☔ View full report in Codecov by Sentry. |
9bc31db
to
a3771cf
Compare
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.
I'm not going to pretend I looked at every line, but I did look at the algorithmic section and it looked okay.
New script migrates schema from
NameKeyCollectionManager
toSynthIntKeyCollectionManager
, replacing collection names with integer IDs. Tested on a copy of main repo at USDF.Checklist