[accumulo] branch main updated: Make it easier to debug GcMetricsIT failures

[ct...@apache.org]

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

ctubbsii 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 391cfbf  Make it easier to debug GcMetricsIT failures
391cfbf is described below

commit 391cfbff3a29f735fb80ddd0695f50b06dd72f28
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Thu Dec 3 09:02:52 2020 -0500

    Make it easier to debug GcMetricsIT failures
    
    Use debug instead of trace for GcMetricsIT logs
---
 .../java/org/apache/accumulo/test/functional/GcMetricsIT.java | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/test/src/main/java/org/apache/accumulo/test/functional/GcMetricsIT.java b/test/src/main/java/org/apache/accumulo/test/functional/GcMetricsIT.java
index 9cfaba5..94884b4 100644
--- a/test/src/main/java/org/apache/accumulo/test/functional/GcMetricsIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/functional/GcMetricsIT.java
@@ -83,8 +83,7 @@ public class GcMetricsIT extends ConfigurableMacBase {
       return;
     }
 
-    // log.trace("Client started, properties:{}", accumuloClient.properties());
-    log.trace("Client started, properties:{}", accumuloClient.properties());
+    log.debug("Client started, properties:{}", accumuloClient.properties());
 
     MetricsFileTailer gcTail = new MetricsFileTailer("accumulo.sink.file-gc");
     Thread t1 = new Thread(gcTail);
@@ -102,8 +101,8 @@ public class GcMetricsIT extends ConfigurableMacBase {
 
       Map<String,Long> firstSeenMap = parseLine(firstUpdate.getLine());
 
-      log.trace("L:{}", firstUpdate.getLine());
-      log.trace("M:{}", firstSeenMap);
+      log.debug("L:{}", firstUpdate.getLine());
+      log.debug("M:{}", firstSeenMap);
 
       assertTrue(lookForExpectedKeys(firstSeenMap));
       sanity(updateTimestamp, firstSeenMap);
@@ -115,7 +114,7 @@ public class GcMetricsIT extends ConfigurableMacBase {
       Map<String,Long> updateSeenMap = parseLine(nextUpdate.getLine());
 
       log.debug("Line received:{}", nextUpdate.getLine());
-      log.trace("Mapped values:{}", updateSeenMap);
+      log.debug("Mapped values:{}", updateSeenMap);
 
       assertTrue(lookForExpectedKeys(updateSeenMap));
       sanity(updateTimestamp, updateSeenMap);
@@ -264,7 +263,7 @@ public class GcMetricsIT extends ConfigurableMacBase {
         var timestamp = Long.parseLong(m.group("timestamp"));
         return timestamp > prevTimestamp;
       } catch (NumberFormatException ex) {
-        log.trace("Could not parse timestamp from line '{}", line);
+        log.debug("Could not parse timestamp from line '{}", line);
         return false;
       }
     }