[GitHub] spark pull request: [SPARK-15379][SQL] check special invalid date

2016-05-20 Thread wangyang1992
Github user wangyang1992 commented on the pull request: https://github.com/apache/spark/pull/13169#issuecomment-220549615 seems like a irrelevant fail. retest it 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

[GitHub] spark pull request: [SPARK-15308][SQL] RowEncoder should preserve ...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13090#issuecomment-220548418 Build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] spark pull request: [SPARK-15308][SQL] RowEncoder should preserve ...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13090#issuecomment-220548420 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-13135][SQL] Don't print expressions rec...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13192#issuecomment-220548264 **[Test build #58971 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58971/consoleFull)** for PR 13192 at commit [`5ffb249`](https://gi

[GitHub] spark pull request: [SPARK-15312] [SQL] Detect Duplicate Key in Pa...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13095#issuecomment-220548134 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15308][SQL] RowEncoder should preserve ...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13090#issuecomment-220548207 **[Test build #58956 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58956/consoleFull)** for PR 13090 at commit [`0346efe`](https://g

[GitHub] spark pull request: [SPARK-15312] [SQL] Detect Duplicate Key in Pa...

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

[GitHub] spark pull request: [SPARK-15312] [SQL] Detect Duplicate Key in Pa...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13095#issuecomment-220547970 **[Test build #58964 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58964/consoleFull)** for PR 13095 at commit [`d7c2420`](https://g

[GitHub] spark pull request: [SPARK-13135][SQL] Don't print expressions rec...

2016-05-20 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/13192#issuecomment-220547752 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 t

[GitHub] spark pull request: [SPARK-15282][SQL] Make ScalaUDF nondeterminis...

2016-05-20 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/13087#issuecomment-220547374 According to @cloud-fan 's advice, the goal of this PR is now making `ScalaUDF` as a non-deterministic expression. Although this is a correct fix, one noticeable

[GitHub] spark pull request: [SPARK-15379][SQL] check special invalid date

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

[GitHub] spark pull request: [SPARK-15379][SQL] check special invalid date

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13169#issuecomment-220546246 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15282][SQL] Make ScalaUDF nondeterminis...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13087#issuecomment-220546133 **[Test build #58969 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58969/consoleFull)** for PR 13087 at commit [`94ecdd4`](https://gi

[GitHub] spark pull request: [SPARK-15379][SQL] check special invalid date

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13169#issuecomment-220546092 **[Test build #58959 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58959/consoleFull)** for PR 13169 at commit [`193a2f1`](https://g

[GitHub] spark pull request: [SPARK-15114][SQL] Column name generated by ty...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13045#issuecomment-220546138 **[Test build #58970 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58970/consoleFull)** for PR 13045 at commit [`1a83497`](https://gi

[GitHub] spark pull request: [SPARK-15308][SQL] RowEncoder should preserve ...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13090#issuecomment-220545406 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15308][SQL] RowEncoder should preserve ...

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

[GitHub] spark pull request: [SPARK-15435][SQL] Append Command to all comma...

2016-05-20 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/13215#discussion_r64005643 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala --- @@ -106,6 +106,6 @@ case class AnalyzeTable(tableName:

[GitHub] spark pull request: [SPARK-15308][SQL] RowEncoder should preserve ...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13090#issuecomment-220545256 **[Test build #58961 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58961/consoleFull)** for PR 13090 at commit [`cc8acea`](https://g

[GitHub] spark pull request: [SPARK-15438] [SQL] improve explain of whole s...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13204#issuecomment-220545057 **[Test build #3003 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3003/consoleFull)** for PR 13204 at commit [`bc2aed4`](https://

[GitHub] spark pull request: [SPARK-13135][SQL] Don't print expressions rec...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13192#issuecomment-220544914 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-13135][SQL] Don't print expressions rec...

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

