Switch LoggingMixin to return structlog.BoundLogger
Open
area:logging
kind:meta
- Dominant language
- Python
- Stars
- 46.9k
- Forks
- 17.8k
- Avg merge
- 2d 10h
- Merged PRs (30d)
- 483
Description
### Body
This is so that we can switch to structlog universally in the airflow codebase, which lets us use its features re binding vars and context vars.
I started work on this in https://github.com/apache/airflow/pull/46897 but have to drop it for now. Anyone with time and interest could pick it up. May just need to fix the tests.
### Committer
- [x] I acknowledge that I am a maintainer/committer of the Apache Airflow project.
Contributor guide
Research direction
Start by reviewing pull request 46897 and locating the LoggingMixin implementation and its tests. Confirm the intended BoundLogger return behavior and update the affected tests; done means the mixin uses structlog consistently without regressions.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100