[GitHub] spark pull request: [SPARK-6157][CORE]Unroll unsuccessful memory_a...

2015-09-08 Thread suyanNone
Github user suyanNone commented on the pull request: https://github.com/apache/spark/pull/4887#issuecomment-138480264 sorry too late to see that, I will update it today --- 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-10484][SQL] Optimize the cartesian join...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8652#issuecomment-138465820 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-10484][SQL] Optimize the cartesian join...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8652#issuecomment-138465777 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10310][SQL]Using \t as the field delime...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8476#issuecomment-138474776 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-8552] [THRIFTSERVER] Using incorrect da...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7118#issuecomment-138463805 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-8552] [THRIFTSERVER] Using incorrect da...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7118#issuecomment-138463793 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138468564 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-10482] [ML] Add Python interface for ml...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138468566 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138468472 [Test build #42125 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42125/console) for PR 8650 at commit

[GitHub] spark pull request: [SPARK-10149] [CORE] [WIP] Locality Level is a...

2015-09-08 Thread wulei-bj-cn
Github user wulei-bj-cn commented on the pull request: https://github.com/apache/spark/pull/8533#issuecomment-138486424 @CodingCat Thanks so much for your suggestion! It does sound fair/smart to make the patch as cleaner as possible. As you suggested, I moved the fixes into

[GitHub] spark pull request: [SPARK-9741][SQL] Approximate Count Distinct u...

2015-09-08 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/8362#issuecomment-138493630 @rxin the dense version of HLL++ is ready. We could also add this, and add the sparse logic in a follow-up PR. Let me know what you think. I'll close if you'd rather