[GitHub] spark pull request: [SPARK-13135][SQL] Don't print expressions rec...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13192#issuecomment-220544714 **[Test build #58960 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58960/consoleFull)** for PR 13192 at commit [`5ffb249`](https://g

[GitHub] spark pull request: [SPARK-15114][SQL] Column name generated by ty...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13045#issuecomment-220544367 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15114][SQL] Column name generated by ty...

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

[GitHub] spark pull request: [SPARK-15114][SQL] Column name generated by ty...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13045#issuecomment-220544204 **[Test build #58957 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58957/consoleFull)** for PR 13045 at commit [`f410999`](https://g

[GitHub] spark pull request: [SPARK-15206][SQL] add testcases for distinct ...

2016-05-20 Thread xwu0226
Github user xwu0226 commented on the pull request: https://github.com/apache/spark/pull/12984#issuecomment-220544175 @cloud-fan I rebased my local branch and pushed again. --- 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-15435][SQL] Append Command to all comma...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13215#issuecomment-220543522 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15435][SQL] Append Command to all comma...

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

[GitHub] spark pull request: [SPARK-15440] [Core] [Deploy] Add CSRF Filter ...

2016-05-20 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/13218#issuecomment-22054 Hi @yanboliang , from my understanding, we may also have rest servlet in Spark live UI and history UI, do we also need to address the issue in that place? --- If yo

[GitHub] spark pull request: [SPARK-15435][SQL] Append Command to all comma...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13215#issuecomment-220543215 **[Test build #58948 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58948/consoleFull)** for PR 13215 at commit [`2827ca5`](https://g

[GitHub] spark pull request: [SPARK-15364][ML][PySpark] Implement PySpark p...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13219#issuecomment-220542051 **[Test build #58968 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58968/consoleFull)** for PR 13219 at commit [`d1d14f6`](https://gi

[GitHub] spark pull request: [SPARK-15433][PySpark] PySpark core test shoul...

2016-05-20 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/13214#issuecomment-220541883 cc @davies --- 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 en

[GitHub] spark pull request: [SPARK-15364][ML][PySpark] Implement PySpark p...

2016-05-20 Thread viirya
GitHub user viirya opened a pull request: https://github.com/apache/spark/pull/13219 [SPARK-15364][ML][PySpark] Implement PySpark picklers for ml.Vector and ml.Matrix under spark.ml.python ## What changes were proposed in this pull request? Now we have PySpark picklers for

[GitHub] spark pull request: [SPARK-15424][SPARK-15437][SQL] Revert SPARK-1...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13207#issuecomment-220540416 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15424][SPARK-15437][SQL] Revert SPARK-1...

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

[GitHub] spark pull request: [SPARK-15424][SPARK-15437][SQL] Revert SPARK-1...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13207#issuecomment-220540313 **[Test build #58942 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58942/consoleFull)** for PR 13207 at commit [`c77616e`](https://g

[GitHub] spark pull request: [SPARK-15434][SQL] improve EmbedSerializerInFi...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13216#issuecomment-220539981 Build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] spark pull request: [SPARK-15434][SQL] improve EmbedSerializerInFi...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13216#issuecomment-220539982 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15206][SQL] add testcases for distinct ...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12984#issuecomment-220539729 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15206][SQL] add testcases for distinct ...

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

[GitHub] spark pull request: [SPARK-15206][SQL] add testcases for distinct ...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12984#issuecomment-220539592 **[Test build #58952 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58952/consoleFull)** for PR 12984 at commit [`5445805`](https://g

[GitHub] spark pull request: [SPARK-15165][SPARK-15205][SQL] Introduce plac...

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

[GitHub] spark pull request: [SPARK-15165][SPARK-15205][SQL] Introduce plac...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12979#issuecomment-220538994 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15165][SPARK-15205][SQL] Introduce plac...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12979#issuecomment-220538733 **[Test build #58945 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58945/consoleFull)** for PR 12979 at commit [`6ebfa10`](https://g

