Merge pull request #15318 from serathius/linearizability-duplicate-revision

Allow multiple operations with same mod revisions, but forbid …
This commit is contained in:
Marek Siarkowicz 2023-02-14 17:03:47 +01:00 committed by GitHub
commit c60e5954b3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -99,10 +99,13 @@ func validateMemberWatchResponses(t *testing.T, responses []watchResponse, expec
if resp.Header.Revision == lastHeadRevision && len(resp.Events) != 0 {
t.Errorf("Got two non-empty responses about same revision")
}
if len(resp.Events) > 0 && resp.Events[0].Kv.ModRevision <= lastEventModRevision {
t.Errorf("Events with same revision should not be split between multiple responses, lastResponseModRevision: %d, firstEventModRevision: %d", lastEventModRevision, resp.Events[0].Kv.ModRevision)
}
for _, event := range resp.Events {
if event.Kv.ModRevision != lastEventModRevision+1 {
t.Errorf("Expect event revision to grow by 1, last: %d, mod: %d", lastEventModRevision, event.Kv.ModRevision)
if event.Kv.ModRevision != lastEventModRevision && event.Kv.ModRevision != lastEventModRevision+1 {
t.Errorf("Event mod revision should stay the same or increment by one, last: %d, current: %d", lastEventModRevision, event.Kv.ModRevision)
}
lastEventModRevision = event.Kv.ModRevision
}