[GitHub] [pinot] richardstartin commented on a change in pull request #7450: Unify CombineOperator multi-threading logic

richardstartin commented on a change in pull request #7450: URL: https://github.com/apache/pinot/pull/7450#discussion_r712465217 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/combine/GroupByOrderByCombineOperator.java ## @@ -125,43 +123,44 @@ public Str

[GitHub] [pinot] richardstartin commented on a change in pull request #7450: Unify CombineOperator multi-threading logic

richardstartin commented on a change in pull request #7450: URL: https://github.com/apache/pinot/pull/7450#discussion_r712465217 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/combine/GroupByOrderByCombineOperator.java ## @@ -125,43 +123,44 @@ public Str

[GitHub] [pinot] richardstartin commented on a change in pull request #7450: Unify CombineOperator multi-threading logic

richardstartin commented on a change in pull request #7450: URL: https://github.com/apache/pinot/pull/7450#discussion_r712463571 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/combine/GroupByOrderByCombineOperator.java ## @@ -125,43 +123,44 @@ public Str

[GitHub] [pinot] richardstartin commented on a change in pull request #7450: Unify CombineOperator multi-threading logic

richardstartin commented on a change in pull request #7450: URL: https://github.com/apache/pinot/pull/7450#discussion_r711524613 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/streaming/StreamingSelectionOnlyCombineOperator.java ## @@ -71,30 +70,19 @@ pu

[GitHub] [pinot] richardstartin commented on a change in pull request #7450: Unify CombineOperator multi-threading logic

richardstartin commented on a change in pull request #7450: URL: https://github.com/apache/pinot/pull/7450#discussion_r711524613 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/streaming/StreamingSelectionOnlyCombineOperator.java ## @@ -71,30 +70,19 @@ pu

[GitHub] [pinot] richardstartin commented on a change in pull request #7450: Unify CombineOperator multi-threading logic

richardstartin commented on a change in pull request #7450: URL: https://github.com/apache/pinot/pull/7450#discussion_r711524176 ## File path: pinot-core/src/main/java/org/apache/pinot/core/operator/combine/GroupByOrderByCombineOperator.java ## @@ -125,43 +123,44 @@ public Str