ctubbsii commented on a change in pull request #1579: Closes #1576: Update 
importTable with directory list as Set<String>
URL: https://github.com/apache/accumulo/pull/1579#discussion_r405591496
 
 

 ##########
 File path: 
core/src/test/java/org/apache/accumulo/core/clientImpl/TableOperationsHelperTest.java
 ##########
 @@ -199,6 +199,9 @@ public void clearLocatorCache(String tableName) {}
     @Override
     public void importTable(String tableName, String exportDir) {}
 
 Review comment:
   If you're using a default method in the interface, the corresponding methods 
can be deleted from the implementing classes. I would check all subclasses and 
remove any that aren't needed.

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to