Skip to content

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 #751
jeffsteinmetz wants to merge 8 commits intoapache:masterfrom
jeffsteinmetz:ZEPPELIN-700

Conversation

@jeffsteinmetz
Copy link
Copy Markdown
Contributor

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:

  • 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

@jeffsteinmetz jeffsteinmetz mentioned this pull request Mar 8, 2016
3 tasks
@Leemoonsoo
Copy link
Copy Markdown
Member

Thanks for taking care of virtual machine script.

Looks good to me.

@Leemoonsoo
Copy link
Copy Markdown
Member

Merge into master if there're no more discussions

@echarles
Copy link
Copy Markdown
Member

echarles commented Apr 2, 2016

LGTM

@asfgit asfgit closed this in f8265bb Apr 3, 2016
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
@jeffsteinmetz jeffsteinmetz deleted the ZEPPELIN-700 branch April 6, 2016 17:50
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
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