[GitHub] spark pull request: [SPARK-14507] [SPARK-15646] [SQL] When spark.s...

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

[GitHub] spark pull request: [Minor][ML] Enable java save/load tests for ML

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

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13380#issuecomment-222343969 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-9876][SQL][FOLLOWUP] Enable string and ...

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

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

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

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

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

[GitHub] spark pull request: [SPARK-15286] [SQL] Make the output readable f...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13070#issuecomment-222343174 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-15286] [SQL] Make the output readable f...

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

[GitHub] spark pull request: [SPARK-15286] [SQL] Make the output readable f...

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

[GitHub] spark pull request: [SPARK-14507] [SPARK-15646] [SQL] When spark.s...

2016-05-28 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/13386#issuecomment-222343075 test this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-9876][SQL][FOLLOWUP] Enable string and ...

2016-05-28 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/13389 [SPARK-9876][SQL][FOLLOWUP] Enable string and binary tests for Parquet predicate pushdown ## What changes were proposed in this pull request? It seems Parquet has been upgraded to

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

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

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13147#issuecomment-222343015 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-6320][SQL] Move planLater method i...

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

[GitHub] spark pull request: [SPARK-14507] [SPARK-15646] [SQL] When spark.s...

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

[GitHub] spark pull request: [SPARK-14507] [SPARK-15646] [SQL] When spark.s...

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

[GitHub] spark pull request: [SPARK-14507] [SPARK-15646] [SQL] When spark.s...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13386#issuecomment-222342719 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-15286] [SQL] Make the output readable f...

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

[GitHub] spark pull request: [SPARK-15286] [SQL] Make the output readable f...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13070#issuecomment-222342638 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-15286] [SQL] Make the output readable f...

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

[GitHub] spark pull request: [SPARK-14507] [SPARK-15646] [SQL] When spark.s...

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

[GitHub] spark pull request: [SPARK-15608][ml][doc] add_isotonic_regression...

2016-05-28 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/13381#issuecomment-222342274 The Java & Python version don't appear to currently exist in master - did you mean to include them in this PR? --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [Docs] fix example code issues in documentatio...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13388#issuecomment-222342223 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: fix example code issues in documentation

2016-05-28 Thread mawise
GitHub user mawise opened a pull request: https://github.com/apache/spark/pull/13388 fix example code issues in documentation ## What changes were proposed in this pull request? Fixed broken java code examples in streaming documentation Attn: @tdas You can merge

[GitHub] spark pull request: [Docs] Fix streaming java doc examples

2016-05-28 Thread mawise
Github user mawise commented on the pull request: https://github.com/apache/spark/pull/13387#issuecomment-222342012 Let me re-do this without the merge commit --- 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: [Docs] Fix streaming java doc examples

2016-05-28 Thread mawise
Github user mawise closed the pull request at: https://github.com/apache/spark/pull/13387 --- 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: [Docs] Fix streaming java doc examples

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13387#issuecomment-222341862 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: [Docs] Fix streaming java doc examples

2016-05-28 Thread mawise
GitHub user mawise opened a pull request: https://github.com/apache/spark/pull/13387 [Docs] Fix streaming java doc examples ## What changes were proposed in this pull request? Fixed code examples in docs Attn @tdas (your name is all over the git-blame for these docs)

[GitHub] spark pull request: [SPARK-15100][ML][Doc] doc update for stopword...

2016-05-28 Thread hhbyyh
Github user hhbyyh commented on the pull request: https://github.com/apache/spark/pull/13375#issuecomment-222341706 Thanks for the review @srowen --- 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-15100][ML][Doc] doc update for stopword...

2016-05-28 Thread hhbyyh
Github user hhbyyh commented on a diff in the pull request: https://github.com/apache/spark/pull/13375#discussion_r64997369 --- Diff: docs/ml-features.md --- @@ -251,11 +251,12 @@ frequently and don't carry as much meaning. `StopWordsRemover` takes as input a sequence of

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

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

[GitHub] spark pull request: [SPARK-15286] [SQL] Make the output readable f...

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

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

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

[GitHub] spark pull request: [SPARK-15286] [SQL] Make the output readable f...

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

[GitHub] spark pull request: [SPARK-15608][ml][doc] add_isotonic_regression...

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

[GitHub] spark pull request: [SPARK-15608][ml][doc] add_isotonic_regression...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13381#issuecomment-222340649 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-15608][ml][doc] add_isotonic_regression...

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

