Skip to content

ZEPPELIN-375 - #670

Closed
tzolov wants to merge 5 commits into
apache:masterfrom
tzolov:ZEPPELIN-375
Closed

ZEPPELIN-375#670
tzolov wants to merge 5 commits into
apache:masterfrom
tzolov:ZEPPELIN-375

Conversation

@tzolov

Copy link
Copy Markdown
Contributor

Hot Fix to allow using Zeppelin with the latest (Java 8 based) Geode see: ZEPPELIN-375

Changes:

  • Make sure the that the zeppelin-distribution is build after the geode module. By default if you activate the geode profile (-Pgeode) the maven reactor will order the geode after the zeppelin-distribution. In result the geode is never added to the zeppelin binary distro.
  • Document how to build Zeppelin with Geode. Note that the only way to use Zeppelin with Geode is to build Zeppelin with Java8

@tzolovtzolov closed this Jan 27, 2016
@tzolovtzolov reopened this Jan 27, 2016
@tzolovtzolov closed this Jan 27, 2016
@tzolovtzolov reopened this Jan 27, 2016
@felixcheung

Copy link
Copy Markdown
Member

should this be enabled in travis?

@tzolov

Copy link
Copy Markdown
ContributorAuthor

@felixcheung, for sure it will be helpful (and useful) if some of the travis builds run on JDK8.
Not sure i can changed it though?

@Leemoonsoo

Copy link
Copy Markdown
Member

Adding a new entry under matrix.include section in the .travis.yml will run test on JDK8.
for example

 - jdk: "oraclejdk8"
env: SPARK_VER="1.6.0" HADOOP_VER="2.3" PROFILE="-Pspark-1.6 -Phadoop-2.3 -Ppyspark -Pscalding" BUILD_FLAG="package -Pbuild-distr" TEST_FLAG="verify -Pusing-packaged-distr"

If the test on jdk8 does not pass, because of other modules of Zeppelin, i suggest revert modification of .travis.yml in this PR and make separate PR and resolve https://issues.apache.org/jira/browse/ZEPPELIN-411

@felixcheung

Copy link
Copy Markdown
Member

@tzolov how would you like to proceed?

@asfgitasfgit closed this in c38a0a0May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close#83close#86close#125close#133close#139close#146close#193close#203close#246close#262close#264close#273close#291close#299close#320close#347close#389close#413close#423close#543close#560close#658close#670close#728close#765close#777close#782close#783close#812close#822close#841close#843close#878close#884close#918close#989close#1076close#1135close#1187close#1231close#1304close#1316close#1361close#1385close#1390close#1414close#1422close#1425close#1447close#1458close#1466close#1485close#1492close#1495close#1497close#1536close#1545close#1561close#1577close#1600close#1603close#1678close#1695close#1739close#1748close#1765close#1767close#1776close#1783close#1799
Sign up for freeto 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

@tzolov@felixcheung@Leemoonsoo