Skip to content

Conversation

@echarles
Copy link
Member

@echarles echarles commented Nov 1, 2015

Allow to configure the context path of the webapp via zeppelin.server.context.path property or ZEPPELIN_CONTEXT_PATH env variable + add doc on this

@Leemoonsoo
Copy link
Member

Thanks for the nice feature. Could you rebase and trigger CI?

@echarles
Copy link
Member Author

echarles commented Nov 4, 2015

From https://s3.amazonaws.com/archive.travis-ci.org/jobs/88620950/log.txt, the build is success

[INFO] Zeppelin: Packaging distribution ................... SUCCESS [ 35.204 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 02:26 min

Any reason for the red flag on github?

Should I rebase and push again, or close/reopen to trigger the CI?

@Leemoonsoo
Copy link
Member

it's because of

�[0K$ ./testing/startSparkCluster.sh 1.4.0 2.3
SPARK_HOME is /home/travis/build/apache/incubator-zeppelin/spark-1.4.0-bin-hadoop2.3 


No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.

issue resolved by #380. This time, you'll need rebase and push to apply #380.

….context.path property or ZEPPELIN_CONTEXT_PATH env variable + add doc on this
Copy link
Member

Choose a reason for hiding this comment

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

could this be ZEPPELIN_SERVER_CONTEXT_PATH?
there is a object called ZeppelinContext in the Spark interpreter group and this name might be confusing

@echarles
Copy link
Member Author

Any more feedback on this?

@Leemoonsoo
Copy link
Member

I'm not sure why CI is failing. You can trigger CI again.
And the branch seems removed, can not try anymore.

@echarles
Copy link
Member Author

Please use #429 to track this.

asfgit pushed a commit that referenced this pull request Nov 22, 2015
Allow to configure the context path of the webapp via zeppelin.server.context.path property or ZEPPELIN_SERVER_CONTEXT_PATH env variable + documentation for this.

(replaces #385)

Author: Eric Charles <[email protected]>

This patch had conflicts when merged, resolved by
Committer: Lee moon soo <[email protected]>

Closes #429 from echarles/ZEPPELIN-337-Configurable-ContextPath and squashes the following commits:

37e07f9 [Eric Charles] Ensure context path is also setup when running with the distribution
70194fc [Eric Charles] Set /api path on servlet level, not context to fix test
54f6b07 [Eric Charles] ZEPPELIN_CONTEXT_PATH is now ZEPPELIN_SERVER_CONTEXT_PATH
9095d5d [Eric Charles] Allow to configure the context path of the webapp via zeppelin.server.context.path property or ZEPPELIN_CONTEXT_PATH env variable + add doc on this
lelou6666 pushed a commit to lelou6666/incubator-zeppelin that referenced this pull request Mar 25, 2016
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.

3 participants