[GitHub] spark pull request: [SPARK-15608][ml][doc] add_isotonic_regression...

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

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13380#discussion_r64997106 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala --- @@ -48,7 +48,7 @@ class BroadcastJoinSuite extends

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/13380#discussion_r64997103 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala --- @@ -48,7 +48,7 @@ class BroadcastJoinSuite

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/13380#discussion_r64997097 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala --- @@ -48,7 +48,7 @@ class BroadcastJoinSuite

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13380#discussion_r64996988 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala --- @@ -48,7 +48,7 @@ class BroadcastJoinSuite extends

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/13380#discussion_r64996971 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala --- @@ -48,7 +48,7 @@ class BroadcastJoinSuite

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/13380#discussion_r64996960 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala --- @@ -48,7 +48,7 @@ class BroadcastJoinSuite

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

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

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13147#issuecomment-222339353 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-6320][SQL] Move planLater method i...

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

[GitHub] spark pull request: [SPARK-14857] [SQL] Table/Database Name Valida...

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

[GitHub] spark pull request: [SPARK-14857] [SQL] Table/Database Name Valida...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12618#issuecomment-222338132 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-14857] [SQL] Table/Database Name Valida...

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

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13380#issuecomment-222337714 @dongjoon-hyun Thank you! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/13380#issuecomment-222337348 Hi, @gatorsmile . Personally, I love this PR. :) I just hesitated to change the function signatures of MLLIB in #13352 . --- If your project is set up

[GitHub] spark pull request: [DO_NOT_MERGE][ML] Enable java save/load tests...

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

[GitHub] spark pull request: [DO_NOT_MERGE][ML] Enable java save/load tests...

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

[GitHub] spark pull request: [DO_NOT_MERGE][ML] Enable java save/load tests...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13376#issuecomment-222337301 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-14507] [SPARK-15646] [SQL] When spark.s...

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

[GitHub] spark pull request: [SPARK-14507] [SPARK-15646] [SQL] When spark.s...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13386#issuecomment-222337215 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-14507] [SPARK-15646] [SQL] When spark.s...

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

[GitHub] spark pull request: [SPARK-15645][Streaming] Fix some typos of Str...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13385#issuecomment-222337168 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-15645][Streaming] Fix some typos of Str...

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

[GitHub] spark pull request: [SPARK-15645][Streaming] Fix some typos of Str...

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

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

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

[GitHub] spark pull request: [DO_NOT_MERGE][ML] Enable java save/load tests...

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

[GitHub] spark pull request: [SPARK-14857] [SQL] Table/Database Name Valida...

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

[GitHub] spark pull request: [SPARK-15370] [SQL] Update RewriteCorrelatedSc...

2016-05-28 Thread frreiss
Github user frreiss commented on a diff in the pull request: https://github.com/apache/spark/pull/13155#discussion_r64996012 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -1695,16 +1695,176 @@ object

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13380#issuecomment-222335661 @dongjoon-hyun @andrewor14 This PR removes `SQLContext` from `MLlib`. Let me know if we should keep it. Thanks! --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-15370] [SQL] Update RewriteCorrelatedSc...

2016-05-28 Thread frreiss
Github user frreiss commented on a diff in the pull request: https://github.com/apache/spark/pull/13155#discussion_r64995985 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -1695,16 +1695,176 @@ object

[GitHub] spark pull request: [SPARK-15603][MLLIB] Replace SQLContext with S...

2016-05-28 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13352#issuecomment-222335566 Sorry, I did not notice this PR. I submitted another PR (https://github.com/apache/spark/pull/13380) that removes `SQLContext` from `MLlib`. Any reason why you

[GitHub] spark pull request: [SPARK-14279] [Build] Pick the spark version f...

2016-05-28 Thread dhruve
Github user dhruve commented on the pull request: https://github.com/apache/spark/pull/13061#issuecomment-222335144 @jodersky you can get the context if you follow the conversation as to why its placed in target/extra-resources. --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-14507] [SPARK-15646] [SQL] When spark.s...

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

[GitHub] spark pull request: [SPARK-14507] [SPARK-15646] [SQL] When spark.s...

2016-05-28 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/13386#issuecomment-222334796 @ericl @andrewor14 @liancheng Can you review this PR? --- 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-14507] [SPARK-15646] [SQL] When spark.s...

2016-05-28 Thread yhuai
GitHub user yhuai opened a pull request: https://github.com/apache/spark/pull/13386 [SPARK-14507] [SPARK-15646] [SQL] When spark.sql.hive.convertCTAS is true, we should not convert the table stored as TEXTFILE/SEQUENCEFILE and we need respect the user-defined location ## What

