You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ed...@apache.org on 2021/12/08 14:50:37 UTC

[accumulo] branch main updated: reduce root table mutation logging to single line (#2376)

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

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


The following commit(s) were added to refs/heads/main by this push:
     new 2fd40dd  reduce root table mutation logging to single line (#2376)
2fd40dd is described below

commit 2fd40dd8fd68c94bad6fb887d6b1cd99d39481c3
Author: EdColeman <de...@etcoleman.com>
AuthorDate: Wed Dec 8 09:50:26 2021 -0500

    reduce root table mutation logging to single line (#2376)
    
    Co-authored-by: Ed Coleman etcoleman <ed...@apache.org>
---
 .../org/apache/accumulo/core/metadata/schema/RootTabletMetadata.java   | 2 +-
 .../org/apache/accumulo/server/metadata/RootTabletMutatorImpl.java     | 3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/core/src/main/java/org/apache/accumulo/core/metadata/schema/RootTabletMetadata.java b/core/src/main/java/org/apache/accumulo/core/metadata/schema/RootTabletMetadata.java
index 81d204e..c62b131 100644
--- a/core/src/main/java/org/apache/accumulo/core/metadata/schema/RootTabletMetadata.java
+++ b/core/src/main/java/org/apache/accumulo/core/metadata/schema/RootTabletMetadata.java
@@ -52,7 +52,7 @@ import com.google.gson.GsonBuilder;
  */
 public class RootTabletMetadata {
 
-  private static final Gson GSON = new GsonBuilder().setPrettyPrinting().create();
+  private static final Gson GSON = new GsonBuilder().create();
 
   private static final ByteSequence CURR_LOC_FAM =
       new ArrayByteSequence(CurrentLocationColumnFamily.STR_NAME);
diff --git a/server/base/src/main/java/org/apache/accumulo/server/metadata/RootTabletMutatorImpl.java b/server/base/src/main/java/org/apache/accumulo/server/metadata/RootTabletMutatorImpl.java
index 4779dc5..7681b59 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/metadata/RootTabletMutatorImpl.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/metadata/RootTabletMutatorImpl.java
@@ -103,11 +103,10 @@ public class RootTabletMutatorImpl extends TabletMutatorBase implements Ample.Ta
       // TODO examine implementation of getZooReaderWriter().mutate()
       context.getZooReaderWriter().mutateOrCreate(zpath, new byte[0], currVal -> {
         String currJson = new String(currVal, UTF_8);
-        log.debug("Before mutating : {}, ", currJson);
         var rtm = RootTabletMetadata.fromJson(currJson);
         rtm.update(mutation);
         String newJson = rtm.toJson();
-        log.debug("After mutating : {} ", newJson);
+        log.debug("mutation: from:[{}] to: [{}]", currJson, newJson);
         return newJson.getBytes(UTF_8);
       });