commit 3df488ed826664edbe397b805b31656cd95c1e4d
author Wyatt Hepler <hepler@google.com> Fri Dec 16 20:45:40 2022 +0000
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Dec 16 20:45:40 2022 +0000
tree 3ecd9863b65d22addfdd4686e7e9fc0caf55d3da
parent dcba7d5b0935b0aac12ae8a8e5b3d60d50173d67

pw_transfer: Reduce Java client log verbosity

Disable a few logs and lower the log level of some others.

Change-Id: Ie12d56830a9a08375010cf427aebe1736d5daad9
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/124671
Pigweed-Auto-Submit: Wyatt Hepler <hepler@google.com>
Reviewed-by: Alexei Frolov <frolv@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed.google.com.iam.gserviceaccount.com>

pw_transfer/java/main/dev/pigweed/pw_transfer/ReadTransfer.java

diff --git a/pw_transfer/java/main/dev/pigweed/pw_transfer/ReadTransfer.java b/pw_transfer/java/main/dev/pigweed/pw_transfer/ReadTransfer.java
index 4834ff8..15e3cd3 100644
--- a/pw_transfer/java/main/dev/pigweed/pw_transfer/ReadTransfer.java
+++ b/pw_transfer/java/main/dev/pigweed/pw_transfer/ReadTransfer.java
@@ -138,7 +138,7 @@
           remainingWindowSize <= parameters.maxPendingBytes() / EXTEND_WINDOW_DIVISOR;
 
       if (remainingWindowSize == 0) {
-        logger.atFiner().log(
+        logger.atFinest().log(
             "%s received all pending bytes; sending transfer parameters update", ReadTransfer.this);
         sendChunk(prepareTransferParameters(/*extend=*/false));
       } else if (extendWindow) {

pw_transfer/java/main/dev/pigweed/pw_transfer/Transfer.java

diff --git a/pw_transfer/java/main/dev/pigweed/pw_transfer/Transfer.java b/pw_transfer/java/main/dev/pigweed/pw_transfer/Transfer.java
index 365bbba..fef83e59 100644
--- a/pw_transfer/java/main/dev/pigweed/pw_transfer/Transfer.java
+++ b/pw_transfer/java/main/dev/pigweed/pw_transfer/Transfer.java
@@ -34,6 +34,9 @@
   // Largest nanosecond instant. Used to block indefinitely when no transfers are pending.
   static final Instant NO_TIMEOUT = Instant.ofEpochSecond(0, Long.MAX_VALUE);
 
+  // Whether to output some particularly noisy logs.
+  static final boolean VERBOSE_LOGGING = false;
+
   private final int resourceId;
   private final ProtocolVersion desiredProtocolVersion;
   private final TransferEventHandler.TransferInterface eventHandler;
@@ -277,7 +280,9 @@
     }
 
     try {
-      logger.atFinest().log("%s sending %s", this, chunk);
+      if (VERBOSE_LOGGING) {
+        logger.atFinest().log("%s sending %s", this, chunk);
+      }
       eventHandler.sendChunk(chunk.toMessage());
     } catch (TransferError transferError) {
       changeState(new Completed(transferError));
@@ -305,7 +310,7 @@
     long durationNanos = Duration.between(startTime, Instant.now()).toNanos();
     long totalRate = durationNanos == 0 ? 0 : (bytesSent * 1_000_000_000 / durationNanos);
 
-    logger.atFine().log("%s progress: "
+    logger.atFiner().log("%s progress: "
             + "%5.1f%% (%d B sent, %d B confirmed received of %s B total) at %d B/s",
         this,
         progress.percentReceived(),

pw_transfer/java/main/dev/pigweed/pw_transfer/WriteTransfer.java

diff --git a/pw_transfer/java/main/dev/pigweed/pw_transfer/WriteTransfer.java b/pw_transfer/java/main/dev/pigweed/pw_transfer/WriteTransfer.java
index d89ed87..daf5213 100644
--- a/pw_transfer/java/main/dev/pigweed/pw_transfer/WriteTransfer.java
+++ b/pw_transfer/java/main/dev/pigweed/pw_transfer/WriteTransfer.java
@@ -90,12 +90,14 @@
       ByteString chunkData = ByteString.copyFrom(
           data, sentOffset, min(windowEndOffset - sentOffset, maxChunkSizeBytes));
 
-      logger.atFiner().log("%s sending bytes %d-%d (%d B chunk, max size %d B)",
-          WriteTransfer.this,
-          sentOffset,
-          sentOffset + chunkData.size() - 1,
-          chunkData.size(),
-          maxChunkSizeBytes);
+      if (VERBOSE_LOGGING) {
+        logger.atFinest().log("%s sending bytes %d-%d (%d B chunk, max size %d B)",
+            WriteTransfer.this,
+            sentOffset,
+            sentOffset + chunkData.size() - 1,
+            chunkData.size(),
+            maxChunkSizeBytes);
+      }
 
       sendChunk(buildDataChunk(chunkData));