Skip to content

Add read timeout for retry#1896

Merged
AI-IshanBhatt merged 1 commit into
elastic:masterfrom
AI-IshanBhatt:add_retry_for_read_timeout
Dec 4, 2024
Merged

Add read timeout for retry#1896
AI-IshanBhatt merged 1 commit into
elastic:masterfrom
AI-IshanBhatt:add_retry_for_read_timeout

Conversation

@AI-IshanBhatt
Copy link
Copy Markdown
Contributor

This PR deals with the ReadTimeoutError when downloading track data.
It is a transient error and it should be handled by retry mechanism.

Following is the stack trace

File "/home/esbench/.local/lib/python3.11/site-packages/urllib3/response.py", line 566, in read                                                                                                                                                    
    with self._error_catcher():                                                                                                                                                                                                                      
  File "/home/esbench/.pyenv/versions/3.11.7/lib/python3.11/contextlib.py", line 158, in __exit__                                                                                                                                                    
    self.gen.throw(typ, value, traceback)                                                                                                                                                                                                            
  File "/home/esbench/.local/lib/python3.11/site-packages/urllib3/response.py", line 449, in _error_catcher                                                                                                                                          
    raise ReadTimeoutError(self._pool, None, "Read timed out.")                                                                                                                                                                                      
urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='rally-tracks.elastic.co', port=443): Read timed out.

@AI-IshanBhatt AI-IshanBhatt requested a review from a team November 29, 2024 09:37
@AI-IshanBhatt AI-IshanBhatt merged commit 644706f into elastic:master Dec 4, 2024
@dpifke-elastic dpifke-elastic added the bug Something's wrong label Mar 28, 2025
@dpifke-elastic dpifke-elastic added this to the 2.12.0 milestone Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something's wrong

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants