[GitHub] spark pull request #16409: [SPARK-19004][SQL] Fix `JDBCWriteSuite.testH2Dial...

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

[GitHub] spark pull request #16404: [SPARK-18969][SQL] Support grouping by nondetermi...

2016-12-26 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16404#discussion_r93903360 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -1918,28 +1918,37 @@ class Analyzer( case

[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_r93902952 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCWriteSuite.scala --- @@ -44,9 +44,6 @@ class JDBCWriteSuite extends SharedSQLContext with

[GitHub] spark pull request #16393: [SPARK-18993] [Build] Revert Split test-tags into...

2016-12-26 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16393#discussion_r93902771 --- Diff: common/tags/pom.xml --- @@ -34,6 +34,14 @@ tags + + + org.scalatest +

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

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16392 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 #16392: [SPARK-18992] [SQL] Move spark.sql.hive.thriftServer.sin...

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

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

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

[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 finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70625/testReport)** for PR 16392 at commit

[GitHub] spark issue #12574: [SPARK-13857][ML][WIP] Add "recommend all" functionality...

2016-12-26 Thread hhbyyh
Github user hhbyyh commented on the issue: https://github.com/apache/spark/pull/12574 Hi @debasish83 SPARK-18235 was already closed as duplicated. I think @MLnick put a more thorough solution here. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark issue #16415: [SPARK-19007]Speedup and optimize the GradientBoostedTre...

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16415 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 pull request #16415: [SPARK-19007]Speedup and optimize the GradientBoo...

2016-12-26 Thread zdh2292390
GitHub user zdh2292390 opened a pull request: https://github.com/apache/spark/pull/16415 [SPARK-19007]Speedup and optimize the GradientBoostedTrees JIRA Issue: https://issues.apache.org/jira/browse/SPARK-19007 Speedup and optimize the GradientBoostedTrees in the

[GitHub] spark issue #16414: [SPARK-19009][DOC] Add streaming rest api doc

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

[GitHub] spark issue #16414: [SPARK-19009][DOC] Add streaming rest api doc

2016-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16414 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 #16409: [SPARK-19004][SQL] Fix `testH2Dialect` by removin...

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

[GitHub] spark issue #16414: [SPARK-19009][DOC] Add streaming rest api doc

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

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

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

[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_r93901583 --- 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 #16414: [SPARK-19009][DOC] Add streaming rest api doc

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

[GitHub] spark pull request #16404: [SPARK-18969][SQL] Support grouping by nondetermi...

2016-12-26 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/16404#discussion_r93901178 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -1918,28 +1918,37 @@ class Analyzer( case

[GitHub] spark pull request #16414: [SPARK-19009][DOC] Add streaming rest api doc

2016-12-26 Thread uncleGen
GitHub user uncleGen opened a pull request: https://github.com/apache/spark/pull/16414 [SPARK-19009][DOC] Add streaming rest api doc ## What changes were proposed in this pull request? add streaming rest api doc ## How was this patch tested? You can merge this

[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 #70627 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70627/testReport)** for PR 15219 at commit

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

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

[GitHub] spark pull request #16393: [SPARK-18993] [Build] Revert Split test-tags into...

2016-12-26 Thread wangmiao1981
Github user wangmiao1981 commented on a diff in the pull request: https://github.com/apache/spark/pull/16393#discussion_r93900810 --- Diff: common/tags/pom.xml --- @@ -34,6 +34,14 @@ tags + + + org.scalatest +

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

2016-12-26 Thread kiszk
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/15219 Jenkins, retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[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 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 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 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 #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_r93898562 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -658,6 +720,17 @@ class Analyzer(

[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 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 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_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 #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 #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

[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 #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 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 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 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 #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 #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 #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 #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 #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 #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 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 #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 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 #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 #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 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 #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 #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 #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 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 #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 #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 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 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 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 #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 #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 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_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_r93891833 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala --- @@ -47,6 +49,11 @@ class LinearRegressionSuite

[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 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 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 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 #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 #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 #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 #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 #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 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 #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 #70620 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70620/testReport)** for PR 16405 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 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

  1   2   3   4   >