Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1339
@bzz @Leemoonsoo
Sorry for my late response. I spent some time to test various cases on the
different OS.
I think it's ready for review(CI is green at last!).
I updated the PR description accordingly. Hope it helps you to remind the
purpose of this PR :)
Here is the list of changes after my initial commits
- Directory of Spark bin
I changed the dir of Spark bin from `interpreter/spark/` to `local-spark`.
Since `mvn clean` will remove `interpreter/`, users will see âDo you want to
download local Spark?â whenever they re-build and restart Zeppelin. So I
think creating a new dir(`local-spark`) would be better in this case.
- Supporting Windows users
I wanted to create `download-spark.cmd` for Windows. But sadly we canât
use many shell commands such as `curl`, `tar` and `sed` in the batch script. It
was hard to find 100% compatible commands for Windows. Maybe we can guide the
Windows users to install those commands by themselves, but itâs a bit
overdoing i think. Actually `download-spark` script is only for downloading the
latest version of Spark and set `SPARK_HOME`. So I updated some docs to explain
this as a alternative way.
- Documentations
I think itâs quite big change that ppl need to enter âYes/Noâ when
they start Zeppelin. Even though itâs only one time. So I updated
`README.md`, `install.md` and `spark.md`. The below screenshot is `spark.md`.
And as @bzz said before,
>How about EMR\Dataproc\Juju\BigTop users, will the proposed change affect
them?
Do we need to provide this local Spark mode for them? Actually it's my
question.. :D
And please feel free to point me for anything if it's needed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
