Re: [PR] Adding 1 second sleep time between retries for publishing tasks [airflow]

2024-02-04 Thread via GitHub
github-actions[bot] closed pull request #35791: Adding 1 second sleep time between retries for publishing tasks URL: https://github.com/apache/airflow/pull/35791 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [PR] Adding 1 second sleep time between retries for publishing tasks [airflow]

2023-12-01 Thread via GitHub
potiuk commented on PR #35791: URL: https://github.com/apache/airflow/pull/35791#issuecomment-1835835446 I don't think hardcoding it to 1 second is a good idea. it sounds pretty magical. Also it has potentially undesireable side effects - there might be cases where the retry is actually goo

Re: [PR] Adding 1 second sleep time between retries for publishing tasks [airflow]

2023-11-23 Thread via GitHub
sathish9094 commented on PR #35791: URL: https://github.com/apache/airflow/pull/35791#issuecomment-1825122613 @hussein-awala Can you please help to review/approve this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[PR] Adding 1 second sleep time between retries for publishing tasks [airflow]

2023-11-22 Thread via GitHub
sathish9094 opened a new pull request, #35791: URL: https://github.com/apache/airflow/pull/35791 This retry loop is pretty fast and it doesn't wait for number of seconds value that we pass from airflow.cfg ``` # The Maximum number of retries for publishing task messages to the broker

Re: [PR] Adding 1 second sleep time between retries for publishing tasks [airflow]

2023-11-22 Thread via GitHub
boring-cyborg[bot] commented on PR #35791: URL: https://github.com/apache/airflow/pull/35791#issuecomment-1822301201 Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution