Skip to content

fixed finished job is no need alert.#4293

Merged
RocMarshal merged 2 commits intoapache:devfrom
Mrart:noAlertFinished
Jan 23, 2026
Merged

fixed finished job is no need alert.#4293
RocMarshal merged 2 commits intoapache:devfrom
Mrart:noAlertFinished

Conversation

@Mrart
Copy link
Contributor

@Mrart Mrart commented Sep 24, 2025

If batch job finished , No need alert.

@sonarqubecloud
Copy link

Copy link
Contributor

@RocMarshal RocMarshal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Mrart for the contribution!
Left a few of comments, PTAL ~ :)

@Mrart
Copy link
Contributor Author

Mrart commented Jan 23, 2026

@wolfboys PLTK

@sonarqubecloud
Copy link

@wolfboys wolfboys requested a review from RocMarshal January 23, 2026 07:56
@wolfboys
Copy link
Member

cc @RocMarshal PTAL, thanks

Copy link
Contributor

@RocMarshal RocMarshal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Mrart for the catch & @wolfboys for the review.
LGTM +1.
Merging...

@RocMarshal RocMarshal merged commit fecba85 into apache:dev Jan 23, 2026
29 of 42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants