Skip to content

Missing double quote causes server not to start#500

Merged
gyogal merged 1 commit into
apache:masterfrom
edwardcapriolo:livy-kickoff-script
Dec 3, 2025
Merged

Missing double quote causes server not to start#500
gyogal merged 1 commit into
apache:masterfrom
edwardcapriolo:livy-kickoff-script

Conversation

@edwardcapriolo

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

bin/livy-server doesnt work at all.

How was this patch tested?

edward@fedora:~/incubator-livy$ bin/livy-server 
WARNING: Unknown module: ALL_UNNAMED specified to --add-exports
log4j:WARN No appenders could be found for logger (org.apache.livy.server.AccessManager).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.
Exception in thread "main" java.lang.IllegalArgumentException: Livy requires the SPARK_HOME environment variable
	at org.apache.livy.utils.LivySparkUtils$$anonfun$1.apply(LivySparkUtils.scala:55)
	at org.apache.livy.utils.LivySparkUtils$$anonfun$1.apply(LivySparkUtils.scala:55)
	at scala.Option.getOrElse(Option.scala:121)
	at org.apache.livy.utils.LivySparkUtils$.testSparkHome(LivySparkUtils.scala:54)
	at org.apache.livy.server.LivyServer.start(LivyServer.scala:79)
	at org.apache.livy.server.LivyServer$.main(LivyServer.scala:439)
	at org.apache.livy.server.LivyServer.main(LivyServer.scala)
edward@fedora:~/incubator-livy$ git status

Please review https://livy.incubator.apache.org/community/ before opening a pull request.

@edwardcapriolo

Copy link
Copy Markdown
Contributor Author

@gyogal gyogal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thank you for fixing this!

@gyogal gyogal merged commit 49a3616 into apache:master Dec 3, 2025
2 checks passed
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