You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ct...@apache.org on 2023/03/13 02:21:52 UTC

[accumulo] branch 2.1 updated: fixes unencountered bug in ample (#3237)

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

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


The following commit(s) were added to refs/heads/2.1 by this push:
     new c67a1dca5a fixes unencountered bug in ample (#3237)
c67a1dca5a is described below

commit c67a1dca5ab63b3b4891e89a8a42f9da6ac1e1c7
Author: Keith Turner <kt...@apache.org>
AuthorDate: Sun Mar 12 22:21:47 2023 -0400

    fixes unencountered bug in ample (#3237)
    
    Fixes a bug in Ample that no current code is encountering.  Ran into
    this bug while working on #3232 and trying to write new code that
    splits a tablet using conditional mutations. Fixing this bug in 2.1
    in case any new code is written that would encounter the bug.
---
 .../java/org/apache/accumulo/server/metadata/TabletMutatorBase.java    | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/server/base/src/main/java/org/apache/accumulo/server/metadata/TabletMutatorBase.java b/server/base/src/main/java/org/apache/accumulo/server/metadata/TabletMutatorBase.java
index d0a3f880d2..cc4db1371a 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/metadata/TabletMutatorBase.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/metadata/TabletMutatorBase.java
@@ -69,8 +69,7 @@ public abstract class TabletMutatorBase implements Ample.TabletMutator {
   @Override
   public Ample.TabletMutator putPrevEndRow(Text per) {
     Preconditions.checkState(updatesEnabled, "Cannot make updates after calling mutate.");
-    TabletColumnFamily.PREV_ROW_COLUMN.put(mutation,
-        TabletColumnFamily.encodePrevEndRow(extent.prevEndRow()));
+    TabletColumnFamily.PREV_ROW_COLUMN.put(mutation, TabletColumnFamily.encodePrevEndRow(per));
     return this;
   }