[GitHub] spark pull request: [SPARK-15451][build] Use jdk7's rt.jar when av...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13272#issuecomment-221641399 **[Test build #59282 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59282/consoleFull)** for PR 13272 at commit

[GitHub] spark pull request: [SPARK-15451][build] Use jdk7's rt.jar when av...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13272#issuecomment-221641407 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-25 Thread wangmiao1981
Github user wangmiao1981 commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221641136 I comment out the failed case. There is no other failures. As mentioned by @shivaram , we can try to fix the pipeRDD in a separate JIRA. I suspect it is related

[GitHub] spark pull request: [SPARK-15451][build] Use jdk7's rt.jar when av...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13272#issuecomment-221640245 **[Test build #59282 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59282/consoleFull)** for PR 13272 at commit

[GitHub] spark pull request: [SPARK-15451][build] Use jdk7's rt.jar when av...

2016-05-25 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/13272#issuecomment-221639155 one more round of negative tests, then this is good to go. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-13850] Force the sorter to Spill when n...

2016-05-25 Thread sitalkedia
Github user sitalkedia commented on the pull request: https://github.com/apache/spark/pull/13107#issuecomment-221638365 @rxin - We have seen this issue go away when we limit the pointer array size within 1G. Updated the PR to set that as a default value. @davies - There is

[GitHub] spark pull request: typos fix for files in [mllib] [streaming] and...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13298#issuecomment-221637482 **[Test build #59281 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59281/consoleFull)** for PR 13298 at commit

[GitHub] spark pull request: typos fix for files in [mllib] and [streaming]...

2016-05-25 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13298#issuecomment-221637148 Jenkins test this please --- 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

[GitHub] spark pull request: [YARN][Doc][Minor] Remove several obsolete env...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13296#issuecomment-221631478 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [MINOR][CORE] Fix a HadoopRDD log message and ...

2016-05-25 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/13294#issuecomment-221635222 Thank you for review, @srowen . Yep. Those are all I've seen in `core/src/main/scala/../rdd`. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: typos fix for files in [mllib] and [streaming]...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13298#issuecomment-221631904 Can one of the admins verify this patch? --- 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

[GitHub] spark pull request: [YARN][Doc][Minor] Remove several obsolete env...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13296#issuecomment-221631466 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

[GitHub] spark pull request: [YARN][Doc][Minor] Remove several obsolete env...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13296#issuecomment-221631201 **[Test build #59278 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59278/consoleFull)** for PR 13296 at commit

[GitHub] spark pull request: typos fix for files in [mllib] and [streaming]...

2016-05-25 Thread lfzCarlosC
GitHub user lfzCarlosC opened a pull request: https://github.com/apache/spark/pull/13298 typos fix for files in [mllib] and [streaming] ## What changes were proposed in this pull request? fixed typos for source code for component [mllib] and [streaming]

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/13262#discussion_r64603279 --- Diff: docs/ml-advanced.md --- @@ -4,10 +4,85 @@ title: Advanced topics - spark.ml displayTitle: Advanced topics - spark.ml --- -#

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/13262#discussion_r64602664 --- Diff: docs/ml-advanced.md --- @@ -4,10 +4,85 @@ title: Advanced topics - spark.ml displayTitle: Advanced topics - spark.ml --- -#

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/13262#discussion_r64602162 --- Diff: docs/ml-advanced.md --- @@ -4,10 +4,85 @@ title: Advanced topics - spark.ml displayTitle: Advanced topics - spark.ml --- -#

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/13262#discussion_r64601181 --- Diff: docs/ml-advanced.md --- @@ -4,10 +4,85 @@ title: Advanced topics - spark.ml displayTitle: Advanced topics - spark.ml --- -#

[GitHub] spark pull request: [SPARK-15491][SQL]fix assertion failure for JD...

2016-05-25 Thread huaxingao
Github user huaxingao commented on the pull request: https://github.com/apache/spark/pull/13287#issuecomment-221617823 @rxin Sorry, I didn't notice your comment until this morning. Reformatted the description. Thanks! --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13297#issuecomment-221612556 **[Test build #59280 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59280/consoleFull)** for PR 13297 at commit

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread vruusmann
Github user vruusmann closed the pull request at: https://github.com/apache/spark/pull/13293 --- 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

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread vruusmann
GitHub user vruusmann opened a pull request: https://github.com/apache/spark/pull/13297 [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.15 ## What changes were proposed in this pull request? See https://issues.apache.org/jira/browse/SPARK-15523 This PR replaces PR

