accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [accumulo] drewfarris commented on a change in pull request #1060: Closes #1053 add multi-volume support to import table command.
Date Tue, 16 Apr 2019 12:33:46 GMT
drewfarris commented on a change in pull request #1060: Closes #1053 add multi-volume support
to import table command.
URL: https://github.com/apache/accumulo/pull/1060#discussion_r275772782
 
 

 ##########
 File path: test/src/main/java/org/apache/accumulo/test/ImportExportIT.java
 ##########
 @@ -119,23 +126,25 @@ public void testExportImportThenScan() throws Exception {
       FSDataInputStream is = fs.open(distcp);
       BufferedReader reader = new BufferedReader(new InputStreamReader(is));
 
-      // Copy each file that was exported to the import directory
+      // Copy each file that was exported to one of the imports directory
       String line;
+
       while ((line = reader.readLine()) != null) {
         Path p = new Path(line.substring(5));
         assertTrue("File doesn't exist: " + p, fs.exists(p));
-
+        Path importDir = importDirAry[r.nextInt(importDirAry.length)];
         Path dest = new Path(importDir, p.getName());
         assertFalse("Did not expect " + dest + " to exist", fs.exists(dest));
         FileUtil.copy(fs, p, fs, dest, false, fs.getConf());
       }
 
       reader.close();
 
-      log.info("Import dir: {}", Arrays.toString(fs.listStatus(importDir)));
+      log.info("Import dir A: {}", Arrays.toString(fs.listStatus(importDirA)));
 
 Review comment:
   Unfortunately not. I need to extend the test to properly do this.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message