Suvrat1629 commented on PR #36296: URL: https://github.com/apache/beam/pull/36296#issuecomment-3691072494
@kennknowles I have addressed your comments and changed some of the pr logic. I went through some of the code and realised that the `IOException` being wrapped in the `RuntimeException` was causing the infinite retries(according to my understanding), so I introduced a new typed exception that explicitly specified that the retries had exhausted and then threw a `IllegalStateException` wrapped in a `RuntimeException`. I think the logs and the `IllegalStateException` should act as a terminal error for dataflow. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
