ctubbsii opened a new pull request #1553: Simplify some VolumeManager tooling URL: https://github.com/apache/accumulo/pull/1553 * Replace getVolumeByPath(p).getFileSystem() with getFileSystemByPath(p) (no caller used getVolumeByPath without immediately calling getFileSystem) * Remove redundant null checks for path argument by pushing down into getFileSystemByPath method * Remove unneeded NonConfiguredVolume class and test * Remove unneeded (and unused) VolumeManager.getContentSummary * Rename create method with overwrite capability to make API more clear * Retain all exceptions when closing filesystems (as suppressed) * Simplify/shorten implementations for VolumeManagerImpl methods * Make VolumeManager AutoCloseable (it already had a close method) * Fix tests (ZooLockTest -> ZooLockIT, and numerous incorrect usages of TemporaryFolder)
---------------------------------------------------------------- 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
