shahar1 commented on PR #61325:
URL: https://github.com/apache/airflow/pull/61325#issuecomment-3831652977

   > @shahar1 , I don’t understand why we are doing this. What I mean is: if we 
increase the time, it will fail after 10 seconds instead of 5 seconds, but it 
will still fail eventually. So how can we achieve our goal?
   > 
   > I also observed this in many test pattern - we generally increase the 
delay time 
   
   Not too familiar with Celery specifically, so I trusted Buğra with his 
suggested numbers - but in general falky tests are associated with sort of 
instability (unexpected latency in response, lack of resources, race 
conditions, etc.), which is usually unknown but could be managed.
   So by increasing delay time we try to overcome the flakiness to reduce 
chances of failure - it might take a couple iterations until we figure out the 
"right" numbers.
   Unfortunately the root cause might not be related though (which seems that 
you try to handle with the other PR), but "spacing" between retries wouldn't 
harm :)


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

Reply via email to