milleruntime commented on issue #1620: URL: https://github.com/apache/accumulo/issues/1620#issuecomment-645459605
> There is a possibly related change for import table #1569 > > I think it would be nice to have a VolumeManager.bulkRename() method for renaming lots of files. Then we can have a single implementation of this in the code to fix and maintain. I started looking into this and noticed some subtle differences in debugging and exception handling betwen tableImport and bulk import use of the thread pool. I am not sure if the behavior is specific to the actual FATE operation or it is just how the code evolved independently. For example, BulkImportMove has this [check](https://github.com/apache/accumulo/blob/15a95feebc74bd10e5ba73d4c2ecfc726435bc55/server/master/src/main/java/org/apache/accumulo/master/tableOps/bulkVer2/BulkImportMove.java#L131) for existence while the [MoveExportedFiles](https://github.com/apache/accumulo/blob/27acec198401942fbd5d90edfea71ac859533470/server/master/src/main/java/org/apache/accumulo/master/tableOps/tableImport/MoveExportedFiles.java#L117) does not. ---------------------------------------------------------------- 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]