[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138464068 [Test build #42123 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42123/consoleFull) for PR 8583 at commit

[GitHub] spark pull request: [SPARK-10484][SQL] Optimize the cartesian join...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8652#issuecomment-138467535 [Test build #42126 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42126/consoleFull) for PR 8652 at commit

[GitHub] spark pull request: [SPARK-10310][SQL]Using \t as the field delime...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8476#issuecomment-138476140 [Test build #42127 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42127/consoleFull) for PR 8476 at commit

[GitHub] spark pull request: [SPARK-9963] [ML] RandomForest cleanup: replac...

2015-09-08 Thread trevorcai
Github user trevorcai commented on the pull request: https://github.com/apache/spark/pull/8609#issuecomment-138480624 Took a brief look - I have a quick high level thought. The motivation behind the patch is to replace the redundant functionality of `predictNodeIndex` with the

[GitHub] spark pull request: [SPARK-9818][SQL][WIP]Revert SPARK-6136 to ena...

2015-09-08 Thread yjshen
Github user yjshen commented on the pull request: https://github.com/apache/spark/pull/8101#issuecomment-138497422 Sorry for my late reply. MySQLIntegrationSuite would fail with: ``` Error Message Image pull failed: mysql:latest: ProgressMessage{id=null,

[GitHub] spark pull request: [SPARK-10441][SQL] Save data correctly to json...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8597#issuecomment-138449666 [Test build #1725 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1725/console) for PR 8597 at commit

[GitHub] spark pull request: [SPARK-10441] [SQL] [BRANCH-1.5] Save data cor...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8651#issuecomment-138456590 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-10441][SQL] Save data correctly to json...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8597#issuecomment-138456626 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-10441][SQL] Save data correctly to json...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8597#issuecomment-138456628 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10441] [SQL] [BRANCH-1.5] Save data cor...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8651#issuecomment-138456591 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10441] [SQL] [BRANCH-1.5] Save data cor...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8651#issuecomment-138456453 [Test build #42120 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42120/console) for PR 8651 at commit

[GitHub] spark pull request: [SPARK-10441][SQL] Save data correctly to json...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8597#issuecomment-138456494 [Test build #42119 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42119/console) for PR 8597 at commit

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138459076 [Test build #42122 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42122/console) for PR 8650 at commit

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138459005 [Test build #42122 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42122/consoleFull) for PR 8650 at commit

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138459078 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138459077 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-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138462685 Merged build started. --- 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

[GitHub] spark pull request: [Spark-7179][SQL]add tablenamepattern to hive ...

2015-09-08 Thread baishuo
Github user baishuo commented on the pull request: https://github.com/apache/spark/pull/6745#issuecomment-138462580 hi @andrewor14 @liancheng , This patch didnt pass some test. Can close it :) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...

2015-09-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/8583#discussion_r38897017 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetQuerySuite.scala --- @@ -229,4 +232,322 @@ class

[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138462606 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-8552] [THRIFTSERVER] Using incorrect da...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7118#issuecomment-138465214 [Test build #42124 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42124/consoleFull) for PR 7118 at commit

[GitHub] spark pull request: [SPARK-10480] [ML] Fix ML.LinearRegressionMode...

2015-09-08 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/8641#issuecomment-138493977 @yanboliang how about fixing SPARK-10479 here too? it's very similar. --- 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-9851] Support submitting map stages ind...

2015-09-08 Thread ryan-williams
Github user ryan-williams commented on a diff in the pull request: https://github.com/apache/spark/pull/8180#discussion_r38893879 --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala --- @@ -45,17 +45,65 @@ import

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138458027 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138458014 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138465382 [Test build #42125 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42125/consoleFull) for PR 8650 at commit

[GitHub] spark pull request: [SPARK-10484][SQL] Optimize the cartesian join...

2015-09-08 Thread chenghao-intel
GitHub user chenghao-intel opened a pull request: https://github.com/apache/spark/pull/8652 [SPARK-10484][SQL] Optimize the cartesian join with broadcast join for some cases In some cases, we can broadcast the smaller relation in cartesian join, which improve the performance

[GitHub] spark pull request: [SPARK-8552] [THRIFTSERVER] Using incorrect da...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7118#issuecomment-138469526 [Test build #42124 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42124/console) for PR 7118 at commit

[GitHub] spark pull request: [SPARK-8552] [THRIFTSERVER] Using incorrect da...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7118#issuecomment-138469627 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-8552] [THRIFTSERVER] Using incorrect da...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7118#issuecomment-138469620 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-10478][ML] Performance, organization, a...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8648#issuecomment-138447507 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10478][ML] Performance, organization, a...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8648#issuecomment-138447506 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-10478][ML] Performance, organization, a...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8648#issuecomment-138447486 [Test build #42121 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42121/console) for PR 8648 at commit

[GitHub] spark pull request: [SPARK-10441][SQL] Save data correctly to json...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8597#issuecomment-138451789 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10441][SQL] Save data correctly to json...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8597#issuecomment-138451786 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-10441][SQL] Save data correctly to json...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8597#issuecomment-138451412 [Test build #42118 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42118/console) for PR 8597 at commit

[GitHub] spark pull request: [SPARK-10301] [SQL] Fixes schema merging for n...

2015-09-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/8509#discussion_r38897192 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetQuerySuite.scala --- @@ -229,4 +229,81 @@ class

[GitHub] spark pull request: [SPARK-10310][SQL]Using \t as the field delime...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8476#issuecomment-138474836 Merged build started. --- 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

[GitHub] spark pull request: [DOC] Added R to the list of languages with "h...

2015-09-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/8646#discussion_r38905968 --- Diff: docs/quick-start.md --- @@ -217,13 +217,13 @@ a cluster, as described in the [programming guide](programming-guide.html#initia #

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138464950 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-10482] [ML] Add Python interface for ml...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8650#issuecomment-138464930 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10259] [ML] Add @since annotation to ml...

2015-09-08 Thread taishi-oss
Github user taishi-oss commented on the pull request: https://github.com/apache/spark/pull/8534#issuecomment-138488219 Sorry for being late. I modified @since annotations. --- 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-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138501734 [Test build #42123 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42123/console) for PR 8583 at commit

[GitHub] spark pull request: [SPARK-10117][MLLIB] Implement SQL data source...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8537#issuecomment-138511440 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-10117][MLLIB] Implement SQL data source...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8537#issuecomment-138511420 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10117][MLLIB] Implement SQL data source...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8537#issuecomment-138515041 [Test build #42129 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42129/consoleFull) for PR 8537 at commit

[GitHub] spark pull request: [SPARK-10480] [ML] Fix ML.LinearRegressionMode...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8641#issuecomment-138514053 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-10480] [ML] Fix ML.LinearRegressionMode...

2015-09-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/8641#discussion_r38912199 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala --- @@ -468,7 +468,9 @@ class LogisticRegressionModel private[ml]

[GitHub] spark pull request: Docs small fixes

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8629#issuecomment-138519652 [Test build #1728 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1728/console) for PR 8629 at commit

[GitHub] spark pull request: [SPARK-10117][MLLIB] Implement SQL data source...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8537#issuecomment-138525124 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10117][MLLIB] Implement SQL data source...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8537#issuecomment-138525122 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-10117][MLLIB] Implement SQL data source...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8537#issuecomment-138524680 [Test build #42129 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42129/console) for PR 8537 at commit

[GitHub] spark pull request: [SPARK-9851] Support submitting map stages ind...

2015-09-08 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8180#discussion_r38916650 --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala --- @@ -45,17 +45,65 @@ import

[GitHub] spark pull request: [SPARK-9851] Support submitting map stages ind...

2015-09-08 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8180#discussion_r38916643 --- Diff: core/src/main/scala/org/apache/spark/scheduler/ActiveJob.scala --- @@ -23,18 +23,42 @@ import org.apache.spark.TaskContext import

[GitHub] spark pull request: [SPARK-9851] Support submitting map stages ind...

2015-09-08 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8180#discussion_r38916631 --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala --- @@ -720,31 +843,82 @@ class DAGScheduler( try { // New

[GitHub] spark pull request: [SPARK-9851] Support submitting map stages ind...

2015-09-08 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/8180#issuecomment-138533050 LGTM except some nits. --- 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-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138502075 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-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138502078 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10484][SQL] Optimize the cartesian join...

2015-09-08 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/8652#issuecomment-138510593 Looks this is a part of #7417 --- 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

[GitHub] spark pull request: [SPARK-10480] [ML] Fix ML.LinearRegressionMode...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8641#issuecomment-138513973 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10480] [ML] Fix ML.LinearRegressionMode...

2015-09-08 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/8641#issuecomment-138513235 @srowen OK, I have updated this PR to fix both [SPARK-10479](https://issues.apache.org/jira/browse/SPARK-10479) and

[GitHub] spark pull request: [SPARK-6350][Mesos] Fine-grained mode schedule...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8653#issuecomment-138533670 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-6350][Mesos] Fine-grained mode schedule...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8653#issuecomment-138533603 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-6350][Mesos] Fine-grained mode schedule...

2015-09-08 Thread dragos
GitHub user dragos opened a pull request: https://github.com/apache/spark/pull/8653 [SPARK-6350][Mesos] Fine-grained mode scheduler respects mesosExecutor.cores This is a regression introduced in #4960, this commit fixes it and adds a test. @tnachen @andrewor14 please

[GitHub] spark pull request: [SPARK-10227] fatal warnings with sbt on Scala...

2015-09-08 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/8433#issuecomment-138507718 Just giving this one more go through Jenkins, but still LGTM. our test framework is just testing Scala 2.10, not 2.11. However, it seems like this patch can only make it

[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138519275 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138519290 Merged build started. --- 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

[GitHub] spark pull request: [WIP] [SPARK-8518] [ML] Log-linear models for ...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8611#issuecomment-138521533 [Test build #42128 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42128/console) for PR 8611 at commit

[GitHub] spark pull request: [SPARK-10149] [CORE] [WIP] Locality Level is a...

2015-09-08 Thread CodingCat
Github user CodingCat commented on a diff in the pull request: https://github.com/apache/spark/pull/8533#discussion_r38914955 --- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskLocation.scala --- @@ -35,16 +37,26 @@ case class ExecutorCacheTaskLocation(override val host:

[GitHub] spark pull request: [SPARK-10149] [CORE] [WIP] Locality Level is a...

2015-09-08 Thread CodingCat
Github user CodingCat commented on a diff in the pull request: https://github.com/apache/spark/pull/8533#discussion_r38915025 --- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskLocation.scala --- @@ -35,16 +37,26 @@ case class ExecutorCacheTaskLocation(override val host:

[GitHub] spark pull request: [SPARK-10480] [ML] Fix ML.LinearRegressionMode...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8641#issuecomment-138527276 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6350][Mesos] Fine-grained mode schedule...

2015-09-08 Thread dragos
Github user dragos commented on the pull request: https://github.com/apache/spark/pull/8653#issuecomment-138534010 I'm working on another fix for SPARK-7874, and will remove some of the duplication in tests, but I wanted to keep this PR as simple as possible (in line with the trivial

[GitHub] spark pull request: [SPARK-9066][SQL] Improve cartesian performanc...

2015-09-08 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/7417#issuecomment-138544481 @Sephiroth-Lin can you rebase 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 your project does

[GitHub] spark pull request: [SPARK-10484][SQL] Optimize the cartesian join...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8652#issuecomment-138500699 [Test build #42126 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42126/console) for PR 8652 at commit

[GitHub] spark pull request: [SPARK-10484][SQL] Optimize the cartesian join...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8652#issuecomment-138500846 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [WIP] [SPARK-8518] [ML] Log-linear models for ...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8611#issuecomment-138509440 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138520479 [Test build #42131 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42131/consoleFull) for PR 8583 at commit

[GitHub] spark pull request: [SPARK-10480] [ML] Fix ML.LinearRegressionMode...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8641#issuecomment-138527174 [Test build #42130 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42130/console) for PR 8641 at commit

[GitHub] spark pull request: [SPARK-10480] [ML] Fix ML.LinearRegressionMode...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8641#issuecomment-138527275 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-10310][SQL]Using \t as the field delime...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8476#issuecomment-138505965 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-10310][SQL]Using \t as the field delime...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8476#issuecomment-138505968 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10310][SQL]Using \t as the field delime...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8476#issuecomment-138505828 [Test build #42127 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42127/console) for PR 8476 at commit

[GitHub] spark pull request: [SPARK-10380] - Confusing examples in pyspark ...

2015-09-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/8647#discussion_r38910982 --- Diff: python/pyspark/sql/column.py --- @@ -329,6 +329,8 @@ def cast(self, dataType): [Row(ages=u'2'), Row(ages=u'5')] >>>

[GitHub] spark pull request: [WIP] [SPARK-8518] [ML] Log-linear models for ...

2015-09-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8611#issuecomment-138510966 [Test build #42128 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42128/consoleFull) for PR 8611 at commit

[GitHub] spark pull request: [SPARK-10480] [ML] Fix ML.LinearRegressionMode...

2015-09-08 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/8641#discussion_r38912950 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala --- @@ -468,7 +468,9 @@ class LogisticRegressionModel

[GitHub] spark pull request: [SPARK-10149] [CORE] [WIP] Locality Level is a...

2015-09-08 Thread CodingCat
Github user CodingCat commented on the pull request: https://github.com/apache/spark/pull/8533#issuecomment-138526861 ok, this patch is looking good to me (after the changes I mentioned above) time for others to judge @kayousterhout @srowen @andrewor14 @JoshRosen --- If

[GitHub] spark pull request: [SPARK-10484][SQL] Optimize the cartesian join...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8652#issuecomment-138500841 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-8518] [ML] Log-linear models for ...

2015-09-08 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/8611#issuecomment-138509653 @rotationsymmetry I agree with you, updated PR. Thanks for your comments. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [WIP] [SPARK-8518] [ML] Log-linear models for ...

2015-09-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8611#issuecomment-138509491 Merged build started. --- 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

  1   2   3   4   5   6   7   8   >