bziobrowski commented on code in PR #14727:
URL: https://github.com/apache/pinot/pull/14727#discussion_r1910344614
##########
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/BrokerReduceService.java:
##########
@@ -59,7 +59,8 @@ public BrokerReduceService(PinotConfiguration config) {
}
public BrokerResponseNative reduceOnDataTable(BrokerRequest brokerRequest,
BrokerRequest serverBrokerRequest,
- Map<ServerRoutingInstance, DataTable> dataTableMap, long
reduceTimeOutMs, BrokerMetrics brokerMetrics) {
+ Map<ServerRoutingInstance,
DataTable> dataTableMap,
+ long reduceTimeOutMs,
BrokerMetrics brokerMetrics) {
Review Comment:
Reverted.
##########
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:
Reverted.
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
