[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13217 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-221133254 @shivaram Thank you so much. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-221132865 @HyukjinKwon - Thanks a lot for updating the documentation and for working on #13165 - I think there are a number of R users who use Windows and having SparkR work on Windows (at least for the purposes of trying it out) is useful for the Spark project. That said it has been hard to find people who have the time / resources to test SparkR on Windows machines. I wrote the initial instructions for building in Windows but a bunch of stuff has changed since then. I think this documentation change LGTM and we can come back and update it if we find a problem. Merging this to master, branch-2.0 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220780347 @HyukjinKwon, maybe we can merge this first. I will probably have a try later. cc @shivaram --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220751922 @sun-rui @steveloughran While it seems obviously better for someone to follow and test this, I wonder who is going to test this and leave some comments here. I opened this PR during working on https://github.com/apache/spark/pull/13165, which I started to work on because https://github.com/apache/spark/pull/7025 is getting stale for several months. In this way, it seems nobody would want to try this. (In that way, https://github.com/apache/spark/pull/13165 will also never be merged unless someone is trying to test. I would quit and close it if I should work on something which it seems will never be merged). Maybe it would be better, at least for this one, to merge this for users/developers to follow this document to test, since it is a minor which does not affect the code base. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220751693 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59051/ Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220751692 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220751654 **[Test build #59051 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59051/consoleFull)** for PR 13217 at commit [`aa2839c`](https://github.com/apache/spark/commit/aa2839ca7bc8b18dafa849a8bdc8d5119d1e46a1). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220751065 @sun-rui @steveloughran While it seems obviously better for someone to follow this and test, I wonder who is going to test this and leave some comments here. I opened this PR during working on https://github.com/apache/spark/pull/13165, which I started to work on because https://github.com/apache/spark/pull/7025 is getting stale for a few months. In this way, it seems nobody would want to try this. Maybe it would be better to merge this for users/developers to follow this document to test. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220750852 **[Test build #59051 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59051/consoleFull)** for PR 13217 at commit [`aa2839c`](https://github.com/apache/spark/commit/aa2839ca7bc8b18dafa849a8bdc8d5119d1e46a1). --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220624217 wait to see if someone can test the documentation. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user steveloughran commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220592819 1. It's nice to see someone sitting down to deal with the windows test problem. 1. Hadoop 2.8+ will fail meaningfully here, with an exception including a link to the Hadoop wiki entry [HADOOP-10775](https://issues.apache.org/jira/browse/HADOOP-10775). I've put a link to the entry in my suggested changes, so people trying to get things working have a direct link to that page, so can stay current with whatever changes are made (more docs, ASF official releases), ... 1. Someone should really test the documentation by following it. I must excuse myself on the basis that my VM is set up and I don't want to break things. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user steveloughran commented on a diff in the pull request: https://github.com/apache/spark/pull/13217#discussion_r64032005 --- Diff: R/WINDOWS.md --- @@ -11,3 +11,19 @@ include Rtools and R in `PATH`. directory in Maven in `PATH`. 4. Set `MAVEN_OPTS` as described in [Building Spark](http://spark.apache.org/docs/latest/building-spark.html). 5. Open a command shell (`cmd`) in the Spark directory and run `mvn -DskipTests -Psparkr package` + +## Unit tests + +To run existing unit tests in SparkR on Windows, the following setps are required (the steps below suppose you are in Spark root directory) + +1. Set `HADOOP_HOME`. +2. Download `winutils.exe` and locate this in `$HADOOP_HOME/bin`. + +It seems not requiring installing Hadoop but only this `winutils.exe`. It seems not included in Hadoop official binary releases so it should be built from source but it seems it is able to be downloaded from community (e.g. [steveloughran/winutils](https://github.com/steveloughran/winutils)). --- End diff -- I wouldn't recommend putting it under the root of the project, as that only complicates the source tree and path cleanup; an adjacent directory works. And I think you may find that `HADOOP.DLL` is needed in places, as there are some JNI calls related to local file access and permissions/ACLs I'd suggest the following text: To run the SparkR unit tests on Windows, the following steps are required âassuming you are in the Spark root directory and do not have Apache Hadoop installed already: 1. `cd ..` 1. `mkdir hadoop` 1. Download the relevant Hadoop bin package from [steveloughran/winutils](https://github.com/steveloughran/winutils). While these are not official ASF artifacts, they are built from the ASF release git hashes by a Hadoop PMC member on a dedicated Windows VM. 1. Install the files into `hadoop\bin`; make sure that `winutils.exe` and `hadoop.dll` are present. 1. Set the environment variable `HADOOP_HOME` to the full path to the newly created `hadoop` directory. . --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/13217#discussion_r64026341 --- Diff: R/WINDOWS.md --- @@ -11,3 +11,19 @@ include Rtools and R in `PATH`. directory in Maven in `PATH`. 4. Set `MAVEN_OPTS` as described in [Building Spark](http://spark.apache.org/docs/latest/building-spark.html). 5. Open a command shell (`cmd`) in the Spark directory and run `mvn -DskipTests -Psparkr package` + +## Unit tests + +To run existing unit tests in SparkR on Windows, the following setps are required (the steps below suppose you are in Spark root directory) + +1. Set `HADOOP_HOME`. +2. Download `winutils.exe` and locate this in `$HADOOP_HOME/bin`. + +It seems not requiring installing Hadoop but only this `winutils.exe`. It seems not included in Hadoop official binary releases so it should be built from source but it seems it is able to be downloaded from community (e.g. [steveloughran/winutils](https://github.com/steveloughran/winutils)). --- End diff -- Thank you. I will wait for the comment and will fix. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/13217#discussion_r64025769 --- Diff: R/WINDOWS.md --- @@ -11,3 +11,19 @@ include Rtools and R in `PATH`. directory in Maven in `PATH`. 4. Set `MAVEN_OPTS` as described in [Building Spark](http://spark.apache.org/docs/latest/building-spark.html). 5. Open a command shell (`cmd`) in the Spark directory and run `mvn -DskipTests -Psparkr package` + +## Unit tests + +To run existing unit tests in SparkR on Windows, the following setps are required (the steps below suppose you are in Spark root directory) + +1. Set `HADOOP_HOME`. +2. Download `winutils.exe` and locate this in `$HADOOP_HOME/bin`. + +It seems not requiring installing Hadoop but only this `winutils.exe`. It seems not included in Hadoop official binary releases so it should be built from source but it seems it is able to be downloaded from community (e.g. [steveloughran/winutils](https://github.com/steveloughran/winutils)). + +3. Run unit-tests for SparkR by running below (you need to install the [testthat](http://cran.r-project.org/web/packages/testthat/index.html) package first): --- End diff -- Thank you. I will fix. I wrote it just in case becase the commends are a little bit different (I am not used to Windows and it took me a while to find out the equivalent commands). --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/13217#discussion_r64025198 --- Diff: R/WINDOWS.md --- @@ -11,3 +11,19 @@ include Rtools and R in `PATH`. directory in Maven in `PATH`. 4. Set `MAVEN_OPTS` as described in [Building Spark](http://spark.apache.org/docs/latest/building-spark.html). 5. Open a command shell (`cmd`) in the Spark directory and run `mvn -DskipTests -Psparkr package` + +## Unit tests + +To run existing unit tests in SparkR on Windows, the following setps are required (the steps below suppose you are in Spark root directory) --- End diff -- Oh. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13217#discussion_r64022383 --- Diff: R/WINDOWS.md --- @@ -11,3 +11,19 @@ include Rtools and R in `PATH`. directory in Maven in `PATH`. 4. Set `MAVEN_OPTS` as described in [Building Spark](http://spark.apache.org/docs/latest/building-spark.html). 5. Open a command shell (`cmd`) in the Spark directory and run `mvn -DskipTests -Psparkr package` + +## Unit tests + +To run existing unit tests in SparkR on Windows, the following setps are required (the steps below suppose you are in Spark root directory) --- End diff -- Parenthetical is fine as a sentence by itself. "in _the_ Spark root ..." --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13217#discussion_r64022461 --- Diff: R/WINDOWS.md --- @@ -11,3 +11,19 @@ include Rtools and R in `PATH`. directory in Maven in `PATH`. 4. Set `MAVEN_OPTS` as described in [Building Spark](http://spark.apache.org/docs/latest/building-spark.html). 5. Open a command shell (`cmd`) in the Spark directory and run `mvn -DskipTests -Psparkr package` + +## Unit tests + +To run existing unit tests in SparkR on Windows, the following setps are required (the steps below suppose you are in Spark root directory) + +1. Set `HADOOP_HOME`. +2. Download `winutils.exe` and locate this in `$HADOOP_HOME/bin`. + +It seems not requiring installing Hadoop but only this `winutils.exe`. It seems not included in Hadoop official binary releases so it should be built from source but it seems it is able to be downloaded from community (e.g. [steveloughran/winutils](https://github.com/steveloughran/winutils)). + +3. Run unit-tests for SparkR by running below (you need to install the [testthat](http://cran.r-project.org/web/packages/testthat/index.html) package first): --- End diff -- "unit tests" and "by running the command below". Again parenthetical can be a sentence. This step is already documented in R docs though. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13217#discussion_r64022325 --- Diff: R/WINDOWS.md --- @@ -11,3 +11,19 @@ include Rtools and R in `PATH`. directory in Maven in `PATH`. 4. Set `MAVEN_OPTS` as described in [Building Spark](http://spark.apache.org/docs/latest/building-spark.html). 5. Open a command shell (`cmd`) in the Spark directory and run `mvn -DskipTests -Psparkr package` + +## Unit tests + +To run existing unit tests in SparkR on Windows, the following setps are required (the steps below suppose you are in Spark root directory) + +1. Set `HADOOP_HOME`. +2. Download `winutils.exe` and locate this in `$HADOOP_HOME/bin`. + +It seems not requiring installing Hadoop but only this `winutils.exe`. It seems not included in Hadoop official binary releases so it should be built from source but it seems it is able to be downloaded from community (e.g. [steveloughran/winutils](https://github.com/steveloughran/winutils)). --- End diff -- CC @steveloughran for comment. I think the paragraph should start with "It is not included in the Hadoop binary releases, so However it is downloadable from, for example [...]" --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220536852 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220536855 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58962/ Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220536743 **[Test build #58962 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58962/consoleFull)** for PR 13217 at commit [`acb9363`](https://github.com/apache/spark/commit/acb93639d2183c8d52281db6d21ec9107e282461). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220535184 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220535185 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58963/ Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220535153 **[Test build #58963 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58963/consoleFull)** for PR 13217 at commit [`b62b631`](https://github.com/apache/spark/commit/b62b631e98570b16d4d000e23b2e5928662ff941). * This patch **fails MiMa tests**. * This patch merges cleanly. * This patch adds no public classes. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220533401 **[Test build #58963 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58963/consoleFull)** for PR 13217 at commit [`b62b631`](https://github.com/apache/spark/commit/b62b631e98570b16d4d000e23b2e5928662ff941). --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220532615 **[Test build #58962 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58962/consoleFull)** for PR 13217 at commit [`acb9363`](https://github.com/apache/spark/commit/acb93639d2183c8d52281db6d21ec9107e282461). --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220532414 cc @sun-rui --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org