polish code

Author: jan-elastic

x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/SampleOperator.java

@@ -27,38 +27,37 @@
 
 public class SampleOperator implements Operator {
 
-    private boolean finished;
+    public record Factory(double probability, int seed) implements OperatorFactory {
+
+        @Override
+        public SampleOperator get(DriverContext driverContext) {
+            return new SampleOperator(probability, seed);
+        }
+
+        @Override
+        public String describe() {
+            return "SampleOperator[probability = " + probability + ", seed = " + seed + "]";
+        }
+    }
+
     private final Deque<Page> outputPages;
     private final RandomSamplingQuery.RandomSamplingIterator randomSamplingIterator;
+    private boolean finished;
 
     private int pagesProcessed = 0;
     private int rowsReceived = 0;
     private int rowsEmitted = 0;
-
     private long collectNanos;
     private long emitNanos;
 
-    public SampleOperator(double probability, int seed) {
+    private SampleOperator(double probability, int seed) {
         finished = false;
         outputPages = new LinkedList<>();
         SplittableRandom random = new SplittableRandom(seed);
         randomSamplingIterator = new RandomSamplingQuery.RandomSamplingIterator(Integer.MAX_VALUE, probability, random::nextInt);
         randomSamplingIterator.nextDoc();
     }
 
-    public record Factory(double probability, int seed) implements OperatorFactory {
-
-        @Override
-        public SampleOperator get(DriverContext driverContext) {
-            return new SampleOperator(probability, seed);
-        }
-
-        @Override
-        public String describe() {
-            return "SampleOperator[probability = " + probability + ", seed = " + seed + "]";
-        }
-    }
-
     /**
      * whether the given operator can accept more input pages
      */