Update server/src/main/java/io/deephaven/server/table/ops/SortTableGrpcImpl.java

cpwright · Copilot · web-flow · commit 8c20b3d9b1d7 · 2026-04-24T09:24:43.000-04:00
Co-authored-by: Copilot &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/server/src/main/java/io/deephaven/server/table/ops/SortTableGrpcImpl.java b/server/src/main/java/io/deephaven/server/table/ops/SortTableGrpcImpl.java
@@ -70,7 +70,7 @@ public Table create(final SortTableRequest request,
         }
 
         // This loop does two optimizations:
-        // 1. Consolidate all sorts into a SortPair array to only call one sort on the table
+        // 1. Consolidate all sorts into a List<SortColumn> so the table is sorted only once
         // 2. Move all the reverses to the back:
         // - For an odd number of reverses only call one reverse
         // - For an even number of reverses do not call reverse (they cancel out)

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ public Table create(final SortTableRequest request,`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`// This loop does two optimizations:`
`73`		`- // 1. Consolidate all sorts into a SortPair array to only call one sort on the table`
	`73`	`+ // 1. Consolidate all sorts into a List<SortColumn> so the table is sorted only once`
`74`	`74`	`// 2. Move all the reverses to the back:`
`75`	`75`	`// - For an odd number of reverses only call one reverse`
`76`	`76`	`// - For an even number of reverses do not call reverse (they cancel out)`