arvindshmicrosoft commented on a change in pull request #1555: Fix idempotency
bug in importtable
URL: https://github.com/apache/accumulo/pull/1555#discussion_r390540627
##########
File path:
server/master/src/main/java/org/apache/accumulo/master/tableOps/MoveExportedFiles.java
##########
@@ -48,17 +49,22 @@
Map<String,String> fileNameMappings =
PopulateMetadataTable.readMappingFile(fs, tableInfo);
+ FileStatus[] exportedFiles = fs.listStatus(new
Path(tableInfo.exportDir));
+ FileStatus[] importedFiles = fs.listStatus(new
Path(tableInfo.importDir));
+
Review comment:
Thanks for the tip! I used the
[guava-beta-checker](https://github.com/google/guava-beta-checker) plugin to
quickly establish if there is any new impact introduced by the usage of Sets.
Thankfully, there is none, but in the process, I noticed there are many other
@Beta APIs used by previously existing code which are flagged by the plugin.
For example, the use of Stopwatch and RateLimiter classes (and others) is
flagged. I think that is a separate cleanup project on its own!
----------------------------------------------------------------
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