[GitHub] spark pull request: [SPARK-15433][PySpark] PySpark core test shoul...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13214#issuecomment-220538790 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15433][PySpark] PySpark core test shoul...

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

[GitHub] spark pull request: [SPARK-15282][SQL] PushDownPredicate should no...

2016-05-20 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/13087#discussion_r64002341 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -1025,7 +1025,8 @@ object PushDownPredicate exten

[GitHub] spark pull request: [SPARK-15433][PySpark] PySpark core test shoul...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13214#issuecomment-220538560 **[Test build #58943 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58943/consoleFull)** for PR 13214 at commit [`760a4cd`](https://g

[GitHub] spark pull request: [SPARK-15400][SQL] CreateNamedStruct and Creat...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13193#issuecomment-220538439 **[Test build #58966 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58966/consoleFull)** for PR 13193 at commit [`c9acff7`](https://gi

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13176#issuecomment-220538455 **[Test build #58967 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58967/consoleFull)** for PR 13176 at commit [`46408bb`](https://gi

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-20 Thread MLnick
Github user MLnick commented on the pull request: https://github.com/apache/spark/pull/13176#issuecomment-220537993 ok to test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature ena

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-20 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/13176#discussion_r64001816 --- Diff: docs/ml-features.md --- @@ -114,7 +116,10 @@ for more details on the API. During the fitting process, `CountVectorizer` will select the top `v

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-20 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/13176#discussion_r64001752 --- Diff: docs/ml-features.md --- @@ -114,7 +116,10 @@ for more details on the API. During the fitting process, `CountVectorizer` will select the top `v

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-20 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/13176#discussion_r64001546 --- Diff: docs/ml-features.md --- @@ -1064,7 +1069,8 @@ categorical features. The bin ranges are chosen by taking a sample of the data and dividing it in

