[GitHub] spark issue #16409: [SPARK-19004][SQL] Fix `testH2Dialect` by removing `getC...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16409 **[Test build #70612 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70612/testReport)** for PR 16409 at commit

[GitHub] spark issue #16409: [SPARK-19004][SQL] Fix `testH2Dialect` by removing `getC...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16409 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70612/ Test PASSed. ---

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16405 **[Test build #70620 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70620/testReport)** for PR 16405 at commit

[GitHub] spark issue #16404: [SPARK-18969][SQL] Support grouping by nondeterministic ...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16404 **[Test build #70617 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70617/testReport)** for PR 16404 at commit

[GitHub] spark issue #16404: [SPARK-18969][SQL] Support grouping by nondeterministic ...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16404 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 project does not have this feature

[GitHub] spark issue #16404: [SPARK-18969][SQL] Support grouping by nondeterministic ...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16404 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70617/ Test PASSed. ---

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-26 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93898349 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -510,32 +510,94 @@ class Analyzer( * Replaces

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-26 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93898300 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -510,32 +510,94 @@ class Analyzer( * Replaces

[GitHub] spark issue #15219: [SPARK-14098][SQL] Generate Java code to build CachedCol...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15219 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70626/ Test FAILed. ---

[GitHub] spark issue #15219: [SPARK-14098][SQL] Generate Java code to build CachedCol...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15219 **[Test build #70626 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70626/testReport)** for PR 15219 at commit

[GitHub] spark issue #15219: [SPARK-14098][SQL] Generate Java code to build CachedCol...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15219 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 project does not have this feature

[GitHub] spark pull request #16410: [SPARK-19005][SQL] Keep column ordering when a sc...

2016-12-26 Thread maropu
GitHub user maropu opened a pull request: https://github.com/apache/spark/pull/16410 [SPARK-19005][SQL] Keep column ordering when a schema is explicitly specified ## What changes were proposed in this pull request? This pr is to keep column ordering when a schema is explicitly

[GitHub] spark issue #16410: [SPARK-19005][SQL] Keep column ordering when a schema is...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16410 **[Test build #70613 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70613/testReport)** for PR 16410 at commit

[GitHub] spark issue #13909: [SPARK-16213][SQL] Reduce runtime overhead of a program ...

2016-12-26 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/13909 > Should use dataType.containsNull for allowNull? no we shouldn't, `containsNull` is kind of an optimizing hint that guarantees the value will not be null, and it's guaranteed by the

[GitHub] spark issue #13909: [SPARK-16213][SQL] Reduce runtime overhead of a program ...

2016-12-26 Thread ueshin
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/13909 @cloud-fan Hmm, I agree that it's guaranteed by the source, but I thought that's why we should use `containsNull` for `allowNull` to guarantee the value will not be null when `containsNull == false`

[GitHub] spark issue #16233: [SPARK-18801][SQL] Add `View` operator to help resolve a...

2016-12-26 Thread jiangxb1987
Github user jiangxb1987 commented on the issue: https://github.com/apache/spark/pull/16233 @yhuai @hvanhovell @cloud-fan @gatorsmile Could you please review this PR when you have time? Thanks! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16405 **[Test build #70624 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70624/testReport)** for PR 16405 at commit

[GitHub] spark pull request #16405: [SPARK-19002][BUILD] Check pep8 against merge_spa...

2016-12-26 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/16405#discussion_r93886149 --- Diff: dev/lint-python --- @@ -23,6 +23,7 @@ PATHS_TO_CHECK="./python/pyspark/ ./examples/src/main/python/ ./dev/sparktestsup # TODO: fix pep8

[GitHub] spark pull request #16405: [SPARK-19002][BUILD] Check pep8 against merge_spa...

2016-12-26 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/16405#discussion_r93886392 --- Diff: dev/lint-python --- @@ -23,6 +23,7 @@ PATHS_TO_CHECK="./python/pyspark/ ./examples/src/main/python/ ./dev/sparktestsup # TODO: fix pep8

[GitHub] spark pull request #16412: [SPARK-19006][DOCS]mention spark.kryoserializer.b...

2016-12-26 Thread cnZach
GitHub user cnZach opened a pull request: https://github.com/apache/spark/pull/16412 [SPARK-19006][DOCS]mention spark.kryoserializer.buffer.max must be less than 2048m in doc ## What changes were proposed in this pull request? On configuration doc

[GitHub] spark issue #16410: [SPARK-19005][SQL] Keep column ordering when a schema is...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16410 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70613/ Test FAILed. ---

[GitHub] spark issue #16410: [SPARK-19005][SQL] Keep column ordering when a schema is...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16410 **[Test build #70613 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70613/testReport)** for PR 16410 at commit

[GitHub] spark issue #16410: [SPARK-19005][SQL] Keep column ordering when a schema is...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16410 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 project does not have this feature

[GitHub] spark pull request #16413: Branch 1.3

2016-12-26 Thread Kevy123
GitHub user Kevy123 opened a pull request: https://github.com/apache/spark/pull/16413 Branch 1.3 ## 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 issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16405 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 feature enabled and wishes

[GitHub] spark issue #15721: [SPARK-17772][ML][TEST] Add test functions for ML sample...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15721 **[Test build #70621 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70621/testReport)** for PR 15721 at commit

[GitHub] spark issue #16413: Branch 1.3

2016-12-26 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16413 Hi @Kevy123, it seems this pull request is mistakenly open. Could you please close this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16405 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70622/ Test FAILed. ---

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16405 **[Test build #70622 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70622/testReport)** for PR 16405 at commit

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16405 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 project does not have this feature

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16405 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 feature enabled and wishes

[GitHub] spark issue #16373: [SPARK-18961][SQL] Support `SHOW TABLE EXTENDED ... PART...

2016-12-26 Thread jiangxb1987
Github user jiangxb1987 commented on the issue: https://github.com/apache/spark/pull/16373 @yhuai @hvanhovell @gatorsmile Would you please review this PR when you have time? Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #15721: [SPARK-17772][ML][TEST] Add test functions for ML sample...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15721 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70621/ Test PASSed. ---

[GitHub] spark issue #15721: [SPARK-17772][ML][TEST] Add test functions for ML sample...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15721 **[Test build #70621 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70621/testReport)** for PR 15721 at commit

[GitHub] spark issue #15721: [SPARK-17772][ML][TEST] Add test functions for ML sample...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15721 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 project does not have this feature

[GitHub] spark pull request #16392: [SPARK-18992] [SQL] Move spark.sql.hive.thriftSer...

2016-12-26 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16392#discussion_r93894368 --- Diff: sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala --- @@ -586,7 +586,8 @@ class

[GitHub] spark issue #16391: [SPARK-18990][SQL] make DatasetBenchmark fairer for Data...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16391 **[Test build #70619 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70619/testReport)** for PR 16391 at commit

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-26 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93898018 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -510,32 +510,94 @@ class Analyzer( * Replaces

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-26 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93898130 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -510,32 +510,94 @@ class Analyzer( * Replaces

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-26 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93898210 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -510,32 +510,94 @@ class Analyzer( * Replaces

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against merge_spark_pr.p...

2016-12-26 Thread holdenk
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/16405 I'll try and take a more detailed look later on tonight. --- 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 #16411: [SPARK-17984][YARN][Mesos][Deploy][WIP] add execu...

2016-12-26 Thread xiaochang-wu
GitHub user xiaochang-wu opened a pull request: https://github.com/apache/spark/pull/16411 [SPARK-17984][YARN][Mesos][Deploy][WIP] add executor launch prefix support ## What changes were proposed in this pull request? Complete Standalone, Yarn, Mesos support from #15579 and

[GitHub] spark issue #16412: [SPARK-19006][DOCS]mention spark.kryoserializer.buffer.m...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16412 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 project does not have this

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16405 **[Test build #70616 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70616/testReport)** for PR 16405 at commit

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against merge_spark_pr.p...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16405 **[Test build #70616 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70616/testReport)** for PR 16405 at commit

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16405 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70616/ Test FAILed. ---

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16405 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 project does not have this feature

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16405 **[Test build #70618 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70618/testReport)** for PR 16405 at commit

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16405 **[Test build #70618 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70618/testReport)** for PR 16405 at commit

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16405 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70618/ Test FAILed. ---

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16405 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 project does not have this feature

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16405 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70620/ Test FAILed. ---

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16405 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 project does not have this feature

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16405 **[Test build #70620 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70620/testReport)** for PR 16405 at commit

[GitHub] spark issue #13909: [SPARK-16213][SQL] Reduce runtime overhead of a program ...

2016-12-26 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/13909 @ueshin That will be a big mess if we validate the null here and there, I think we should only do it at the source side. If we do not trust the source side, there are a lot of other places that

[GitHub] spark issue #13909: [SPARK-16213][SQL] Reduce runtime overhead of a program ...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13909 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 project does not have this feature

[GitHub] spark issue #13909: [SPARK-16213][SQL] Reduce runtime overhead of a program ...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13909 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70614/ Test PASSed. ---

[GitHub] spark issue #15721: [SPARK-17772][ML][TEST] Add test functions for ML sample...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15721 **[Test build #70623 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70623/testReport)** for PR 15721 at commit

[GitHub] spark issue #15721: [SPARK-17772][ML][TEST] Add test functions for ML sample...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15721 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 project does not have this feature

[GitHub] spark issue #13909: [SPARK-16213][SQL] Reduce runtime overhead of a program ...

2016-12-26 Thread ueshin
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/13909 @cloud-fan Hmm, I didn't mean we don't trust the source side. Currently we DO validate the null regardless of `allowNull` value. The difference by the `allowNull` is setting null or

[GitHub] spark issue #16410: [SPARK-19005][SQL] Keep column ordering when a schema is...

2016-12-26 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/16410 I'm looking into the failure... --- 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

[GitHub] spark issue #15721: [SPARK-17772][ML][TEST] Add test functions for ML sample...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/15721 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70623/ Test PASSed. ---

[GitHub] spark issue #16392: [SPARK-18992] [SQL] Move spark.sql.hive.thriftServer.sin...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16392 **[Test build #70625 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70625/testReport)** for PR 16392 at commit

[GitHub] spark issue #15219: [SPARK-14098][SQL] Generate Java code to build CachedCol...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15219 **[Test build #70626 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70626/testReport)** for PR 15219 at commit

[GitHub] spark issue #16409: [SPARK-19004][SQL] Fix `testH2Dialect` by removing `getC...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16409 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 project does not have this feature

[GitHub] spark issue #16402: [SPARK-18999][SQL][minor] simplify Literal codegen

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16402 **[Test build #70615 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70615/testReport)** for PR 16402 at commit

[GitHub] spark issue #16411: [SPARK-17984][YARN][Mesos][Deploy][WIP] add executor lau...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16411 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 project does not have this

[GitHub] spark issue #16404: [SPARK-18969][SQL] Support grouping by nondeterministic ...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16404 **[Test build #70617 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70617/testReport)** for PR 16404 at commit

[GitHub] spark pull request #15721: [SPARK-17772][ML][TEST] Add test functions for ML...

2016-12-26 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/15721#discussion_r93891849 --- Diff: mllib/src/test/scala/org/apache/spark/ml/util/MLTestingUtils.scala --- @@ -182,34 +182,18 @@ object MLTestingUtils extends SparkFunSuite {

[GitHub] spark pull request #15721: [SPARK-17772][ML][TEST] Add test functions for ML...

2016-12-26 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/15721#discussion_r93891892 --- Diff: mllib/src/test/scala/org/apache/spark/ml/util/MLTestingUtils.scala --- @@ -224,4 +208,59 @@ object MLTestingUtils extends SparkFunSuite {

[GitHub] spark pull request #15721: [SPARK-17772][ML][TEST] Add test functions for ML...

2016-12-26 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/15721#discussion_r93891859 --- Diff: mllib/src/test/scala/org/apache/spark/ml/util/MLTestingUtils.scala --- @@ -182,34 +182,18 @@ object MLTestingUtils extends SparkFunSuite {

[GitHub] spark pull request #15721: [SPARK-17772][ML][TEST] Add test functions for ML...

2016-12-26 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/15721#discussion_r93891724 --- Diff: mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala --- @@ -1810,52 +1810,23 @@ class LogisticRegressionSuite

[GitHub] spark pull request #15721: [SPARK-17772][ML][TEST] Add test functions for ML...

2016-12-26 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/15721#discussion_r93891824 --- Diff: mllib/src/test/scala/org/apache/spark/ml/classification/NaiveBayesSuite.scala --- @@ -157,50 +162,26 @@ class NaiveBayesSuite extends

[GitHub] spark pull request #15721: [SPARK-17772][ML][TEST] Add test functions for ML...

2016-12-26 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/15721#discussion_r93891797 --- Diff: mllib/src/test/scala/org/apache/spark/ml/classification/NaiveBayesSuite.scala --- @@ -25,31 +25,36 @@ import

[GitHub] spark pull request #15721: [SPARK-17772][ML][TEST] Add test functions for ML...

2016-12-26 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/15721#discussion_r93891844 --- Diff: mllib/src/test/scala/org/apache/spark/ml/util/MLTestingUtils.scala --- @@ -18,15 +18,15 @@ package org.apache.spark.ml.util import

[GitHub] spark pull request #15721: [SPARK-17772][ML][TEST] Add test functions for ML...

2016-12-26 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/15721#discussion_r93891833 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala --- @@ -47,6 +49,11 @@ class LinearRegressionSuite

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16405 **[Test build #70622 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70622/testReport)** for PR 16405 at commit

[GitHub] spark issue #16413: Branch 1.3

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16413 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 project does not have this

[GitHub] spark issue #13909: [SPARK-16213][SQL] Reduce runtime overhead of a program ...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13909 **[Test build #70614 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70614/testReport)** for PR 13909 at commit

[GitHub] spark issue #16402: [SPARK-18999][SQL][minor] simplify Literal codegen

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16402 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 project does not have this feature

[GitHub] spark issue #16402: [SPARK-18999][SQL][minor] simplify Literal codegen

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16402 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70615/ Test FAILed. ---

[GitHub] spark pull request #16392: [SPARK-18992] [SQL] Move spark.sql.hive.thriftSer...

2016-12-26 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16392#discussion_r93896288 --- Diff: sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala --- @@ -262,7 +252,7 @@ class

[GitHub] spark pull request #16388: [SPARK-18989][SQL] DESC TABLE should not fail wit...

2016-12-26 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/16388#discussion_r93897706 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala --- @@ -408,8 +408,15 @@ private[hive] class HiveClientImpl(

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16405 **[Test build #70624 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70624/testReport)** for PR 16405 at commit

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-26 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93898653 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala --- @@ -551,17 +551,26 @@ class SessionCatalog( *

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16405 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70624/ Test PASSed. ---

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16405 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 project does not have this feature

[GitHub] spark issue #13909: [SPARK-16213][SQL] Reduce runtime overhead of a program ...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13909 **[Test build #70614 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70614/testReport)** for PR 13909 at commit

[GitHub] spark issue #16397: [SPARK-18922][TESTS] Fix more path-related test failures...

2016-12-26 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16397 cc @srowen, could I please ask to review this one? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request #16233: [SPARK-18801][SQL] Add `View` operator to help re...

2016-12-26 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/16233#discussion_r93898801 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala --- @@ -860,6 +864,24 @@ abstract class

[GitHub] spark pull request #16409: [SPARK-19004][SQL] Fix `testH2Dialect` by removin...

2016-12-26 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16409#discussion_r93898675 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCWriteSuite.scala --- @@ -44,9 +44,6 @@ class JDBCWriteSuite extends SharedSQLContext with

[GitHub] spark issue #16405: [SPARK-19002][BUILD] Check pep8 against dev/*.py scripts

2016-12-26 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16405 Ah, this seems complaining in Python 3. --- 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 issue #16391: [SPARK-18990][SQL] make DatasetBenchmark fairer for Data...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16391 **[Test build #70619 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70619/testReport)** for PR 16391 at commit

[GitHub] spark issue #16397: [WIP][SPARK-18922][TESTS] Fix more path-related test fai...

2016-12-26 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16397 Here, I concatenated all the logs into single file - https://gist.github.com/HyukjinKwon/58567451773f87322c7009007e4fdc34 I just found each in PR description. --- If your project is

[GitHub] spark issue #15721: [SPARK-17772][ML][TEST] Add test functions for ML sample...

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/15721 **[Test build #70623 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70623/testReport)** for PR 15721 at commit

[GitHub] spark issue #16402: [SPARK-18999][SQL][minor] simplify Literal codegen

2016-12-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/16402 **[Test build #70615 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70615/testReport)** for PR 16402 at commit

[GitHub] spark issue #16391: [SPARK-18990][SQL] make DatasetBenchmark fairer for Data...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16391 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70619/ Test PASSed. ---

[GitHub] spark issue #16391: [SPARK-18990][SQL] make DatasetBenchmark fairer for Data...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16391 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 project does not have this feature

[GitHub] spark pull request #16320: [SPARK-18877][SQL] `CSVInferSchema.inferField` on...

2016-12-26 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16320#discussion_r93897319 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchemaSuite.scala --- @@ -114,4 +114,11 @@ class

[GitHub] spark issue #16344: [SPARK-18929][ML] Add Tweedie distribution in GLM

2016-12-26 Thread actuaryzhang
Github user actuaryzhang commented on the issue: https://github.com/apache/spark/pull/16344 @srowen @yanboliang Any additional issues regarding 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 your

  1   2   3   4   >