saintstack commented on a change in pull request #1797:
URL: https://github.com/apache/hbase/pull/1797#discussion_r432095352
##########
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:
This is not needed w/ this patch?
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java
##########
@@ -1908,9 +1907,9 @@ protected void queueAssign(final RegionStateNode
regionNode) {
}
private void startAssignmentThread() {
- // 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:
Ditto
----------------------------------------------------------------
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]
