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

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

commit 20bfabab5b1e07437a6f54ec4a51e3228ae9e4df
Merge: eb4ff2eec9 a6f22a32ce
Author: Christopher Tubbs <ctubb...@apache.org>
AuthorDate: Thu Dec 21 05:28:05 2023 -0500

    Merge branch 'main' into elasticity

 .github/ISSUE_TEMPLATE/post_vote_checklist.md      |  5 +-
 .github/workflows/maven-full-its.yaml              | 18 ++--
 .github/workflows/maven-on-demand.yaml             |  8 +-
 .github/workflows/maven.yaml                       | 21 +++--
 .github/workflows/scripts.yaml                     |  6 +-
 assemble/pom.xml                                   |  5 --
 core/pom.xml                                       |  4 -
 .../accumulo/core/tabletserver/log/LogEntry.java   | 25 ++++--
 .../org/apache/accumulo/core/trace/TraceUtil.java  |  8 +-
 .../core/tabletserver/log/LogEntryTest.java        | 13 +--
 pom.xml                                            | 99 ++++++++--------------
 .../org/apache/accumulo/server/fs/VolumeUtil.java  |  2 +-
 .../accumulo/server/util/RestoreZookeeper.java     |  7 +-
 .../accumulo/gc/GarbageCollectWriteAheadLogs.java  | 11 +--
 .../accumulo/manager/recovery/RecoveryManager.java | 15 ++--
 server/monitor/pom.xml                             |  6 ++
 16 files changed, 112 insertions(+), 141 deletions(-)

diff --cc 
core/src/main/java/org/apache/accumulo/core/tabletserver/log/LogEntry.java
index 3343d750fc,6bd5ba57b4..57820065b0
--- a/core/src/main/java/org/apache/accumulo/core/tabletserver/log/LogEntry.java
+++ b/core/src/main/java/org/apache/accumulo/core/tabletserver/log/LogEntry.java
@@@ -153,7 -158,12 +158,11 @@@ public final class LogEntry 
    /**
     * Get the Text that should be used as the column qualifier to store this 
as a metadata entry.
     */
 -  @VisibleForTesting
 -  Text getColumnQualifier() {
 +  public Text getColumnQualifier() {
+     return columnQualifier == null ? newCQ() : new Text(columnQualifier);
+   }
+ 
+   private Text newCQ() {
      return new Text("-/" + getPath());
    }
  
diff --cc 
server/gc/src/main/java/org/apache/accumulo/gc/GarbageCollectWriteAheadLogs.java
index 4b4d35918b,d6f3c63851..7e55a5274c
--- 
a/server/gc/src/main/java/org/apache/accumulo/gc/GarbageCollectWriteAheadLogs.java
+++ 
b/server/gc/src/main/java/org/apache/accumulo/gc/GarbageCollectWriteAheadLogs.java
@@@ -294,9 -283,8 +290,8 @@@ public class GarbageCollectWriteAheadLo
        }
        // Tablet is being recovered and has WAL references, remove all the 
WALs for the dead server
        // that made the WALs.
-       for (LogEntry wals : tabletMetadata.getLogs()) {
-         String wal = wals.getPath();
-         UUID walUUID = path2uuid(new Path(wal));
 -      for (LogEntry wal : state.walogs) {
++      for (LogEntry wal : tabletMetadata.getLogs()) {
+         UUID walUUID = wal.getUniqueID();
          TServerInstance dead = result.get(walUUID);
          // There's a reference to a log file, so skip that server's logs
          Set<UUID> idsToIgnore = candidates.remove(dead);
diff --cc 
server/manager/src/main/java/org/apache/accumulo/manager/recovery/RecoveryManager.java
index 7c66d84a2f,530fd6e1c7..7ed9d845b6
--- 
a/server/manager/src/main/java/org/apache/accumulo/manager/recovery/RecoveryManager.java
+++ 
b/server/manager/src/main/java/org/apache/accumulo/manager/recovery/RecoveryManager.java
@@@ -39,10 -39,8 +39,9 @@@ import org.apache.accumulo.core.conf.Pr
  import org.apache.accumulo.core.dataImpl.KeyExtent;
  import org.apache.accumulo.core.fate.zookeeper.ZooCache;
  import org.apache.accumulo.core.tabletserver.log.LogEntry;
 +import org.apache.accumulo.core.util.cache.Caches.CacheName;
  import org.apache.accumulo.core.util.threads.ThreadPools;
  import org.apache.accumulo.manager.Manager;
- import org.apache.accumulo.server.fs.VolumeManager.FileType;
  import org.apache.accumulo.server.fs.VolumeUtil;
  import org.apache.accumulo.server.log.SortedLogState;
  import org.apache.accumulo.server.manager.recovery.HadoopLogCloser;

Reply via email to