saintstack commented on a change in pull request #1955:
URL: https://github.com/apache/hbase/pull/1955#discussion_r444294454
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
##########
@@ -2037,8 +2037,7 @@ private void startServices() throws IOException {
this.splitLogWorker = new SplitLogWorker(sinkConf, this,
this, walFactory);
splitLogWorker.start();
- } else {
- LOG.warn("SplitLogWorker Service NOT started; CoordinatedStateManager is
null");
+ LOG.debug("SplitLogWorker started");
Review comment:
Previous, we used to log when we did NOT start the splitlogworker
service. This goes against our usual pattern of only logging started services
(logging all the things we did not start would be an endless list -- smile)
I changed the log here so that it conforms with our usual pattern.
----------------------------------------------------------------
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
