You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by mm...@apache.org on 2020/02/12 13:28:12 UTC

[accumulo] branch 1.9 updated: Optimize Tablet volume replacement (#1505)

This is an automated email from the ASF dual-hosted git repository.

mmiller pushed a commit to branch 1.9
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/1.9 by this push:
     new 7513626  Optimize Tablet volume replacement (#1505)
7513626 is described below

commit 7513626cc504e5d93760c4218868dcb2dd6d49a5
Author: Mike Miller <mm...@apache.org>
AuthorDate: Wed Feb 12 08:28:02 2020 -0500

    Optimize Tablet volume replacement (#1505)
    
    * Stop volume replacements from checking every file when not used
---
 server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java b/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java
index 4aea803..221bb4c 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java
@@ -187,6 +187,8 @@ public class VolumeUtil {
       VolumeManager vm, KeyExtent extent, TabletFiles tabletFiles, boolean replicate)
       throws IOException {
     List<Pair<Path,Path>> replacements = ServerConstants.getVolumeReplacements();
+    if (replacements.isEmpty())
+      return tabletFiles;
     log.trace("Using volume replacements: " + replacements);
 
     List<LogEntry> logsToRemove = new ArrayList<>();