mqliang commented on a change in pull request #7724:
URL: https://github.com/apache/pinot/pull/7724#discussion_r751791565
##########
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:
@Jackie-Jiang I just double checked:
* delete the `Pinot Style` from my `Intellj->Editor -> Code Style -> Java`
* import codestyle-intellij.xml from pinot/config again.
* reformat the file using the new `Pinot Style`
Nothing changed. Which means the reformatting in the PR followed `Pinot
Style`, while the old file not.
Could you please do me a favor by importing the latest `Pinot Style` and
perform `Reformat code` against this file using your IntellJ (using the current
master branch)?
* If your file does not change, then maybe my IntellJ have some issues, I
will revert those reformatting change
* If your file also get changed, let's keep those reformatting change in the
PR.
Does it sounds good?
--
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]
