-
Notifications
You must be signed in to change notification settings - Fork 12
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
[1.x] Perform clear-and-replace when applying entity Migration
#1405
Conversation
@dmitrykuzmin PTAL. |
Migration
Migration
Codecov Report
@@ Coverage Diff @@
## 1.x-dev #1405 +/- ##
=============================================
- Coverage 91.08% 91.05% -0.04%
+ Complexity 4793 4791 -2
=============================================
Files 614 614
Lines 15201 15203 +2
Branches 860 860
=============================================
- Hits 13846 13843 -3
- Misses 1090 1091 +1
- Partials 265 269 +4 |
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.
LGTM with a couple of minor comments.
/** | ||
* Fills the state of the migrated {@code ProcessManager} with some random values. | ||
*/ | ||
public class RandomFillProcess |
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.
public class RandomFillProcess | |
public final class RandomFillProcess |
This changeset addresses #1403, but just for 1.x branch.
The library version is set to
1.7.7-SNAPSHOT.3
.