Skip to content

Fixed path for rzeppelin package for some users. [ZEPPELIN-810] Revise R Tutorial notebook [ZEPPELIN-811] - #841

Closed
elbamos wants to merge 1 commit into
apache:masterfrom
elbamos:paths
Closed

Fixed path for rzeppelin package for some users. [ZEPPELIN-810] Revise R Tutorial notebook [ZEPPELIN-811]#841
elbamos wants to merge 1 commit into
apache:masterfrom
elbamos:paths

Conversation

@elbamos

Copy link
Copy Markdown
Contributor

What is this PR for?

  1. Fix a bug reported by J. Muedsam, where if Zeppelin is launched from a working directory that isn't the ZEPPELIN_HOME, it could fail to find the rzeppelin R package.
  2. Fixes to the R tutorial notebook.

What type of PR is it?

[Bug Fix | Documentation]

Todos

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-810
https://issues.apache.org/jira/browse/ZEPPELIN-811

How should this be tested?

The existing unit and integration tests should handle 810.
For 811, try the notebook.

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? No.
  • Is there breaking changes for older versions? No.
  • Does this needs documentation? No.

@jeffsteinmetz

Copy link
Copy Markdown
Contributor

It's hard to tell what has changed in notebook/r/note.json, perhaps it could be its own PR and limit this PR to Fixed path for rzeppelin package?
There are 155 additions, 298 deletions to note.json

@bzz

bzz commented Apr 19, 2016

Copy link
Copy Markdown
Member

Not sure if it is ready for review yet, but I second @jeffsteinmetz on preferably having 2 separate PRs.

@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

@elbamos@jeffsteinmetz@bzz