You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/10/21 09:24:55 UTC

[GitHub] [airflow] kosteev commented on pull request #11712: Add num_retries parameter for google MLEngine hook requests

kosteev commented on pull request #11712:
URL: https://github.com/apache/airflow/pull/11712#issuecomment-713435935


   `_poll_with_exponential_delay` is intended to handle api-specific errors, in this case it retries in case of 429 error (and can be extended if needed).
   `num_retries` parameter for `execute` method activates retry mechanism for lower level errors, like `ConnectionError`/`socket.timeot`/`_ssl_SSLError`
   https://github.com/googleapis/google-api-python-client/blob/master/googleapiclient/http.py#L136
   Btw, this parameter is used in all other hooks except this one.
   
   So, they are similar but handle different type of errors.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org