apache / apache/airflow

Generalize on_kill() for SQLExecuteQueryOperator

Open
#27,314 1 comment 1 reaction 0 assignees View on GitHub
area:providers kind:feature provider:common-sql
Dominant language
Python
Stars
46.9k
Forks
17.8k
Avg merge
2d 10h
Merged PRs (30d)
483

Description

### Body

The operator doesn't have `on_kill` method and we try to find a way to make a generalized one.
Referencing TrinoOperator:
https://github.com/apache/airflow/blob/2a34dc9e8470285b0ed2db71109ef4265e29688b/airflow/providers/trino/operators/trino.py#L62-L75

### Committer

- [X] I acknowledge that I am a maintainer/committer of the Apache Airflow project.

Contributor guide

Open the contributing guide

Research direction

Start at the SQLExecuteQueryOperator entry point and compare its lifecycle with on_kill in airflow/providers/trino/operators/trino.py at the linked lines. Determine what behavior can be generalized and verify that the resulting on_kill behavior is covered by the relevant operator tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data-engineering
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.