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

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

commit c9c99935416a0ac1c929ddd6dfad34344961a5df
Merge: 34c329c 7513626
Author: Mike Miller <mmil...@apache.org>
AuthorDate: Wed Feb 12 08:45:08 2020 -0500

    Merge branch '1.9'

 server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java | 2 ++
 1 file changed, 2 insertions(+)

diff --cc 
server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java
index 2e720f9,221bb4c..9457067
--- 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
@@@ -133,11 -183,13 +133,13 @@@ public class VolumeUtil 
     * configured in instance.volumes.replacements. Second, if a tablet dir is 
no longer configured
     * for use it chooses a new tablet directory.
     */
 -  public static TabletFiles updateTabletVolumes(AccumuloServerContext 
context, ZooLock zooLock,
 -      VolumeManager vm, KeyExtent extent, TabletFiles tabletFiles, boolean 
replicate)
 -      throws IOException {
 -    List<Pair<Path,Path>> replacements = 
ServerConstants.getVolumeReplacements();
 +  public static TabletFiles updateTabletVolumes(ServerContext context, 
ZooLock zooLock,
 +      KeyExtent extent, TabletFiles tabletFiles, boolean replicate) {
 +    List<Pair<Path,Path>> replacements =
 +        ServerConstants.getVolumeReplacements(context.getConfiguration(), 
context.getHadoopConf());
+     if (replacements.isEmpty())
+       return tabletFiles;
 -    log.trace("Using volume replacements: " + replacements);
 +    log.trace("Using volume replacements: {}", replacements);
  
      List<LogEntry> logsToRemove = new ArrayList<>();
      List<LogEntry> logsToAdd = new ArrayList<>();

Reply via email to