From 028b589eff00741b80f02057716401fc55566c56 Mon Sep 17 00:00:00 2001
From: Larry Safran <107004254+larry-safran@users.noreply.github.com>
Date: Tue, 19 Jul 2022 12:35:52 -0700
Subject: [PATCH] core: Disable retry by default for in-process transport's
 channel (#9361)

See #8712
---
 .../java/io/grpc/inprocess/InProcessChannelBuilder.java     | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/io/grpc/inprocess/InProcessChannelBuilder.java b/core/src/main/java/io/grpc/inprocess/InProcessChannelBuilder.java
index df396ae2f662..35998a535e2d 100644
--- a/core/src/main/java/io/grpc/inprocess/InProcessChannelBuilder.java
+++ b/core/src/main/java/io/grpc/inprocess/InProcessChannelBuilder.java
@@ -114,6 +114,10 @@ public ClientTransportFactory buildClientTransportFactory() {
     managedChannelImplBuilder.setStatsRecordStartedRpcs(false);
     managedChannelImplBuilder.setStatsRecordFinishedRpcs(false);
     managedChannelImplBuilder.setStatsRecordRetryMetrics(false);
+
+    // By default, In-process transport should not be retriable as that leaks memory.  Since
+    // there is no wire, bytes aren't calculated so buffer limit isn't respected
+    managedChannelImplBuilder.disableRetry();
   }
 
   @Internal
@@ -123,7 +127,7 @@ protected ManagedChannelBuilder<?> delegate() {
   }
 
   @Override
-  public final InProcessChannelBuilder maxInboundMessageSize(int max) {
+  public InProcessChannelBuilder maxInboundMessageSize(int max) {
     // TODO(carl-mastrangelo): maybe throw an exception since this not enforced?
     return super.maxInboundMessageSize(max);
   }