utkarsharma2 commented on code in PR #35919: URL: https://github.com/apache/airflow/pull/35919#discussion_r1417042460
########## airflow/providers/weaviate/hooks/weaviate.py: ########## @@ -119,38 +125,204 @@ def test_connection(self) -> tuple[bool, str]: self.log.error("Error testing Weaviate connection: %s", e) return False, str(e) + @retry(reraise=True, stop=stop_after_attempt(3), retry=retry_if_exception_type(requests.ConnectionError)) Review Comment: @ephraimbuddy I rechecked the PR and we can probably move retry changes for old methods like `create_class` or `create_schema` into the new PR but the methods `get_schema` and `update_config` introduced in this PR would have to be in this PR only, which will lead to related changes scattered across PRs. Would it make sense to discuss the pros/cons for retry here only? -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org