Skip to content
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

Update log level in scheduler critical section edge case #30694

Merged
merged 1 commit into from
Apr 22, 2023

Conversation

jedcunningham
Copy link
Member

This log message can be useful if the scheduler ends up needing to query TIs more than once per scheduler loop, so make it INFO vs DEBUG to increase discoverability.

This log message can be useful if the scheduler ends up needing to query
TIs more than once per scheduler loop, so make it INFO vs DEBUG to
increase discoverability.
@boring-cyborg boring-cyborg bot added the area:Scheduler including HA (high availability) scheduler label Apr 18, 2023
@potiuk potiuk merged commit 3f5280b into apache:main Apr 22, 2023
@potiuk potiuk added this to the Airflow 2.6.0 milestone Apr 22, 2023
@potiuk
Copy link
Member

potiuk commented Apr 22, 2023

Maybe 2.6.0 @jedcunningham @ephraimbuddy @pierrejeambrun ?

@jedcunningham jedcunningham deleted the change_critical_section_loop_logging branch April 22, 2023 20:23
@jedcunningham
Copy link
Member Author

I don't see any harm in it, but it also isn't really needed either.

@potiuk
Copy link
Member

potiuk commented Apr 22, 2023

ok. Remove then.

@ephraimbuddy ephraimbuddy added the type:improvement Changelog: Improvements label Apr 23, 2023
ephraimbuddy pushed a commit that referenced this pull request Apr 23, 2023
This log message can be useful if the scheduler ends up needing to query
TIs more than once per scheduler loop, so make it INFO vs DEBUG to
increase discoverability.

(cherry picked from commit 3f5280b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:Scheduler including HA (high availability) scheduler type:improvement Changelog: Improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants