This is an automated email from the ASF dual-hosted git repository. dcapwell pushed a commit to branch trunk in repository https://gitbox.apache.org/repos/asf/cassandra-accord.git
The following commit(s) were added to refs/heads/trunk by this push: new 8b4f389 AccordGens.rangeDeps did not enforce unique ranges, which caused tests to fail 8b4f389 is described below commit 8b4f3895cb926f937450676b1db2e23d01a8b820 Author: David Capwell <dcapw...@apache.org> AuthorDate: Mon Apr 1 10:31:33 2024 -0700 AccordGens.rangeDeps did not enforce unique ranges, which caused tests to fail --- accord-core/src/test/java/accord/utils/AccordGens.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/accord-core/src/test/java/accord/utils/AccordGens.java b/accord-core/src/test/java/accord/utils/AccordGens.java index d87c2d5..971c70e 100644 --- a/accord-core/src/test/java/accord/utils/AccordGens.java +++ b/accord-core/src/test/java/accord/utils/AccordGens.java @@ -403,9 +403,10 @@ public class AccordGens return rs -> { if (rs.decide(emptyProb)) return RangeDeps.NONE; RangeDeps.Builder builder = RangeDeps.builder(); - for (int i = 0, numKeys = rs.nextInt(1, 10); i < numKeys; i++) + List<? extends Range> uniqRanges = Gens.lists(rangeGen).uniqueBestEffort().ofSize(rs.nextInt(1, 10)).next(rs); + for (Range range : uniqRanges) { - builder.nextKey(rangeGen.next(rs)); + builder.nextKey(range); for (int j = 0, numTxn = rs.nextInt(1, 10); j < numTxn; j++) builder.add(idGen.next(rs)); } --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org