pubsub: make connection log less aggressively (#2615)

pongad · web-flow · commit 4a5156a87a70 · 2017-11-13T17:25:22.000+11:00
Fixes #2614
diff --git a/google-cloud-pubsub/src/main/java/com/google/cloud/pubsub/v1/StreamingSubscriberConnection.java b/google-cloud-pubsub/src/main/java/com/google/cloud/pubsub/v1/StreamingSubscriberConnection.java
@@ -275,7 +275,8 @@ public void sendAckOperations(
         requestObserver.onNext(request);
       }
     } catch (Exception e) {
-      logger.log(Level.WARNING, "failed to send acks", e);
+      Level level = isAlive() ? Level.WARNING : Level.FINER;
+      logger.log(level, "failed to send ack operations", e);
     } finally {
       lock.unlock();
     }

Original file line number	Diff line number	Diff line change
`@@ -275,7 +275,8 @@ public void sendAckOperations(`
`275`	`275`	`requestObserver.onNext(request);`
`276`	`276`	`}`
`277`	`277`	`} catch (Exception e) {`
`278`		`- logger.log(Level.WARNING, "failed to send acks", e);`
	`278`	`+ Level level = isAlive() ? Level.WARNING : Level.FINER;`
	`279`	`+ logger.log(level, "failed to send ack operations", e);`
`279`	`280`	`} finally {`
`280`	`281`	`lock.unlock();`
`281`	`282`	`}`