comnetwork commented on code in PR #4768:
URL: https://github.com/apache/hbase/pull/4768#discussion_r965444994
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactSplit.java:
##########
@@ -312,14 +316,20 @@ public void switchCompaction(boolean onOrOff) {
if (onOrOff) {
// re-create executor pool if compactions are disabled.
if (!isCompactionsEnabled()) {
- LOG.info("Re-Initializing compactions because user switched on
compactions");
+ if (LOG.isInfoEnabled()) {
Review Comment:
In fact in this class ,some logs using the LOG.isInfoEnabled or
LOG.isDebugEnabled, and other logs using {{LOG.info}} or {{LOG.debug}}
directly. If you do not like this, I would revert it.
--
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]
