turboFei opened a new pull request, #7620:
URL: https://github.com/apache/kyuubi/pull/7620
## Why are the changes needed?
`checkPodAppCanceled` was misleading: the method handles two distinct
orphan-pod scenarios:
1. **CANCELED batch** — user explicitly canceled the batch, but the pod
appeared after Kyuubi gave up tracking it.
2. **Submit-timeout batch** — `kyuubi.engine.kubernetes.submit.timeout`
elapsed, Kyuubi recorded app state as `NOT_FOUND` and marked the batch FAILED,
but the pod started up afterwards.
In both cases the pod is orphaned (no owning session), and the old name
only hinted at case 1.
Log messages were also vague ("try to delete the pod", "is in error state
and application not found") — they didn't identify the pod as an orphan or
state the delete action directly.
## How was this patch tested?
No behavior change; this is a rename + log message improvement only.
Existing test coverage in `KubernetesApplicationOperationSuite` covers both the
CANCELED and ERROR+NOT_FOUND branches.
## Was this patch authored or co-authored using generative AI tooling?
Assisted-by: Claude:claude-sonnet-4-6
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
