keith-turner commented on a change in pull request #2098:
URL: https://github.com/apache/accumulo/pull/2098#discussion_r631126872



##########
File path: 
test/src/main/java/org/apache/accumulo/test/functional/FunctionalTestUtils.java
##########
@@ -81,6 +83,19 @@ public static int countRFiles(AccumuloClient c, String 
tableName) throws Excepti
     }
   }
 
+  public static List<String> getRFileNames(AccumuloClient c, String tableName) 
throws Exception {

Review comment:
       This method name seems off.  Seems like its returning paths or URIs and 
not names.  Also could possibly use Ample for the implementation, but not quite 
sure. It may not be easy to use Ample unless you have a ServerContext.




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


Reply via email to