[GitHub] spark pull request: [SPARK-15645][Streaming] Fix some typos of Str...

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

[GitHub] spark pull request: [SPARK-15645][Streaming] Fix some typos of Str...

2016-05-28 Thread keypointt
GitHub user keypointt opened a pull request: https://github.com/apache/spark/pull/13385 [SPARK-15645][Streaming] Fix some typos of Streaming module ## What changes were proposed in this pull request? No code change, just some typo fixing. ## How was this patch

[GitHub] spark pull request: [CORE][MINOR][DOC] Removing incorrect scaladoc

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13384#issuecomment-222333936 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: [CORE][MINOR][DOC] Removing incorrect scaladoc

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

[GitHub] spark pull request: [CORE][MINOR][DOC] Removing incorrect scaladoc

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

[GitHub] spark pull request: [CORE][DOC][MINOR] typos + links

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

[GitHub] spark pull request: [CORE][DOC][MINOR] typos + links

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13383#issuecomment-222333780 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: [CORE][DOC][MINOR] typos + links

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

[GitHub] spark pull request: [SPARK-15331] [SQL] Disallow All the Unsupport...

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

[GitHub] spark pull request: [SPARK-15331] [SQL] Disallow All the Unsupport...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13122#issuecomment-222333666 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-15331] [SQL] Disallow All the Unsupport...

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

[GitHub] spark pull request: [SPARK-15100][ML][Doc] doc update for stopword...

2016-05-28 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13375#discussion_r64994783 --- Diff: docs/ml-features.md --- @@ -251,11 +251,12 @@ frequently and don't carry as much meaning. `StopWordsRemover` takes as input a sequence of

[GitHub] spark pull request: [SPARK-15331] [SQL] Disallow All the Unsupport...

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

[GitHub] spark pull request: [SPARK-15331] [SQL] Disallow All the Unsupport...

2016-05-28 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13122#issuecomment-222330922 @hvanhovell @yhuai This is also ready to review. Thanks! --- 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-15460] [SQL] Issue Exceptions from Thri...

2016-05-28 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13111#issuecomment-222330637 cc @yhuai The code is ready for review. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [CORE][MINOR][DOC] Removing incorrect scaladoc

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

[GitHub] spark pull request: [CORE][MINOR][DOC] Removing incorrect scaladoc

2016-05-28 Thread jaceklaskowski
GitHub user jaceklaskowski opened a pull request: https://github.com/apache/spark/pull/13384 [CORE][MINOR][DOC] Removing incorrect scaladoc ## What changes were proposed in this pull request? I don't think the method will ever throw an exception so removing a false

[GitHub] spark pull request: [CORE][DOC][MINOR] typos + links

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

[GitHub] spark pull request: [SPARK-15636][SQL] Make aggregate expressions ...

2016-05-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13367 --- 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: [CORE][DOC][MINOR] typos + links

2016-05-28 Thread jaceklaskowski
GitHub user jaceklaskowski opened a pull request: https://github.com/apache/spark/pull/13383 [CORE][DOC][MINOR] typos + links ## What changes were proposed in this pull request? A very tiny change to javadoc (which I don't mind if gets merged with a bigger change). I've

[GitHub] spark pull request: [SPARK-15636][SQL] Make aggregate expressions ...

2016-05-28 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/13367#issuecomment-222329931 lgtm. Merging to master and branch 2.0. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-15460] [SQL] Issue Exceptions from Thri...

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

[GitHub] spark pull request: [SPARK-15460] [SQL] Issue Exceptions from Thri...

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

[GitHub] spark pull request: [SPARK-15460] [SQL] Issue Exceptions from Thri...

2016-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13111#issuecomment-222329712 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-15460] [SQL] Issue Exceptions from Thri...

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

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-28 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/13380#issuecomment-222328340 @rxin `SQLContext` is not being used in the `MLlib` test suites. Thus, this PR is just to use the latest `SparkSession` to replace the existing `SQLContext` in

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

2016-05-28 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r64993001 --- Diff: R/pkg/inst/worker/worker.R --- @@ -84,67 +84,78 @@ broadcastElap <- elapsedSecs() # as number of partitions to create. numPartitions <-

[GitHub] spark pull request: [SPARK-13638][SQL] Add quoteAll option to CSV ...

2016-05-28 Thread jurriaan
Github user jurriaan commented on a diff in the pull request: https://github.com/apache/spark/pull/13374#discussion_r64992937 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVParser.scala --- @@ -76,6 +76,7 @@ private[sql] class

  1   2   >