Yuki Tawara created HBASE-20297:
-----------------------------------

             Summary: Remove Logger#isErrorEnabled check
                 Key: HBASE-20297
                 URL: https://issues.apache.org/jira/browse/HBASE-20297
             Project: HBase
          Issue Type: Improvement
            Reporter: Yuki Tawara


Logger#isErrorEnabled check is unnecessary, so it should be removed.

I greped the code base and found 3 occurrences.
(The checks in Logging.scala seem necessary.)

{noformat}
./hbase-common/src/main/java/org/apache/hadoop/hbase/ScheduledChore.java: if 
(LOG.isErrorEnabled()) LOG.error("Caught error", t);
./hbase-spark/src/main/scala/org/apache/hadoop/hbase/spark/Logging.scala: if 
(log.isErrorEnabled) log.error(msg)
./hbase-spark/src/main/scala/org/apache/hadoop/hbase/spark/Logging.scala: if 
(log.isErrorEnabled) log.error(msg, throwable)
{noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to