Skip to content

[AURON #2047] Fix Scala 2.13 CI NoClassDefFoundError#2048

Merged
cxzl25 merged 1 commit intoapache:masterfrom
cxzl25:auron_2047
Mar 1, 2026
Merged

[AURON #2047] Fix Scala 2.13 CI NoClassDefFoundError#2048
cxzl25 merged 1 commit intoapache:masterfrom
cxzl25:auron_2047

Conversation

@cxzl25
Copy link
Contributor

@cxzl25 cxzl25 commented Feb 27, 2026

Which issue does this PR close?

Closes #2047

Rationale for this change

  if [ 2.13 = "2.13" && "spark-3.5" != "spark-4.0" && "spark-3.5" != "spark-4.1" ]; then
    SPARK_FILE="spark-3.5.8-bin-hadoop3-scala2.13.tgz"
  else
    SPARK_FILE="spark-3.5.8-bin-hadoop3.tgz"
  fi
/home/runner/work/_temp/5e39dd44-8905-4fc9-ad30-a316cba0dc50.sh: line 2: [: missing `]'
--2026-02-27 05:16:45--  https://www.apache.org/dyn/closer.lua/spark/spark-3.5.8/spark-3.5.8-bin-hadoop3.tgz?action=download

https://github.com/apache/auron/actions/runs/22473356538/job/65096490529?pr=2040

What changes are included in this PR?

Are there any user-facing changes?

How was this patch tested?

GHA

--2026-02-27 13:27:10--  https://www.apache.org/dyn/closer.lua/spark/spark-3.5.8/spark-3.5.8-bin-hadoop3-scala2.13.tgz?action=download

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a bash syntax error in the GitHub Actions workflow that was causing the Scala 2.13 CI builds to fail with a "missing ]'" error. The issue was that single brackets [ ]cannot properly handle compound conditionals with the&&` operator in bash, leading to a syntax error that prevented the correct Spark binary from being downloaded, which in turn caused the NoClassDefFoundError.

Changes:

  • Fixed bash conditional syntax from single brackets [ ] to double brackets [[ ]] for compound conditions with multiple && operators

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cxzl25 cxzl25 requested a review from richox February 27, 2026 13:50
Copy link
Contributor

@ShreyeshArangath ShreyeshArangath left a comment

Choose a reason for hiding this comment

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

thanks for fixing this

@yew1eb
Copy link
Contributor

yew1eb commented Feb 28, 2026

Thanks, LGTM!

@cxzl25 cxzl25 merged commit d85c40c into apache:master Mar 1, 2026
212 of 214 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.

Scala 2.13 CI NoClassDefFoundError

6 participants