[GitHub] spark pull request: [SPARK-15515] [SPARK-15514] [SQL] Error Handli...

2016-05-25 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13283#issuecomment-221606617 Thank you! --- 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

[GitHub] spark pull request: [YARN][Doc][Minor] Remove several obsolete env...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13296#issuecomment-221598197 **[Test build #59278 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59278/consoleFull)** for PR 13296 at commit

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread vruusmann
Github user vruusmann commented on the pull request: https://github.com/apache/spark/pull/13293#issuecomment-221600087 I'll go and read Apache Spark docs about PR guidelines. Looks like I missed this deps thing. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13293#issuecomment-221598223 **[Test build #59279 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59279/consoleFull)** for PR 13293 at commit

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13293#issuecomment-221597616 Jenkins test this please --- 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

[GitHub] spark pull request: [YARN][Doc][Minor] Remove several obsolete env...

2016-05-25 Thread jerryshao
GitHub user jerryshao opened a pull request: https://github.com/apache/spark/pull/13296 [YARN][Doc][Minor] Remove several obsolete env variables and update the doc ## What changes were proposed in this pull request? Remove several obsolete env variables not supported for

[GitHub] spark pull request: [YARN][Doc][Minor] Remove several obsolete env...

2016-05-25 Thread jerryshao
Github user jerryshao commented on a diff in the pull request: https://github.com/apache/spark/pull/13296#discussion_r64585570 --- Diff: conf/spark-env.sh.template --- @@ -40,10 +40,6 @@ # - SPARK_EXECUTOR_CORES, Number of cores for the executors (Default: 1). # -

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13293#issuecomment-221598788 **[Test build #59279 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59279/consoleFull)** for PR 13293 at commit

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13293#issuecomment-221598803 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13293#issuecomment-221598377 You'll need to run dev/test-dependencies to update the deps files. Are the exclusions definitely not needed? You would know best. Are there any

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13293#issuecomment-221598797 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

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13293#issuecomment-221597567 Jenkins add to whitelist --- 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

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-25 Thread sun-rui
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-221585022 @shivaram, I did not mean SQL row UDF. It's something like below: ``` dapply_row <- function(x, FUN, schema) { dapply(x, function(x) {

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221586554 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221586546 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

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221586539 **[Test build #59277 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59277/consoleFull)** for PR 13165 at commit

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-25 Thread sun-rui
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-221586542 @shivaram, I did not mean SQL row UDF. It's something like below: ``` dapply_row <- function(x, FUN, schema) { dapply(x, function(x) {

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221585787 **[Test build #59277 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59277/consoleFull)** for PR 13165 at commit

[GitHub] spark pull request: [SPARK-15294][SPARKR][MINOR] Add pivot functio...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13295#issuecomment-221585662 Can one of the admins verify this patch? --- 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

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/13165#discussion_r64576800 --- Diff: R/pkg/R/client.R --- @@ -60,6 +60,15 @@ generateSparkSubmitArgs <- function(args, sparkHome, jars, sparkSubmitOpts, pack combinedArgs

[GitHub] spark pull request: [SPARK-15294][SPARKR][MINOR] Add pivot functio...

2016-05-25 Thread mhnatiuk
GitHub user mhnatiuk opened a pull request: https://github.com/apache/spark/pull/13295 [SPARK-15294][SPARKR][MINOR] Add pivot functionality to SparkR ## What changes were proposed in this pull request? Added pivot functionality to SparkR ## How was this patch

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221576056 @shivaram and @sun-rui I addressed the comment, added a test for this PR and fixed some tests not working on Windows due to path separator. This PR also

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread sun-rui
Github user sun-rui commented on a diff in the pull request: https://github.com/apache/spark/pull/13165#discussion_r64576193 --- Diff: R/pkg/R/client.R --- @@ -60,6 +60,15 @@ generateSparkSubmitArgs <- function(args, sparkHome, jars, sparkSubmitOpts, pack combinedArgs

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221581990 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221581253 **[Test build #59276 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59276/consoleFull)** for PR 13165 at commit

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221581975 **[Test build #59276 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59276/consoleFull)** for PR 13165 at commit

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221581986 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

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221577659 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

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221577653 **[Test build #59275 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59275/consoleFull)** for PR 13165 at commit

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221576970 **[Test build #59275 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59275/consoleFull)** for PR 13165 at commit

[GitHub] spark pull request: [SPARK-8603][SPARKR] Incorrect file separator ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-221577662 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: SPARK-9044 Fix "Storage" tab in UI so that it ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13264#issuecomment-221564246 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

[GitHub] spark pull request: SPARK-9044 Fix "Storage" tab in UI so that it ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13264#issuecomment-221563986 **[Test build #59274 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59274/consoleFull)** for PR 13264 at commit

[GitHub] spark pull request: SPARK-9044 Fix "Storage" tab in UI so that it ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13264#issuecomment-221564250 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [MINOR][CORE] Fix a HadoopRDD log message and ...

2016-05-25 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13294#issuecomment-221556317 Ok, a little random as a set of changes but unobjectionable. Are there more unused imports? --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-12194] [Spark Core] Sink for reporting ...

2016-05-25 Thread MLnick
Github user MLnick commented on the pull request: https://github.com/apache/spark/pull/10187#issuecomment-221551531 @kapilsingh5050 could you close this PR please? I think this can live outside of Spark core as a separate repo, and could be listed in Spark packages. --- If your

[GitHub] spark pull request: [MINOR][CORE] Fix a HadoopRDD log message and ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13294#issuecomment-221546279 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

[GitHub] spark pull request: [MINOR][CORE] Fix a HadoopRDD log message and ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13294#issuecomment-221546281 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [MINOR][CORE] Fix a HadoopRDD log message and ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13294#issuecomment-221546078 **[Test build #59273 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59273/consoleFull)** for PR 13294 at commit

[GitHub] spark pull request: [SPARK-12194] [Spark Core] Sink for reporting ...

2016-05-25 Thread gt-rocker
Github user gt-rocker commented on a diff in the pull request: https://github.com/apache/spark/pull/10187#discussion_r6457 --- Diff: core/src/main/scala/org/apache/spark/metrics/sink/OpenTsdbSink.scala --- @@ -0,0 +1,165 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: SPARK-9044 Fix "Storage" tab in UI so that it ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13264#issuecomment-221539959 **[Test build #59274 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59274/consoleFull)** for PR 13264 at commit

[GitHub] spark pull request: SPARK-9044 Fix "Storage" tab in UI so that it ...

2016-05-25 Thread lgieron
Github user lgieron commented on a diff in the pull request: https://github.com/apache/spark/pull/13264#discussion_r64552539 --- Diff: core/src/test/scala/org/apache/spark/ui/storage/StorageTabSuite.scala --- @@ -179,6 +179,25 @@ class StorageTabSuite extends SparkFunSuite with

[GitHub] spark pull request: [SPARK-10530] [CORE] Kill other task attempts ...

2016-05-25 Thread devaraj-kavali
Github user devaraj-kavali commented on the pull request: https://github.com/apache/spark/pull/11996#issuecomment-221531103 @kayousterhout, can you have look into this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-15493][SQL] default QuoteEscapingEnable...

2016-05-25 Thread jurriaan
Github user jurriaan commented on a diff in the pull request: https://github.com/apache/spark/pull/13267#discussion_r64547155 --- Diff: sql/core/pom.xml --- @@ -39,7 +39,7 @@ com.univocity univocity-parsers - 2.1.0 + 2.1.1 ---

[GitHub] spark pull request: [SPARK-12978][SQL] Skip unnecessary final grou...

2016-05-25 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10896#issuecomment-221528445 @hvanhovell yeah, it is up-to-dated. --- 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

[GitHub] spark pull request: SPARK-9044 Fix "Storage" tab in UI so that it ...

2016-05-25 Thread lgieron
Github user lgieron commented on a diff in the pull request: https://github.com/apache/spark/pull/13264#discussion_r64545642 --- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala --- @@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:

[GitHub] spark pull request: [SPARK-12978][SQL] Skip unnecessary final grou...

2016-05-25 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10896#issuecomment-221525267 @maropu I'll take a look today. Is the description up-to-date? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13262#issuecomment-221524890 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

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13262#issuecomment-221524891 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13262#issuecomment-221524773 **[Test build #59271 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59271/consoleFull)** for PR 13262 at commit

[GitHub] spark pull request: [Spark 15129][R][DOC][WIP]R API changes in ML

2016-05-25 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/13285#issuecomment-221524784 @GayathriMurali 1, Please clear the change for ```ml-features.md``` which is irrelevant to this issue. 2, We can use ```include_example``` tag to directly

[GitHub] spark pull request: [SPARK-15493][SQL] default QuoteEscapingEnable...

2016-05-25 Thread jurriaan
Github user jurriaan commented on a diff in the pull request: https://github.com/apache/spark/pull/13267#discussion_r64544204 --- Diff: python/pyspark/sql/readwriter.py --- @@ -787,6 +787,9 @@ def csv(self, path, mode=None, compression=None, sep=None, quote=None, escape=No

[GitHub] spark pull request: [MINOR][CORE] Fix a HadoopRDD log message and ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13294#issuecomment-221524171 **[Test build #59273 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59273/consoleFull)** for PR 13294 at commit

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13262#issuecomment-221523746 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

[GitHub] spark pull request: [MINOR][CORE] Fix a HadoopRDD log message and ...

2016-05-25 Thread dongjoon-hyun
GitHub user dongjoon-hyun opened a pull request: https://github.com/apache/spark/pull/13294 [MINOR][CORE] Fix a HadoopRDD log message and remove unused imports in rdd files. ## What changes were proposed in this pull request? This PR fixes the following typos in log

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13262#issuecomment-221523749 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13262#issuecomment-221523625 **[Test build #59270 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59270/consoleFull)** for PR 13262 at commit

[GitHub] spark pull request: [WIP] [SPARK-10903] [SPARKR] R - Simplify SQLC...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9192#issuecomment-221522960 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

[GitHub] spark pull request: [WIP] [SPARK-10903] [SPARKR] R - Simplify SQLC...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9192#issuecomment-221522855 **[Test build #59272 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59272/consoleFull)** for PR 9192 at commit

[GitHub] spark pull request: [WIP] [SPARK-10903] [SPARKR] R - Simplify SQLC...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9192#issuecomment-221522961 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/13262#issuecomment-221521088 @sethah @jkbradley Thanks for your comments! I addressed most of the comments and responded inline. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13286#issuecomment-221520294 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

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/13262#discussion_r64541660 --- Diff: docs/ml-advanced.md --- @@ -4,10 +4,85 @@ title: Advanced topics - spark.ml displayTitle: Advanced topics - spark.ml --- -#

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13286#issuecomment-221520299 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13286#issuecomment-221520107 **[Test build #59268 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59268/consoleFull)** for PR 13286 at commit

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/13262#discussion_r64541319 --- Diff: docs/ml-advanced.md --- @@ -4,10 +4,85 @@ title: Advanced topics - spark.ml displayTitle: Advanced topics - spark.ml --- -#

[GitHub] spark pull request: [WIP] [SPARK-10903] [SPARKR] R - Simplify SQLC...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9192#issuecomment-221519516 **[Test build #59272 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59272/consoleFull)** for PR 9192 at commit

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/13262#discussion_r64541027 --- Diff: docs/ml-advanced.md --- @@ -4,10 +4,85 @@ title: Advanced topics - spark.ml displayTitle: Advanced topics - spark.ml --- -#

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/13262#discussion_r64540348 --- Diff: docs/ml-advanced.md --- @@ -4,10 +4,85 @@ title: Advanced topics - spark.ml displayTitle: Advanced topics - spark.ml --- -#

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13293#issuecomment-221516043 Can one of the admins verify this patch? --- 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

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13262#issuecomment-221516131 **[Test build #59271 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59271/consoleFull)** for PR 13262 at commit

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/13262#discussion_r64539181 --- Diff: docs/ml-advanced.md --- @@ -4,10 +4,85 @@ title: Advanced topics - spark.ml displayTitle: Advanced topics - spark.ml --- -#

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/13262#discussion_r64538677 --- Diff: docs/ml-advanced.md --- @@ -4,10 +4,85 @@ title: Advanced topics - spark.ml displayTitle: Advanced topics - spark.ml --- -#

[GitHub] spark pull request: [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.1...

2016-05-25 Thread vruusmann
GitHub user vruusmann opened a pull request: https://github.com/apache/spark/pull/13293 [SPARK-15523][ML][MLLIB] Update JPMML to 1.2.15 ## What changes were proposed in this pull request? See https://issues.apache.org/jira/browse/SPARK-15523 ## How was this patch

[GitHub] spark pull request: [SPARK-11959] [SPARK-15484] [Doc] [ML] Documen...

2016-05-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13262#issuecomment-221514935 **[Test build #59270 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59270/consoleFull)** for PR 13262 at commit

[GitHub] spark pull request: [SPARK-15324] [SQL] Add the takeSample functio...

2016-05-25 Thread burness
Github user burness commented on the pull request: https://github.com/apache/spark/pull/13116#issuecomment-221513002 @hvanhovell yeah, I agree with your opinion! Thanks --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-15436][SQL] Remove DescribeFunction and...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13292#issuecomment-221512348 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-15436][SQL] Remove DescribeFunction and...

2016-05-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13292#issuecomment-221512346 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

<    1   2   3   4   5   6   7   >