From 89c6f8ecc84a959750135f4e3e8b424c71d1d9f7 Mon Sep 17 00:00:00 2001 From: MV Shiva Prasad Date: Wed, 9 Oct 2024 06:37:09 +0000 Subject: [PATCH] Fix InProcess Memory Leak --- .../java/io/grpc/internal/AbstractTransportTest.java | 4 ---- .../main/java/io/grpc/inprocess/InProcessChannelBuilder.java | 2 +- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/core/src/testFixtures/java/io/grpc/internal/AbstractTransportTest.java b/core/src/testFixtures/java/io/grpc/internal/AbstractTransportTest.java index 7099b5df6b6..75ea2678709 100644 --- 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 index 3bb3ca2dbb5..4e711a94004 100644 --- 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");