-
Notifications
You must be signed in to change notification settings - Fork 14.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AIRFLOW-5902] avoid unnecessary sleep to maintain local task job heart rate #6553
Conversation
aed8dd5
to
49d68ba
Compare
Codecov Report
@@ Coverage Diff @@
## master #6553 +/- ##
=========================================
- Coverage 83.83% 83.53% -0.3%
=========================================
Files 668 668
Lines 37569 37562 -7
=========================================
- Hits 31497 31379 -118
- Misses 6072 6183 +111
Continue to review full report at Codecov.
|
49d68ba
to
6c5d1d6
Compare
If the unit test changes are not approved, I would really like to see the removal of heart beat logging make its way into the next release. This has considerably inflated our log files. https://github.com/apache/airflow/blob/master/airflow/jobs/local_task_job.py#L118 |
@jj-ian this exactly how I came to this patch :) Feel free to apply this patch to your cluster, we have been running airflow with it in production for a week now. |
6c5d1d6
to
a2db4c6
Compare
a2db4c6
to
b07bbf7
Compare
…rt rate sleep to maintain heart rate is already done by the hearbeat() call
…rt rate (apache#6553) sleep to maintain heart rate is already done by the hearbeat() call
Make sure you have checked all steps below.
Jira
Description
After a heartbeat, time since last hearbeat will always be lower than heart rate, so the subsequent sleep is unnecessary. It also spams the task log with lots of info messages.
Sleep to maintain heart rate is already done by the
heartbeat()
call.Tests
Commits
Documentation