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.

Sign up for GitHub

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

Jump to bottom

Rework SUPERVISOR_COMMS.send_request class/method to have log stored as a attribute #44615

Open
kaxil opened this issue Dec 3, 2024 · 0 comments
Open

Rework SUPERVISOR_COMMS.send_request class/method to have log stored as a attribute #44615

kaxil opened this issue Dec 3, 2024 · 0 comments
Labels
area:task-execution-interface-aip72 AIP-72: Task Execution Interface (TEI) aka Task SDK
Milestone
Airflow 3.0.0

Comments

@kaxil
Copy link
Member

kaxil commented Dec 3, 2024 

          This reminds me: we should rework this class/method to have `log` stored as a attribute so we don't need to pass log in to this class.

(Future work; doing this will make it easier to use this from within XCom or Variable class.)

Originally posted by @ashb in #44590 (comment)
@kaxil kaxil changed the title This reminds me: we should rework this class/method to have log stored as a attribute so we don't need to pass log in to this class. Rework SUPERVISOR_COMMS.send_request class/method to have log stored as a attribute so we don't need to pass log in to this class. Dec 3, 2024
@kaxil kaxil changed the title Rework SUPERVISOR_COMMS.send_request class/method to have log stored as a attribute so we don't need to pass log in to this class. Rework SUPERVISOR_COMMS.send_request class/method to have log stored as a attribute Dec 3, 2024
@kaxil kaxil added the area:task-execution-interface-aip72 AIP-72: Task Execution Interface (TEI) aka Task SDK label Dec 3, 2024
@kaxil kaxil mentioned this issue Dec 3, 2024
Merged
@kaxil kaxil added this to the Airflow 3.0.0 milestone Jan 17, 2025
@ashb ashb moved this from Todo to Icebox in AIP-72 - Task Execution Interface and SDK Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:task-execution-interface-aip72 AIP-72: Task Execution Interface (TEI) aka Task SDK
Projects
AIP-72 - Task Execution Interface and SDK
Status: Icebox
Milestone
Airflow 3.0.0
Development

No branches or pull requests
1 participant
@kaxil