Skip to content

[ZEPPELIN-6712] Fix CLI version options and restore CommandLineUtils - #5488

Merged
tbonelee merged 1 commit into
apache:masterfrom
ParkGyeongTae:ZEPPELIN-6712
Sep 20, 2026
Merged

tbonelee merged 1 commit into
apache:masterfrom
ParkGyeongTae:ZEPPELIN-6712

Conversation

@ParkGyeongTae

Copy link
Copy Markdown
Member

What is this PR for?

Fix bin/zeppelin.sh --version and bin/zeppelin.sh -v, which currently fail with Unsupported argument. because the option parser rejects them before reaching the version-handling code.

Restore CommandLineUtils and its unit tests, as getZeppelinVersion() in bin/common.sh still references the removed class. Update the script to recognize both version options, invoke the version command after loading common.sh, and include the options in the usage message.

What type of PR is it?

Bug Fix

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-6712

How should this be tested?

The restored CommandLineUtilsTest covers --version, -v, and invocation without arguments.

Run:

./mvnw package -pl zeppelin-server --am \
  -Dtest=CommandLineUtilsTest \
  -Dsurefire.failIfNoSpecifiedTests=false

All 3 tests passed with no failures, errors, or skipped tests. The build succeeded.

Manual validation:

bin/zeppelin.sh --version
bin/zeppelin.sh -v
bin/zeppelin.sh --config conf --version

All three commands printed 0.13.0-SNAPSHOT and exited successfully without starting the server in the local development checkout. Bash syntax validation also passed.

Questions:

  • Do the license files need updating? No.
  • Are there breaking changes for older versions? No.
  • Does this need documentation? The script's usage message has been updated.

@tbonelee
tbonelee merged commit bcb428d into apache:master Sep 20, 2026
24 checks passed
@tbonelee

Copy link
Copy Markdown
Contributor

Merged into master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants