Remove dead code from BenchmarkGroupedTypedHistogram

Author: findepi

core/trino-main/src/test/java/io/trino/operator/aggregation/BenchmarkGroupedTypedHistogram.java

@@ -65,13 +65,6 @@ public static class Data
         //        @Param({"1", "5", "50"})
         @Param("32") // size of entries--we have no idea here, could be 8 long (common in anecdotal) or longer strings
         private int rowSize;
-        // these must be manually set in each class now; the mechanism to change and test was removed; the enum was kept in case we want to revisit. Retesting showed linear was superior
-        //        //        @Param({"LINEAR", "SUM_OF_COUNT", "SUM_OF_SQUARE"})
-//        @Param({"LINEAR"}) // found to be best, by about 10-15%
-//        private ProbeType mainProbeTyepe;
-//        //        @Param({"LINEAR", "SUM_OF_COUNT", "SUM_OF_SQUARE"})
-//        @Param({"LINEAR"}) // found to best
-//        private ProbeType valueStoreProbeType;
 
         private final Random random = new Random();
         private Page[] pages;
@@ -155,9 +148,4 @@ static void main()
                 .withOptions(optionsBuilder -> optionsBuilder.addProfiler(GCProfiler.class))
                 .run();
     }
-
-    public enum ProbeType
-    {
-        LINEAR, SUM_OF_COUNT, SUM_OF_SQUARE
-    }
 }

core/trino-spi/src/test/java/io/trino/spi/block/BenchmarkCopyPositions.java

@@ -65,11 +65,7 @@ public static class BenchmarkData
         @Param({"false", "true"})
         private boolean nullsAllowed;
 
-        @Param({
-                "GROUPED",
-                "SEQUENCE",
-                "RANDOM",
-        })
+        @Param
         private SelectedPositions selectedPositions;
 
         @Param({"VARCHAR", "ROW(BIGINT)"})