Remove logging before errors thrown in S2AStub.

rmehta19 · rmehta19 · commit 35084afb3ab3 · 2024-09-09T11:01:04.000-07:00
diff --git a/s2a/src/main/java/io/grpc/s2a/handshaker/S2AStub.java b/s2a/src/main/java/io/grpc/s2a/handshaker/S2AStub.java
@@ -100,11 +100,6 @@ public SessionResp send(SessionReq req) throws IOException, InterruptedException
       }
       responses.clear();
       if (exception != null) {
-        logger.log(
-            Level.WARNING,
-            "Received an unexpected response from a host at the S2A's address. The S2A might be"
-                + " unavailable. "
-                + exception.getMessage());
         throw new IOException(
             "Received an unexpected response from a host at the S2A's address. The S2A might be"
                 + " unavailable."
@@ -115,7 +110,6 @@ public SessionResp send(SessionReq req) throws IOException, InterruptedException
     try {
       writer.onNext(req);
     } catch (RuntimeException e) {
-      logger.log(Level.WARNING, "Error occurred while writing to the S2A.", e);
       writer.onError(e);
       responses.offer(Result.createWithThrowable(e));
     }
@@ -172,7 +166,6 @@ public void onNext(SessionResp resp) {
      */
     @Override
     public void onError(Throwable t) {
-      logger.log(Level.WARNING, "Error occurred while reading from the S2A.", t);
       responses.offer(Result.createWithThrowable(t));
     }
 
