xchwan commented on issue #47950: URL: https://github.com/apache/airflow/issues/47950#issuecomment-3310701239
> [@xchwan](https://github.com/xchwan) it's nearly impossible to me to test if your change is working as the condition only happens on very specific situations of high load. > > > Once the first task call, the second will get 429. However, it will wait and call again since exponential back-off due to this change. > > Are you sure? I doubt that is that simple colleague. You need way more load than that, approximately 5RPS. > > I express my opinion that this check is non sense and yet you said "remove the check function is not a good way". I could help you test if the check was removed. I can't help you test this current implementation. Thanks your reply. But I'm wondering that the check function is called at _get_token if statement and it do _get_aad_token function after check. Does _get_aad_token not have this limit? I mean _get_add_token may possible get 429 even thought remove check function in the case you mention. You still call too much API to Databricks, right? -- 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]
