Add R Dependencies to Virtual Machine Script. Zeppelin-700 #751
Closed
jeffsteinmetz wants to merge 8 commits intoapache:masterfrom
jeffsteinmetz:ZEPPELIN-700
Closed
Add R Dependencies to Virtual Machine Script. Zeppelin-700 #751jeffsteinmetz wants to merge 8 commits intoapache:masterfrom jeffsteinmetz:ZEPPELIN-700
jeffsteinmetz wants to merge 8 commits intoapache:masterfrom
jeffsteinmetz:ZEPPELIN-700
Conversation
…encies for the R Interpreter
…encies for the R Interpreter
… package first so repr can be installed.
… package first so repr can be installed.
3 tasks
Member
|
Thanks for taking care of virtual machine script. Looks good to me. |
Member
|
Merge into master if there're no more discussions |
Member
|
LGTM |
lresende
pushed a commit
to lresende/incubator-zeppelin
that referenced
this pull request
Apr 3, 2016
### What is this PR for? Improvement of Virtual Machine Script to support R interpretor ### What type of PR is it? Improvement of Virtual Machine Script to support R interpretor ### Todos * [x] - Test with apache#208 * [x] - Test with apache#702 ### Is there a relevant Jira issue? Zeppelin-700 ### How should this be tested? Follow the steps in this Read Me to build a VM from scratch: https://github.com/apache/incubator-zeppelin/blob/master/scripts/vagrant/zeppelin-dev/README.md ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? Added to all headers * Is there breaking changes for older versions? No * Does this needs documentation? Yes, this will be a separate PR to update docs and README Author: Jeff Steinmetz <[email protected]> Closes apache#751 from jeffsteinmetz/ZEPPELIN-700 and squashes the following commits: e03dba5 [Jeff Steinmetz] update to support R interpreter sparkr build profile a9a2052 [Jeff Steinmetz] add base64enc, repr and htmltools bdcdf5f [Jeff Steinmetz] removed packages not required for pr702. repr not needed - base64encode not needed - htmltools not needed d497994 [Jeff Steinmetz] fix so that devtools will install 5643fc6 [Jeff Steinmetz] plotting in r interpreter requires the repr package. install devtools package first so repr can be installed. fb18a2b [Jeff Steinmetz] plotting in r interpreter requires the repr package. install devtools package first so repr can be installed. ef8f638 [Jeff Steinmetz] ZEPPELIN-700. Add Ansible R role to Virtual Machine to support dependencies for the R Interpreter b940bcd [Jeff Steinmetz] ZEPPELIN-700. Add Ansible R role to Virtual Machine to support dependencies for the R Interpreter
onkarshedge
pushed a commit
to onkarshedge/incubator-zeppelin
that referenced
this pull request
May 11, 2016
### What is this PR for? Improvement of Virtual Machine Script to support R interpretor ### What type of PR is it? Improvement of Virtual Machine Script to support R interpretor ### Todos * [x] - Test with apache#208 * [x] - Test with apache#702 ### Is there a relevant Jira issue? Zeppelin-700 ### How should this be tested? Follow the steps in this Read Me to build a VM from scratch: https://github.com/apache/incubator-zeppelin/blob/master/scripts/vagrant/zeppelin-dev/README.md ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? Added to all headers * Is there breaking changes for older versions? No * Does this needs documentation? Yes, this will be a separate PR to update docs and README Author: Jeff Steinmetz <[email protected]> Closes apache#751 from jeffsteinmetz/ZEPPELIN-700 and squashes the following commits: e03dba5 [Jeff Steinmetz] update to support R interpreter sparkr build profile a9a2052 [Jeff Steinmetz] add base64enc, repr and htmltools bdcdf5f [Jeff Steinmetz] removed packages not required for pr702. repr not needed - base64encode not needed - htmltools not needed d497994 [Jeff Steinmetz] fix so that devtools will install 5643fc6 [Jeff Steinmetz] plotting in r interpreter requires the repr package. install devtools package first so repr can be installed. fb18a2b [Jeff Steinmetz] plotting in r interpreter requires the repr package. install devtools package first so repr can be installed. ef8f638 [Jeff Steinmetz] ZEPPELIN-700. Add Ansible R role to Virtual Machine to support dependencies for the R Interpreter b940bcd [Jeff Steinmetz] ZEPPELIN-700. Add Ansible R role to Virtual Machine to support dependencies for the R Interpreter
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is this PR for?
Improvement of Virtual Machine Script to support R interpretor
What type of PR is it?
Improvement of Virtual Machine Script to support R interpretor
Todos
Is there a relevant Jira issue?
Zeppelin-700
How should this be tested?
Follow the steps in this Read Me to build a VM from scratch:
https://github.com/apache/incubator-zeppelin/blob/master/scripts/vagrant/zeppelin-dev/README.md
Screenshots (if appropriate)
Questions: