Skip to content

Conversation

@vldpyatkov
Copy link
Contributor

@vldpyatkov vldpyatkov commented Feb 17, 2023

if (hash == 0) {
hash = storage.hashCode();
} else if (hash != storage.hashCode()) {
if (storageIdx == 0) {
Copy link
Contributor

@alievmirza alievmirza Feb 22, 2023

Choose a reason for hiding this comment

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

Please add todo here that we need to change the way how we compare partitions after consistency checker feature will be implemented https://issues.apache.org/jira/browse/IGNITE-18869
And please, add comment to the epic that we have this todo, so we couldn't forget to fix this place

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added the todo in a summary of the method.

…ibuted/ItTxDistributedTestSingleNode.java

Co-authored-by: Mirza Aliev <alievmirza@gmail.com>
@vldpyatkov vldpyatkov merged commit de1ee32 into apache:main Feb 22, 2023
@vldpyatkov vldpyatkov deleted the ignite-18814 branch February 22, 2023 14:28
lowka pushed a commit to gridgain/apache-ignite-3 that referenced this pull request Mar 18, 2023
lowka pushed a commit to gridgain/apache-ignite-3 that referenced this pull request Apr 19, 2023
isapego pushed a commit to isapego/ignite-3 that referenced this pull request Feb 4, 2025
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