Skip to content

[ZEPPELIN-1081] Extract spark.r setting from interpreter-setting.json on Spark interpreter mudule - #1108

Closed
jongyoul wants to merge 1 commit into
apache:masterfrom
jongyoul:ZEPPELIN-1081
Closed

[ZEPPELIN-1081] Extract spark.r setting from interpreter-setting.json on Spark interpreter mudule#1108
jongyoul wants to merge 1 commit into
apache:masterfrom
jongyoul:ZEPPELIN-1081

Conversation

@jongyoul

@jongyouljongyoul commented Jun 30, 2016

Copy link
Copy Markdown
Member

What is this PR for?

Avoid setting spark.r without -Psparkr. This enables to use -Pr correctly.

What type of PR is it?

[Bug Fix]

Todos

  • - Separate settings with profile

What is the Jira issue?

How should this be tested?

  1. Build Zeppelin with -Pr
  2. Run codes with R

Screenshots (if appropriate)

Questions:

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

@jongyoul

Copy link
Copy Markdown
MemberAuthor

Ready to review. This is related to #1077

@jongyouljongyoul changed the title [ZEPPELIN-1081] Fixed not to include spark.r setting while not using -Psparkr[ZEPPELIN-1081] Fix not to include spark.r setting while not using -PsparkrJun 30, 2016
@jongyouljongyoul changed the title [ZEPPELIN-1081] Fix not to include spark.r setting while not using -Psparkr[ZEPPELIN-1081] Extract spark.r setting from interpreter-setting.json on Spark interpreter muduleJun 30, 2016
@jongyoul

Copy link
Copy Markdown
MemberAuthor

I'll merge it into master and branch-0.6

asfgit pushed a commit that referenced this pull request Jun 30, 2016
… on Spark interpreter mudule
### What is this PR for?
Avoid setting `spark.r` without `-Psparkr`. This enables to use `-Pr` correctly.
### What type of PR is it?
[Bug Fix]
### Todos
* [x] - Separate settings with profile
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-1081
### How should this be tested?
1. Build Zeppelin with `-Pr`
1. Run codes with R
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Jongyoul Lee <jongyoul@gmail.com>
Closes#1108 from jongyoul/ZEPPELIN-1081 and squashes the following commits:
d7b7481 [Jongyoul Lee] Fixed not to include spark.r setting while not using -Psparkr
(cherry picked from commit 4c66689)
Signed-off-by: Jongyoul Lee <jongyoul@apache.org>
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.

1 participant

@jongyoul