virajjasani commented on a change in pull request #1797: URL: https://github.com/apache/hbase/pull/1797#discussion_r432290954
########## 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: Thanks @bsglz I just triggered one build. Let it go through and one result is out, will merge the PR. ---------------------------------------------------------------- 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