addressed some comments

Author: liferoad

sdks/python/apache_beam/runners/worker/channel_factory.py

@@ -23,24 +23,12 @@
 
 class GRPCChannelFactory(grpc.StreamStreamClientInterceptor):
   DEFAULT_OPTIONS = [
-      # Default: 30000ms (30s), increased to 180s to reduce ping frequency
-      ("grpc.keepalive_time_ms", 180000),
-      # Default: 5000ms (5s), increased to 10 minutes for stability
+      # Default: 20000ms (20s), increased to 10 minutes for stability
       ("grpc.keepalive_timeout_ms", 600000),
       # Default: 2, set to 0 to allow unlimited pings without data
       ("grpc.http2.max_pings_without_data", 0),
       # Default: False, set to True to allow keepalive pings when no calls
       ("grpc.keepalive_permit_without_calls", True),
-      # Default: 300000ms (5min), increased to 10min for stability
-      ("grpc.http2.min_recv_ping_interval_without_data_ms", 600000),
-      # Default: 300000ms (5min), increased to 120s for conservative pings
-      ("grpc.http2.min_sent_ping_interval_without_data_ms", 120000),
-      # Default: 2, set to 0 to allow unlimited ping strikes
-      ("grpc.http2.max_ping_strikes", 0),
-      # Default: 0 (disabled), enable socket reuse for better handling
-      ("grpc.so_reuseport", 1),
-      # Default: 0 (disabled), set 30s TCP timeout for connection control
-      ("grpc.tcp_user_timeout_ms", 30000),
   ]
 
   def __init__(self):

sdks/python/apache_beam/utils/subprocess_server.py

@@ -188,8 +188,6 @@ def start(self):
       channel_options = [
           ("grpc.max_receive_message_length", -1),
           ("grpc.max_send_message_length", -1),
-          # Default: 30000ms (30s), increased to 180s to reduce ping frequency
-          ("grpc.keepalive_time_ms", 180000),
           # Default: 5000ms (5s), increased to 10 minutes for stability
           ("grpc.keepalive_timeout_ms", 600000),
           # Default: 2, set to 0 to allow unlimited pings without data
@@ -204,8 +202,6 @@ def start(self):
           ("grpc.http2.max_ping_strikes", 0),
           # Default: 0 (disabled), enable socket reuse for better handling
           ("grpc.so_reuseport", 1),
-          # Default: 0 (disabled), set 30s TCP timeout for connection control
-          ("grpc.tcp_user_timeout_ms", 30000),
       ]
       self._grpc_channel = grpc.insecure_channel(
           endpoint, options=channel_options)