This is an automated email from the ASF dual-hosted git repository.
gurwls223 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 0a05e1cd6346 [SPARK-52412][INFRA][FOLLOW-UP] Skip empty patterns when
redacting
0a05e1cd6346 is described below
commit 0a05e1cd6346dc2ed2663358067fb17192803a63
Author: Hyukjin Kwon <[email protected]>
AuthorDate: Sun Jun 8 21:49:08 2025 -0700
[SPARK-52412][INFRA][FOLLOW-UP] Skip empty patterns when redacting
### What changes were proposed in this pull request?
This PR is a followup of https://github.com/apache/spark/pull/51102 that
skips empty strings.
### Why are the changes needed?
To make dry runs passing. It's broken
https://github.com/apache/spark/actions/runs/15515989616/job/43682941208
### Does this PR introduce _any_ user-facing change?
No, dev-only.
### How was this patch tested?
Manually tested.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #51118 from HyukjinKwon/SPARK-52412-followup.
Authored-by: Hyukjin Kwon <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
---
.github/workflows/release.yml | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
index 96f334c49bf0..92ee75409c4f 100644
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -236,8 +236,9 @@ jobs:
[ -f "$file" ] || continue
cp "$file" "$file.bak"
for pattern in "${PATTERNS[@]}"; do
+ [ -n "$pattern" ] || continue # Skip empty patterns
regex="${pattern//\*/.*}"
- sed -i "s/$regex/***/g" "$file"
+ sed -i "s|$regex|***|g" "$file"
done
done
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
