Skip to content

Clarify reattach order of find_spark_job in SparkKubernetesOperator - #64186

Merged
eladkal merged 1 commit into
apache:mainfrom
Nataneljpwd:feature/add-better-comment-for-spark-k8s
Mar 24, 2026
Merged

Clarify reattach order of find_spark_job in SparkKubernetesOperator#64186
eladkal merged 1 commit into
apache:mainfrom
Nataneljpwd:feature/add-better-comment-for-spark-k8s

Conversation

@Nataneljpwd

@Nataneljpwd Nataneljpwd commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

Clarify the logic for pod reattachment order since this might not be trivial to understand reasoning

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@boring-cyborg boring-cyborg Bot added area:providers provider:cncf-kubernetes Kubernetes (k8s) provider related issues labels Mar 24, 2026
@eladkal eladkal changed the title Clarified logic via comment Clarify reattach order of find_spark_job in SparkKubernetesOperator Mar 24, 2026
@eladkal
eladkal merged commit 9c46b21 into apache:main Mar 24, 2026
39 of 40 checks passed
aaron-y-chen pushed a commit to aaron-y-chen/airflow that referenced this pull request Mar 30, 2026
Suraj-kumar00 pushed a commit to Suraj-kumar00/airflow that referenced this pull request Apr 7, 2026
abhijeets25012-tech pushed a commit to abhijeets25012-tech/airflow that referenced this pull request Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:cncf-kubernetes Kubernetes (k8s) provider related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants