Skip to content

[SPARK-55342][K8S] Fix ExecutorPodsLifecycleEventHandler to ExecutorPodsLifecycleManager#54119

Closed
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:SPARK-55342
Closed

[SPARK-55342][K8S] Fix ExecutorPodsLifecycleEventHandler to ExecutorPodsLifecycleManager#54119
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:SPARK-55342

Conversation

@dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Feb 3, 2026

What changes were proposed in this pull request?

This PR aims to fix ExecutorPodsLifecycleEventHandler to ExecutorPodsLifecycleManager in the code and comments because ExecutorPodsLifecycleEventHandler doesn't exist.

-  // Allow removeExecutor to be accessible by ExecutorPodsLifecycleEventHandler
+  // Allow removeExecutor to be accessible by ExecutorPodsLifecycleManager
- val executorPodsLifecycleEventHandler = new ExecutorPodsLifecycleManager(
+ val executorPodsLifecycleManager = new ExecutorPodsLifecycleManager(
- lifecycleEventHandler: ExecutorPodsLifecycleManager,
+ lifecycleManager: ExecutorPodsLifecycleManager,

Why are the changes needed?

To be correct and consistent.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass the CIs.

Was this patch authored or co-authored using generative AI tooling?

Generated-by: Opus 4.5 on Claude Code v2.1.5

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

JIRA Issue Information

=== Sub-task SPARK-55342 ===
Summary: Fix ExecutorPodsLifecycleEventHandler to ExecutorPodsLifecycleManager
Assignee: None
Status: Open
Affected: ["4.2.0"]


This comment was automatically generated by GitHub Actions

@dongjoon-hyun
Copy link
Member Author

Thank you, @HyukjinKwon ! Merged to master.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-55342 branch February 3, 2026 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants