[GitHub] spark pull request: [SPARK-11878][SQL]: Eliminate distribute by in...

2016-01-05 Thread saucam
Github user saucam commented on the pull request: https://github.com/apache/spark/pull/9858#issuecomment-169240395 Hey @marmbrus have reverted to 46e7419 --- 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-7675][ML][PYSpark] sparkml params type ...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9581#issuecomment-169246826 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-7675][ML][PYSpark] sparkml params type ...

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

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48930838 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/patterns.scala --- @@ -170,17 +172,22 @@ object ExtractFiltersAndInnerJoins

[GitHub] spark pull request: [SPARK-7675][ML][PYSpark] sparkml params type ...

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

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10577#issuecomment-169249458 Thank you for your suggestions! Will pay more attentions on this part. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-12433][CORE][TEST][RFC/WIP] Make parts ...

2016-01-05 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10614#issuecomment-169249309 Sorry I don't think we should have this. Also Spark runtime shouldn't depend on a specific version of scalatest. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-5682][Core] Add encrypted shuffle in sp...

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

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

2016-01-05 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/10583#issuecomment-169204037 Does case-sensitivity config still work with the new parser? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-12659] fix NPE in UnsafeExternalSorter ...

2016-01-05 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/10606#issuecomment-169203949 @JoshRosen I'm merging this into master and 1.6 branch to unblock another PR, will add @Nullable in that PR. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-3873] [core] Import ordering fixes.

2016-01-05 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/10578#issuecomment-169205658 Yay. I'm gonna merge this before I have to fix more conflicts... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-3873] [tests] Import ordering fixes.

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

[GitHub] spark pull request: [SPARK-3873] [core] Import ordering fixes.

2016-01-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10578 --- 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-12393] [SparkR] Add read.text and write...

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

[GitHub] spark pull request: [SPARK-12581][SQL] Support case-sensitive tabl...

2016-01-05 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10523#issuecomment-169215749 @yhuai Ah..., I misunderstood. Okay and how about turning on/off case sensitivity in configurations, e.g., `spark.sql.jdbc.caseSensitivity`? --- If your project is set

[GitHub] spark pull request: [SPARK-12295] [SQL] external spilling for wind...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10605#issuecomment-169220562 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-12295] [SQL] external spilling for wind...

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

[GitHub] spark pull request: [SPARK-12317][SQL]Support configurate value fo...

2016-01-05 Thread kevinyu98
Github user kevinyu98 commented on the pull request: https://github.com/apache/spark/pull/10314#issuecomment-169220626 Hello @viirya : Good Point. I just test it, -1 and -1g will have different behavior. It will take -1, and throw IllegalArgumentException for -1g. Thanks. --- If

[GitHub] spark pull request: [SPARK-12582][Test] IndexShuffleBlockResolverS...

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

[GitHub] spark pull request: [SPARK-11531] [ML] : SparseVector error Msg

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

[GitHub] spark pull request: [SPARK-11531] [ML] : SparseVector error Msg

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

[GitHub] spark pull request: [SPARK-11531] [ML] : SparseVector error Msg

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9525#issuecomment-169232216 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-12616] [SQL] Making Logical Operator `U...

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

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10577#issuecomment-169241892 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-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48931170 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala --- @@ -212,22 +212,48 @@ object HiveTypeCoercion {

[GitHub] spark pull request: [SPARK-12433][CORE][TEST][RFC/WIP] Make parts ...

2016-01-05 Thread holdenk
GitHub user holdenk opened a pull request: https://github.com/apache/spark/pull/10614 [SPARK-12433][CORE][TEST][RFC/WIP] Make parts of spark core testing api public Apache Spark has a wide variety of internal tests, but many of the APIs used for testing are private. To allow

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48931187 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala --- @@ -212,22 +212,48 @@ object HiveTypeCoercion {

[GitHub] spark pull request: [SPARK-12368] [ML] [DOC] Better doc for the bi...

2016-01-05 Thread BenFradet
Github user BenFradet commented on a diff in the pull request: https://github.com/apache/spark/pull/10328#discussion_r48931179 --- Diff: docs/ml-guide.md --- @@ -951,4 +951,4 @@ model.transform(test) {% endhighlight %} - \ No newline at end of file +

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48931112 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -1212,7 +1212,12 @@

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48931140 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/patterns.scala --- @@ -170,17 +172,22 @@ object ExtractFiltersAndInnerJoins

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48931154 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -1212,7 +1212,12 @@

[GitHub] spark pull request: [SPARK-11878][SQL]: Eliminate distribute by in...

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

[GitHub] spark pull request: Branch 1.6

2016-01-05 Thread hyl713
Github user hyl713 commented on the pull request: https://github.com/apache/spark/pull/10616#issuecomment-169258359 i am so sorry,it's a mistake... 发送自 Outlook Mobile On Tue, Jan 5, 2016 at 11:33 PM -0800, "UCB AMPLab"

[GitHub] spark pull request: [SPARK-11531] [ML] : SparseVector error Msg

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9525#issuecomment-169202223 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-12638] [API DOC] Parameter explaination...

2016-01-05 Thread Wenpei
Github user Wenpei commented on the pull request: https://github.com/apache/spark/pull/10587#issuecomment-169202207 Thanks, @srowen I will try to add parameter explain to fold. But for treeAggregate, it has comments like below to refer to Aggregate function. I thought it's enough

[GitHub] spark pull request: [SPARK-11531] [ML] : SparseVector error Msg

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

[GitHub] spark pull request: [SPARK-12659] fix NPE in UnsafeExternalSorter ...

2016-01-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10606 --- 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-12659] fix NPE in UnsafeExternalSorter ...

2016-01-05 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10606#issuecomment-169204251 More comments would also be appreciated, since this is pretty confusing. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-12295] [SQL] external spilling for wind...

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

[GitHub] spark pull request: [SPARK-3873] [tests] Import ordering fixes.

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

[GitHub] spark pull request: [SPARK-3873] [tests] Import ordering fixes.

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10582#issuecomment-169206036 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-3873] [tests] Import ordering fixes.

2016-01-05 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/10582#issuecomment-169206265 Yay. Merging to avoid further conflicts. --- 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-12645] [SparkR] SparkR support hash fun...

2016-01-05 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/10597#issuecomment-169209916 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

[GitHub] spark pull request: [SPARK-11815] [ML] [PySpark] PySpark DecisionT...

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

[GitHub] spark pull request: [SPARK-2750][WEB UI] Add https support to the ...

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

[GitHub] spark pull request: [SPARK-11815] [ML] [PySpark] PySpark DecisionT...

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

[GitHub] spark pull request: [SPARK-11815] [ML] [PySpark] PySpark DecisionT...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9807#issuecomment-169212758 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-12317][SQL]Support configurate value fo...

2016-01-05 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10314#issuecomment-169214001 Besides, -1 and -1g will be the same or different behavior? --- 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-12645] [SparkR] SparkR support hash fun...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10597#issuecomment-169215075 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-12645] [SparkR] SparkR support hash fun...

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

[GitHub] spark pull request: [SPARK-12603] [MLlib] PySpark MLlib GaussianMi...

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

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

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

[GitHub] spark pull request: [SPARK-12393] [SparkR] Add read.text and write...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10348#issuecomment-169228020 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-12393] [SparkR] Add read.text and write...

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

[GitHub] spark pull request: [SPARK-7675][ML][PYSpark] sparkml params type ...

2016-01-05 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/9581#issuecomment-169239409 jenkins, retest 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

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48930694 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -40,6 +40,8 @@ abstract class Optimizer extends

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48931193 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/patterns.scala --- @@ -170,17 +172,22 @@ object ExtractFiltersAndInnerJoins

[GitHub] spark pull request: [SPARK-12393] [SparkR] Add read.text and write...

