virajjasani commented on a change in pull request #1797:
URL: https://github.com/apache/hbase/pull/1797#discussion_r432284673
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/master/SplitLogManager.java
##########
@@ -122,9 +121,9 @@ public SplitLogManager(MasterServices master, Configuration
conf)
throws IOException {
this.server = master;
this.conf = conf;
- // Get Server Thread name. Sometimes the Server is mocked so may not
implement HasThread.
+ // Get Server Thread name. Sometimes the Server is mocked so may not
implement Thread.
// For example, in tests.
- String name = master instanceof HasThread? ((HasThread)master).getName():
+ String name = master instanceof Thread? ((Thread)master).getName():
Review comment:
However, there is no harm w/o this change as Stack said. Maybe good to
take this clean-up in separate Jira if you would like.
Will merge this PR in a while.
----------------------------------------------------------------
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:
[email protected]
