Fix random params in testSoftDeletesRetentionLock (elastic#38114)

Since elastic#37992 the retainingSequenceNumber is initialized with 0 while the global checkpoint can be -1. Relates elastic#37992
dnhatn · Feb 1, 2019 · 024647b · 024647b
1 parent 7077cb3
commit 024647b
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/server/src/test/java/org/elasticsearch/index/engine/SoftDeletesPolicyTests.java b/server/src/test/java/org/elasticsearch/index/engine/SoftDeletesPolicyTests.java
@@ -74,7 +74,7 @@ public void testSoftDeletesRetentionLock() {
             // Advances the global checkpoint and the local checkpoint of a safe commit
             globalCheckpoint.addAndGet(between(0, 1000));
             for (final AtomicLong retainingSequenceNumber : retainingSequenceNumbers) {
-                retainingSequenceNumber.set(randomLongBetween(retainingSequenceNumber.get(), globalCheckpoint.get()));
+                retainingSequenceNumber.set(randomLongBetween(retainingSequenceNumber.get(), Math.max(globalCheckpoint.get(), 0L)));
             }
             safeCommitCheckpoint = randomLongBetween(safeCommitCheckpoint, globalCheckpoint.get());
             policy.setLocalCheckpointOfSafeCommit(safeCommitCheckpoint);