Jackie-Jiang commented on a change in pull request #7724:
URL: https://github.com/apache/pinot/pull/7724#discussion_r751806904
##########
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java
##########
@@ -160,9 +160,9 @@ public BaseBrokerRequestHandler(PinotConfiguration config,
RoutingManager routin
_numDroppedLogRateLimiter = RateLimiter.create(1.0);
_brokerReduceService = new BrokerReduceService(_config);
- LOGGER.info(
- "Broker Id: {}, timeout: {}ms, query response limit: {}, query log
length: {}, query log max rate: {}qps",
- _brokerId, _brokerTimeoutMs, _queryResponseLimit, _queryLogLength,
_queryLogRateLimiter.getRate());
+ LOGGER
Review comment:
I actually tried that before leaving the comment and the current master
file didn't change.
In the `Editor -> Code Style -> Java` setting, under `Wrapping and Braces ->
Keep when reformatting`, can you uncheck the `Line breaks` and try reformatting
again? Current setting preserves the line break when reformatting because in
certain cases we want to break the line in a different way.
--
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]
