mqliang commented on a change in pull request #6742:
URL: https://github.com/apache/incubator-pinot/pull/6742#discussion_r611806930
##########
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/reduce/BrokerReduceService.java
##########
@@ -55,15 +56,12 @@
@ThreadSafe
public class BrokerReduceService {
- private static final Logger LOGGER =
LoggerFactory.getLogger(BrokerReduceService.class);
-
- // brw -> Shorthand for broker reduce worker threads.
- private static final String REDUCE_THREAD_NAME_FORMAT = "brw-%d";
-
// Set the reducer priority higher than NORM but lower than MAX, because if
a query is complete
// we want to deserialize and return response as soon. This is the same as
server side 'pqr' threads.
protected static final int QUERY_RUNNER_THREAD_PRIORITY = 7;
-
+ private static final Logger LOGGER =
LoggerFactory.getLogger(BrokerReduceService.class);
Review comment:
This change is performed by Intellj reformat, so I's like to not change
it back.
--
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.
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
