Fix InProcess Memory Leak

grpc · Oct 9, 2024 · 89c6f8e · 89c6f8e
1 parent 4f2fe03
commit 89c6f8e
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/core/src/testFixtures/java/io/grpc/internal/AbstractTransportTest.java b/core/src/testFixtures/java/io/grpc/internal/AbstractTransportTest.java
@@ -136,10 +136,6 @@ protected abstract InternalServer newServer(
    */
   protected abstract String testAuthority(InternalServer server);
 
-  /**
-   * Returns true (which is default) if the transport reports message sizes to StreamTracers.
-   */
-
   protected final Attributes eagAttrs() {
     return EAG_ATTRS;
   }

diff --git a/inprocess/src/main/java/io/grpc/inprocess/InProcessChannelBuilder.java b/inprocess/src/main/java/io/grpc/inprocess/InProcessChannelBuilder.java
@@ -229,7 +229,7 @@ public InProcessChannelBuilder propagateCauseWithStatus(boolean enable) {
    * nothing else needs the serialized message.
    * @param assumedMessageSize length of InProcess transport's messageSize.
    * @return this
-   * @throws IllegalArgumentException if assumedMessageSize is non-positive
+   * @throws IllegalArgumentException if assumedMessageSize is negative.
    */
   public InProcessChannelBuilder assumedMessageSize(long assumedMessageSize) {
     checkArgument(assumedMessageSize >= 0, "assumedMessageSize must be >= 0");