Chores: reorder util functions

Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>

Author: yuancu

opensearch/src/main/java/org/opensearch/sql/opensearch/storage/scan/CalciteLogicalIndexScan.java

@@ -309,6 +309,12 @@ public CalciteLogicalIndexScan pushDownSort(List<RelFieldCollation> collations)
     return null;
   }
 
+  private List<String> getCollationNames(List<RelFieldCollation> collations) {
+    return collations.stream()
+        .map(collation -> getRowType().getFieldNames().get(collation.getFieldIndex()))
+        .toList();
+  }
+
   /**
    * Check if the sort by collations contains any aggregators that are pushed down. E.g. In `stats
    * avg(age) as avg_age by state | sort avg_age`, the sort clause has `avg_age` which is an
@@ -327,10 +333,17 @@ private boolean hasAggregatorInSortBy(List<String> collations) {
         .reduce(false, Boolean::logicalOr);
   }
 
-  private List<String> getCollationNames(List<RelFieldCollation> collations) {
+  private static boolean isAnyCollationNameInAggregateOutput(
+      LogicalAggregate aggregate, List<String> collations) {
+    List<String> fieldNames = aggregate.getRowType().getFieldNames();
+    // The output fields of the aggregate are in the format of
+    // [...grouping fields, ...aggregator fields], so we set an offset to skip
+    // the grouping fields.
+    int groupOffset = aggregate.getGroupSet().cardinality();
+    List<String> fieldsWithoutGrouping = fieldNames.subList(groupOffset, fieldNames.size());
     return collations.stream()
-        .map(collation -> getRowType().getFieldNames().get(collation.getFieldIndex()))
-        .toList();
+        .map(fieldsWithoutGrouping::contains)
+        .reduce(false, Boolean::logicalOr);
   }
 
   /**
@@ -357,19 +370,6 @@ private static List<RelFieldCollation> mergeCollations(
     return new ArrayList<>(mergedCollations.values());
   }
 
-  private static boolean isAnyCollationNameInAggregateOutput(
-      LogicalAggregate aggregate, List<String> collations) {
-    List<String> fieldNames = aggregate.getRowType().getFieldNames();
-    // The output fields of the aggregate are in the format of
-    // [...grouping fields, ...aggregator fields], so we set an offset to skip
-    // the grouping fields.
-    int groupOffset = aggregate.getGroupSet().cardinality();
-    List<String> fieldsWithoutGrouping = fieldNames.subList(groupOffset, fieldNames.size());
-    return collations.stream()
-        .map(fieldsWithoutGrouping::contains)
-        .reduce(false, Boolean::logicalOr);
-  }
-
   /**
    * Create a new {@link PushDownContext} without the collation action.
    *