shunping commented on code in PR #34786: URL: https://github.com/apache/beam/pull/34786#discussion_r2069055224
########## sdks/python/apache_beam/io/gcp/gcsio_retry.py: ########## @@ -54,9 +55,14 @@ def __call__(self, exc): _LOGGER.warning('cannot inspect the caller stack frame') return - # next_sleep is one of the arguments in the caller + # sleep_iterator is one of the arguments in the caller Review Comment: Agreeļ¼ Let's check the package version before choosing which arg name to access. I remembered the current approach was the only feasible way to retrieve wait time in an accurate way during my implementation. And I acknowledged it is a bit fragile. -- 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