[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220536852 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your projec

[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220536855 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-20 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/13176#discussion_r64001345 --- Diff: docs/ml-features.md --- @@ -1064,7 +1069,8 @@ categorical features. The bin ranges are chosen by taking a sample of the data and dividing it in

[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220536743 **[Test build #58962 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58962/consoleFull)** for PR 13217 at commit [`acb9363`](https://g

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-20 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/13176#discussion_r64001064 --- Diff: examples/src/main/java/org/apache/spark/examples/ml/JavaQuantileDiscretizerExample.java --- @@ -58,7 +58,8 @@ public static void main(String[] args

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-20 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/13176#discussion_r64001008 --- Diff: examples/src/main/java/org/apache/spark/examples/ml/JavaCountVectorizerExample.java --- @@ -54,6 +54,7 @@ public static void main(String[] args) {

[GitHub] spark pull request: [SPARK-15431][SQL] Support LIST FILE(s)|JAR(s)...

2016-05-20 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13212#discussion_r64000637 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala --- @@ -46,3 +51,56 @@ case class AddFile(path: String) extends Run

[GitHub] spark pull request: [SPARK-15282][SQL] PushDownPredicate should no...

2016-05-20 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13087#discussion_r64000518 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -1025,7 +1025,8 @@ object PushDownPredicate extends R

[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220535184 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your projec

[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220535185 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220535153 **[Test build #58963 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58963/consoleFull)** for PR 13217 at commit [`b62b631`](https://g

[GitHub] spark pull request: [SPARK-15440] [Core] [Deploy] Add CSRF Filter ...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13218#issuecomment-220534995 **[Test build #58965 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58965/consoleFull)** for PR 13218 at commit [`93b1c6f`](https://gi

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-20 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/13176#discussion_r64000156 --- Diff: docs/ml-features.md --- @@ -1064,7 +1069,8 @@ categorical features. The bin ranges are chosen by taking a sample of the data and dividing it in

[GitHub] spark pull request: [SPARK-15440] [Core] [Deploy] Add CSRF Filter ...

2016-05-20 Thread yanboliang
GitHub user yanboliang opened a pull request: https://github.com/apache/spark/pull/13218 [SPARK-15440] [Core] [Deploy] Add CSRF Filter for REST APIs to Spark ## What changes were proposed in this pull request? CSRF prevention for REST APIs can be provided through a common servlet

[GitHub] spark pull request: [SPARK-15431][SQL] Support LIST FILE(s)|JAR(s)...

2016-05-20 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13212#discussion_r63999888 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala --- @@ -46,3 +51,56 @@ case class AddFile(path: String) extends R

[GitHub] spark pull request: [SPARK-14321][SQL] Reduce date format cost and...

2016-05-20 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/12105#issuecomment-220533781 hi @rajeshbalamohan , sorry for the delay, do you wanna try this again? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-14321][SQL] Reduce date format cost and...

2016-05-20 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/12105#discussion_r63999680 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala --- @@ -391,21 +393,24 @@ abstract class UnixTim

[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220533401 **[Test build #58963 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58963/consoleFull)** for PR 13217 at commit [`b62b631`](https://gi

[GitHub] spark pull request: [SPARK-15431][SQL] Support LIST FILE(s)|JAR(s)...

2016-05-20 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13212#discussion_r63999507 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala --- @@ -46,3 +51,56 @@ case class AddFile(path: String) extends Run

[GitHub] spark pull request: [SPARK-15312] [SQL] Detect Duplicate Key in Pa...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13095#issuecomment-220533402 **[Test build #58964 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58964/consoleFull)** for PR 13095 at commit [`d7c2420`](https://gi

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

2016-05-20 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13165#issuecomment-220533282 @sun-rui Thank you so much. I will try to add a test. --- 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-15308][SQL] RowEncoder should preserve ...

2016-05-20 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/13090#issuecomment-220532877 LGTM, pending jenkins --- 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 th

[GitHub] spark pull request: [SPARK-15431][SQL] Support LIST FILE(s)|JAR(s)...

2016-05-20 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13212#discussion_r63999183 --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala --- @@ -295,9 +294,7 @@ private[hive] class

[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220532615 **[Test build #58962 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58962/consoleFull)** for PR 13217 at commit [`acb9363`](https://gi

[GitHub] spark pull request: [SPARK-13135][SQL] Don't print expressions rec...

2016-05-20 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/13192#issuecomment-220532622 As @rxin told, what was really needed is removing `overlapping` comments. So, I rethink about that and revert the change on `Expression.gen` which removes the

[GitHub] spark pull request: [MINOR][SPARKR][DOC] Add a description for run...

2016-05-20 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13217#issuecomment-220532414 cc @sun-rui --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feat

[GitHub] spark pull request: [SPARK-15312] [SQL] Detect Duplicate Key in Pa...

2016-05-20 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13095#issuecomment-220532545 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-15431][SQL] Support LIST FILE(s)|JAR(s)...

2016-05-20 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13212#discussion_r63999050 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -713,13 +713,40 @@ class SparkSqlAstBuilder(conf: SQLConf) exten

[GitHub] spark pull request: [SPARK-15438] [SQL] improve explain of whole s...

2016-05-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13204#issuecomment-220532288 **[Test build #3003 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3003/consoleFull)** for PR 13204 at commit [`bc2aed4`](https://g

[GitHub] spark pull request: [SPARK-15431][SQL] Support LIST FILE(s)|JAR(s)...

2016-05-20 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13212#discussion_r63999023 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/resources.scala --- @@ -46,3 +51,56 @@ case class AddFile(path: String) extends R

[GitHub] spark pull request: [MI

2016-05-20 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/13217 [MI ## What changes were proposed in this pull request? (Please fill in changes proposed in this fix) ## How was this patch tested? (Please explain how this patch

[GitHub] spark pull request: [SPARK-15312] [SQL] Detect Duplicate Key in Pa...

2016-05-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13095#issuecomment-220532100 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15312] [SQL] Detect Duplicate Key in Pa...

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

<    3   4   5   6   7   8