2016-01-05 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/10348#issuecomment-169247943 LGTM. Thanks @yanboliang for the patch an @sun-rui @felixcheung for reviewing. Merging this to master and `branch-1.6` --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48931219 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -1212,7 +1212,12 @@

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48931211 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala --- @@ -235,6 +261,11 @@ object HiveTypeCoercion {

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48931223 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -40,6 +40,8 @@ abstract class Optimizer extends

[GitHub] spark pull request: [SPARK-12436][SQL] Now InferSchema should retu...

2016-01-05 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10474#issuecomment-169252314 cc @yhuai --- 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

[GitHub] spark pull request: [SPARK-12125][SQL] pull out nondeterministic e...

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

[GitHub] spark pull request: [SPARK-12433][CORE][TEST][RFC/WIP] Make parts ...

2016-01-05 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/10614#issuecomment-169258070 Ok - I've got the design document and the APIs are based on current internal APIs if you have a chance to look at the API thoughts and give some feedback I'd

[GitHub] spark pull request: Branch 1.6

2016-01-05 Thread hyl713
Github user hyl713 closed the pull request at: https://github.com/apache/spark/pull/10616 --- 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: Branch 1.6

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10616#issuecomment-169258106 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-12433][CORE][TEST][RFC/WIP] Make parts ...

2016-01-05 Thread holdenk
Github user holdenk closed the pull request at: https://github.com/apache/spark/pull/10614 --- 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-6363][BUILD] Make Scala 2.11 the defaul...

2016-01-05 Thread JoshRosen
Github user JoshRosen closed the pull request at: https://github.com/apache/spark/pull/10608 --- 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-12616] [SQL] Making Logical Operator `U...

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

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

2016-01-05 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10583#discussion_r48923378 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystQl.scala --- @@ -0,0 +1,969 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-2750][WEB UI] Add https support to the ...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10238#issuecomment-169209702 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-11815] [ML] [PySpark] PySpark DecisionT...

2016-01-05 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/9807#discussion_r48924285 --- Diff: python/pyspark/ml/classification.py --- @@ -287,7 +287,7 @@ class DecisionTreeClassifier(JavaEstimator, HasFeaturesCol, HasLabelCol, HasPred

[GitHub] spark pull request: [SPARK-2750][WEB UI] Add https support to the ...

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

[GitHub] spark pull request: [SPARK-11815] [ML] [PySpark] PySpark DecisionT...

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

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10577#issuecomment-169249167 The other thing you gotta be careful with is the performance of fold/foldRight/foldLeft/tail/head on these collections. Please make sure we are not doing some linear

[GitHub] spark pull request: [SPARK-8410][SPARK-8475] remove previous ivy r...

2016-01-05 Thread brkyvz
Github user brkyvz commented on a diff in the pull request: https://github.com/apache/spark/pull/7089#discussion_r48931472 --- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala --- @@ -922,6 +922,15 @@ private[spark] object SparkSubmitUtils { //

[GitHub] spark pull request: [SPARK-12604] [CORE] Java count(AprroxDistinct...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10554#discussion_r48932093 --- Diff: core/src/main/scala/org/apache/spark/api/java/JavaPairRDD.scala --- @@ -288,17 +288,18 @@ class JavaPairRDD[K, V](val rdd: RDD[(K, V)]) *

[GitHub] spark pull request: [SPARK-12420][SQL] Have a built-in CSV data so...

2016-01-05 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/10615#discussion_r48932885 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala --- @@ -0,0 +1,231 @@ +/* + * Licensed to

[GitHub] spark pull request: [SPARK-12420][SQL] Have a built-in CSV data so...

2016-01-05 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/10615#discussion_r48932909 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala --- @@ -0,0 +1,231 @@ +/* + * Licensed to

[GitHub] spark pull request: [SPARK-12573][SPARK-12574][SQL] Move SQL Parse...

2016-01-05 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10583#discussion_r48934251 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystQl.scala --- @@ -0,0 +1,969 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-11945] [ML] [PySpark] Add computeCost t...

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

[GitHub] spark pull request: [SPARK-12645] [SparkR] SparkR support hash fun...

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

[GitHub] spark pull request: [SPARK-11945] [ML] [PySpark] Add computeCost t...

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

[GitHub] spark pull request: [SPARK-11945] [ML] [PySpark] Add computeCost t...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9931#issuecomment-169214879 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-12619 Combine small files in a hadoop di...

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

[GitHub] spark pull request: [SPARK-12616] [SQL] Adding a New Logical Opera...

2016-01-05 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10577#issuecomment-169216470 Major changes in this commit: - Combine the Union during the construction (i.e., in the DataFrame and Dataset APIs) - Use tail recursive with extractor

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169216595 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-11531] [ML] : SparseVector error Msg

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

[GitHub] spark pull request: [SPARK-11531] [ML] : SparseVector error Msg

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

[GitHub] spark pull request: [SPARK-12665][CORE][GRAPHX] Remove deprecated ...

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

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169233964 retest 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 this

[GitHub] spark pull request: [SPARK-7675][ML][PYSpark] sparkml params type ...

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

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48930777 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicOperators.scala --- @@ -123,6 +115,35 @@ case class Except(left:

[GitHub] spark pull request: [SPARK-12616] [SQL] Making Logical Operator `U...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10577#discussion_r48930717 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -153,32 +154,45 @@ object SetOperationPushDown extends

[GitHub] spark pull request: [SPARK-12604] [CORE] Java count(AprroxDistinct...

2016-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10554#discussion_r48931848 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/api/java/JavaPairDStream.scala --- @@ -848,6 +848,6 @@ object JavaPairDStream { def

[GitHub] spark pull request: [SPARK-12420][SQL] Have a built-in CSV data so...

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

[GitHub] spark pull request: [SPARK-12420][SQL] Have a built-in CSV data so...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10615#issuecomment-169253681 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-12420][SQL] Have a built-in CSV data so...

2016-01-05 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/10615#discussion_r48933011 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVRelation.scala --- @@ -0,0 +1,305 @@ +/* + * Licensed to

[GitHub] spark pull request: [SPARK-11878][SQL]: Eliminate distribute by in...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9858#issuecomment-169258612 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-11878][SQL]: Eliminate distribute by in...

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

<    1   2   3   4   5   6   7   8   9   >