Align fuzz probabilities order

questdb · bluestreak01 · Jan 9, 2025 · Dec 9, 2024 · Dec 9, 2024 · Dec 9, 2024
commit 6011d442927608d9a780ee5825844b3334d46554
diff --git a/core/src/test/java/io/questdb/test/cairo/fuzz/AbstractFuzzTest.java b/core/src/test/java/io/questdb/test/cairo/fuzz/AbstractFuzzTest.java
@@ -120,10 +120,10 @@ protected void fullRandomFuzz(Rnd rnd) throws Exception {
                 rnd.nextDouble(),
                 rnd.nextDouble(),
                 rnd.nextDouble(),
+                0.01,
                 rnd.nextDouble(),
                 0.1 * rnd.nextDouble(),
-                rnd.nextDouble(),
-                0.01
+                rnd.nextDouble()
         );
 
         fuzzer.setFuzzCounts(
@@ -247,7 +247,8 @@ protected void setFuzzProbabilities(
     ) {
         fuzzer.setFuzzProbabilities(cancelRowsProb, notSetProb, nullSetProb, rollbackProb,
                 colAddProb, colRemoveProb, colRenameProb, colTypeChangeProb, dataAddProb,
-                partitionDropProb, truncateProb, tableDropProb, equalTsRowsProb);
+                equalTsRowsProb, partitionDropProb, truncateProb, tableDropProb
+        );
     }
 
     protected void setFuzzProperties(long maxApplyTimePerTable, long splitPartitionThreshold, int o3PartitionSplitMaxCount) {

diff --git a/core/src/test/java/io/questdb/test/cairo/fuzz/CheckpointFuzzTest.java b/core/src/test/java/io/questdb/test/cairo/fuzz/CheckpointFuzzTest.java
@@ -82,10 +82,10 @@ public void testCheckpointEjectedWalApply() throws Exception {
                 0,
                 0,
                 1,
+                0.5,
                 0.0,
                 0,
-                0,
-                0.5
+                0
         );
 
         fuzzer.setFuzzCounts(
@@ -122,10 +122,10 @@ public void testCheckpointFrequentTableDrop() throws Exception {
                 0,
                 0,
                 0.1,
+                0.5,
                 0.0,
                 0,
-                1,
-                0.5
+                1
         );
 
         fuzzer.setFuzzCounts(
@@ -259,9 +259,10 @@ private void fullFuzz(Rnd rnd) {
                 rnd.nextDouble(),
                 rnd.nextDouble(),
                 rnd.nextDouble(),
+                0.01,
                 0.0,
                 0.1 * rnd.nextDouble(),
-                rnd.nextDouble(), 0.01
+                rnd.nextDouble()
         );
 
         fuzzer.setFuzzCounts(

diff --git a/core/src/test/java/io/questdb/test/cairo/fuzz/FuzzRunner.java b/core/src/test/java/io/questdb/test/cairo/fuzz/FuzzRunner.java
@@ -523,10 +523,10 @@ public void setFuzzProbabilities(
             double colRenameProb,
             double colTypeChangeProb,
             double dataAddProb,
+            double equalTsRowsProb,
             double partitionDropProb,
             double truncateProb,
-            double tableDropProb,
-            double equalTsRowsProb
+            double tableDropProb
     ) {
         this.cancelRowsProb = cancelRowsProb;
         this.notSetProb = notSetProb;
@@ -962,10 +962,10 @@ protected void runFuzz(Rnd rnd, String tableNameBase, int tableCount, boolean ra
                         rnd.nextDouble(),
                         rnd.nextDouble(),
                         rnd.nextDouble(),
+                        0.01,
                         0.0,
                         0.1 * rnd.nextDouble(),
-                        rnd.nextDouble(),
-                        0.01
+                        rnd.nextDouble()
                 );
             }
             if (randomiseCounts) {

diff --git a/core/src/test/java/io/questdb/test/cairo/fuzz/ReaderReloadFuzzTest.java b/core/src/test/java/io/questdb/test/cairo/fuzz/ReaderReloadFuzzTest.java
@@ -54,10 +54,10 @@ public void testReaderDoesNotReopenFilesFuzz() throws Exception {
                 0,
                 0,
                 0.2,
+                0.5,
                 0.0,
                 0,
-                0,
-                0.5
+                0
         );
 
         // Basic load to keep the test lite, we just want to fuzz different transaction types, not intensive inserting
-Original file line number
+Diff line change
@@ Expand Up @@
 ,
 ,
 .2,
+.5,
 .0,
 ,
-,
-.5
             );
             // Basic load to keep the test lite, we just want to fuzz different transaction types, not intensive inserting
@@ Expand Down @@