Skip to content
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

[2.x] Perform clear-and-replace when applying entity Migration #1419

Merged
merged 7 commits into from
Nov 16, 2021

Conversation

armiol
Copy link
Contributor

@armiol armiol commented Nov 16, 2021

This changeset addresses #1403, now for 2.x master branch. In short, previously during the entity migration the old state was being merged with the new state returned by the Migration instance. That led to a number of unwanted side effects and bugs.

Now, the entity Migration routines fully replace the old state with the new state, as per its original API design.

(a similar PR has been merged into 1.x branch)

The library version has been set to 2.0.0-SNAPSHOT.78.

@armiol armiol self-assigned this Nov 16, 2021
@armiol armiol marked this pull request as ready for review November 16, 2021 15:41
@armiol armiol requested a review from a team November 16, 2021 15:41
@codecov
Copy link

codecov bot commented Nov 16, 2021

Codecov Report

Merging #1419 (1b40891) into master (894d7a8) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #1419   +/-   ##
=========================================
  Coverage     91.12%   91.12%           
  Complexity     4930     4930           
=========================================
  Files           628      628           
  Lines         15420    15421    +1     
  Branches        893      893           
=========================================
+ Hits          14051    14053    +2     
  Misses         1066     1066           
+ Partials        303      302    -1     

Copy link
Contributor

@dmdashenkov dmdashenkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with a single comment.

.setId(newUuid().hashCode())
.build();
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please kill this line.

@armiol armiol merged commit 27abb3d into master Nov 16, 2021
@armiol armiol deleted the migration-clear-replace branch November 16, 2021 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants