gortiz commented on code in PR #14727:
URL: https://github.com/apache/pinot/pull/14727#discussion_r1910207344
##########
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/GroupByDataTableReducer.java:
##########
@@ -140,9 +141,12 @@ public void reduceAndSetResults(String tableName,
DataSchema dataSchema,
* @param brokerMetrics broker metrics (meters)
* @throws TimeoutException If unable complete within timeout.
*/
- private void reduceResult(BrokerResponseNative brokerResponseNative,
DataSchema dataSchema,
- Collection<DataTable> dataTables, DataTableReducerContext
reducerContext, String rawTableName,
- BrokerMetrics brokerMetrics)
+ private void reduceResult(BrokerResponseNative brokerResponseNative,
+ DataSchema dataSchema,
+ Collection<DataTable> dataTables,
+ DataTableReducerContext reducerContext,
+ String rawTableName,
Review Comment:
nit: another place where alignment is different than expected.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
