kyuubi icon indicating copy to clipboard operation
kyuubi copied to clipboard

[KYUUBI #5863] BatchJobSubmission close repeatedly

Open ASiegeLion opened this issue 1 year ago โ€ข 1 comments

:mag: Description

Issue References ๐Ÿ”—

This pull request fixes #

Describe Your Solution ๐Ÿ”ง

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Types of changes :bookmark:

  • [ ] Bugfix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan ๐Ÿงช

Behavior Without This Pull Request :coffin:

Behavior With This Pull Request :tada:

Related Unit Tests


Checklists

๐Ÿ“ Author Self Checklist

  • [ ] My code follows the style guidelines of this project
  • [ ] I have performed a self-review
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [ ] My changes generate no new warnings
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] New and existing unit tests pass locally with my changes
  • [ ] This patch was not authored or co-authored using Generative Tooling

๐Ÿ“ Committer Pre-Merge Checklist

  • [ ] Pull request title is okay.
  • [ ] No license issues.
  • [ ] Milestone correctly set?
  • [ ] Test coverage is ok
  • [ ] Assignees are selected.
  • [ ] Minimum number of approvals
  • [ ] No changes are requested

Be nice. Be informative.

ASiegeLion avatar Dec 16 '23 11:12 ASiegeLion

According to the description of your issue, we may be able to modify BatchJobSubmission#close like this:

  override def close(): Unit = withLockRequired {
    if (!isClosedOrCanceled) {
      withClosingOperationLog {

wForget avatar Jan 25 '24 01:01 wForget