ryan-mbuashundip commented on code in PR #35391:
URL: https://github.com/apache/beam/pull/35391#discussion_r2190723167
##########
sdks/python/apache_beam/runners/worker/sdk_worker.py:
##########
@@ -409,6 +416,10 @@ def create_worker(self):
return SdkWorker(
self._bundle_processor_cache, profiler_factory=self._profiler_factory)
+ def _shutdown_due_to_element_processing_timeout(self, errMsg: str) -> None:
+ _LOGGER.error('%sThe SDK harness will be terminated.', errMsg)
Review Comment:
Keep just one of the messages about the SDK harness terminated here, where
the `TimeoutError` is caught, and where the Timeout error is thrown. Having all
3 is redundant. Prefer to leave the message here as it fits the context of the
method.
--
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
