ryan-mbuashundip commented on code in PR #35298:
URL: https://github.com/apache/beam/pull/35298#discussion_r2167577148


##########
sdks/python/apache_beam/runners/worker/sdk_worker.py:
##########
@@ -176,7 +176,7 @@ def __init__(
       # that should be reported to the runner when proocessing the first 
bundle.
       deferred_exception=None,  # type: Optional[Exception]
       runner_capabilities=frozenset(),  # type: FrozenSet[str]
-  ):
+      sdk_options=None):

Review Comment:
   ```suggestion
         sdk_options=None,  # type: PipelineOptions
     ):
   ```



##########
sdks/python/apache_beam/runners/worker/sdk_worker.py:
##########
@@ -230,7 +231,8 @@ def default_factory(id):
             status_address,
             self._bundle_processor_cache,
             self._state_cache,
-            enable_heap_dump)  # type: Optional[FnApiWorkerStatusHandler]
+            enable_heap_dump,

Review Comment:
   ```suggestion
               enable_heap_dump,  # type: Optional[FnApiWorkerStatusHandler]
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to