This is an automated email from the ASF dual-hosted git repository.
jackylee-ch pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gluten.git
The following commit(s) were added to refs/heads/main by this push:
new 3a305e46d8 [MINOR][CI] Stop unrelated PR comments from cancelling
Delta and ANSI runs (#12906)
3a305e46d8 is described below
commit 3a305e46d87a2ab6101b34ff20ae5839eed2d72b
Author: YangJie <[email protected]>
AuthorDate: Fri Aug 28 06:24:42 2026 -0400
[MINOR][CI] Stop unrelated PR comments from cancelling Delta and ANSI runs
(#12906)
---
.github/workflows/delta_spark_ut.yml | 18 +++++++++++++++++-
.github/workflows/velox_backend_ansi.yml | 14 +++++++++++++-
2 files changed, 30 insertions(+), 2 deletions(-)
diff --git a/.github/workflows/delta_spark_ut.yml
b/.github/workflows/delta_spark_ut.yml
index bfba320e5e..066a418e09 100644
--- a/.github/workflows/delta_spark_ut.yml
+++ b/.github/workflows/delta_spark_ut.yml
@@ -149,8 +149,24 @@ env:
DELTA_NUM_SHARDS: '8'
# Cancel older `pull_request` and `/delta-test` runs for the same PR.
+#
+# `issue_comment` fires on every comment posted on an issue or a PR, and
+# `github.event.issue.number` is that issue's number, which for a comment on a
+# PR is the PR's own number. Such a run would therefore join the PR's group and
+# cancel an in-flight suite while `delta-test-requested` below skips it, losing
+# the suite and replacing it with nothing. Give only that case a group of its
+# own, so every other trigger keeps the exact group name it has today. The
+# negated condition is that job's `if` verbatim.
concurrency:
- group: ${{ github.repository }}-${{ github.event.pull_request.number ||
github.event.issue.number || github.head_ref || github.sha }}-${{
github.workflow }}
+ group: >-
+ ${{ github.repository }}-${{ github.event.pull_request.number ||
github.event.issue.number || github.head_ref || github.sha }}-${{
github.workflow }}${{
+ !(github.event_name != 'issue_comment' ||
+ (github.event.issue.pull_request &&
+ (github.event.comment.body == '/delta-test' ||
+ startsWith(github.event.comment.body, '/delta-test ') ||
+ startsWith(github.event.comment.body, format('/delta-test{0}',
fromJSON('"\r"'))) ||
+ startsWith(github.event.comment.body, format('/delta-test{0}',
fromJSON('"\n"'))))))
+ && format('-comment-{0}', github.run_id) || '' }}
cancel-in-progress: true
jobs:
diff --git a/.github/workflows/velox_backend_ansi.yml
b/.github/workflows/velox_backend_ansi.yml
index 2ed44123ce..64c158f96d 100644
--- a/.github/workflows/velox_backend_ansi.yml
+++ b/.github/workflows/velox_backend_ansi.yml
@@ -40,8 +40,20 @@ env:
CCACHE_DIR: "${{ github.workspace }}/.ccache"
SPARK_ANSI_SQL_MODE: true
+# `issue_comment` fires on every comment posted on an issue or a PR, and
+# `github.event.issue.number` puts such a run in the same group as an in-flight
+# ANSI run for that PR, so `cancel-in-progress` kills it while `check-comment`
+# below skips the newcomer, leaving no replacement. Give only that case a group
+# of its own, so every other trigger keeps the exact group name it has today.
+# The negated condition is that job's `if` verbatim.
concurrency:
- group: ${{ github.repository }}-ansi-${{ github.event.issue.number ||
inputs.pr_number }}
+ group: >-
+ ${{ github.repository }}-ansi-${{ github.event.issue.number ||
inputs.pr_number }}${{
+ !((github.event_name == 'workflow_dispatch') ||
+ (github.event.issue.pull_request &&
+ (contains(github.event.comment.body, '/ansi-test') ||
+ contains(github.event.comment.body, '/ansi-analyze'))))
+ && format('-comment-{0}', github.run_id) || '' }}
cancel-in-progress: true
jobs:
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
