[
https://issues.apache.org/jira/browse/GEODE-7784?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17042116#comment-17042116
]
Xiaojian Zhou commented on GEODE-7784:
--------------------------------------
This is the test issue.
When putAll or removeAll returns the list of version tag. It's not guaranteed
to be ordered. We just used an "ArrayList". It's expensive to guarantee the
order for each putAll/removeAll and customer does not request it either.
In that test, occasionally the returned arraylist is misordered, such as the
reproduced one. But
assertThat(versions.getVersionTags()).isEqualTo(versionsAfterRetry.getVersionTags());
expects the arraylist to be the same order.
I can easy reproduce it by adding following code:
List<VersionTag> list1 = versions.getVersionTags();
List<VersionTag> list2 = versionsAfterRetry.getVersionTags();
assertThat(list1).isEqualTo(list2);
VersionTag tag1 = list1.remove(0);
list1.add(tag1);
System.out.println("GGG1:"+list1);
System.out.println("GGG2:"+list2);
assertThat(list1).isEqualTo(list2);
So the fix should be to compare the list size, then compare item in the list
one by one.
> Region entry has wrong RVV version after removeAll retried
> ----------------------------------------------------------
>
> Key: GEODE-7784
> URL: https://issues.apache.org/jira/browse/GEODE-7784
> Project: Geode
> Issue Type: Bug
> Components: core
> Reporter: Kirk Lund
> Assignee: Jianxia Chen
> Priority: Major
> Labels: ConcurrencyChecksEnabled, GeodeCommons, RVV, removeAll
>
> I believe this is a product bug that
> PutAllClientServerDistributedTest.shouldReturnVersionTagOfTombstoneVersionWhenRemoveAllRetried
> intermittently reproduces.
> The test fails because rv1 is discovered instead of expected rv12 during
> validation:
> {noformat}
> {v2; rv12]; mbr=782257b7f21e4b...> but was:<[{v2; rv1[1;
> mbr=782257b7f21e4b70-910600606032139d; time=1581098169625; remote},
> {noformat}
> Here's the full failure stack trace:
> {noformat}
> org.apache.geode.internal.cache.PutAllClientServerDistributedTest >
> shouldReturnVersionTagOfTombstoneVersionWhenRemoveAllRetried FAILED
> org.junit.ComparisonFailure: expected:<[{v2; rv1[5;
> mbr=782257b7f21e4b70-910600606032139d; time=1581098169624; remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169624;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169624;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169624;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169624;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv12]; mbr=782257b7f21e4b...> but was:<[{v2; rv1[1;
> mbr=782257b7f21e4b70-910600606032139d; time=1581098169625; remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169624;
> remote},
> {v2; rv11; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169624;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv12; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169624;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv13; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169624;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv14; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169624;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv15; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv16; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv17; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv18; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv19; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169627;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169626;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169625;
> remote},
> {v2; rv20; mbr=782257b7f21e4b70-910600606032139d; time=1581098169628;
> remote},
> {v2; rv20]; mbr=782257b7f21e4b...>
> at
> jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
> at
> jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
> at
> jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
> at
> org.apache.geode.internal.cache.PutAllClientServerDistributedTest.shouldReturnVersionTagOfTombstoneVersionWhenRemoveAllRetried(PutAllClientServerDistributedTest.java:484)
> {noformat}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
