[GitHub] spark pull request: [SPARK-12438][SQL] hiveContext.table() throws ...

2015-12-31 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/10529#issuecomment-168152257 OK to test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark pull request: [SPARK-4819] [WIP] Remove Guava's "Optional" f...

2015-12-31 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10513#issuecomment-168158630 If we are creating our own Optional, why don't we also support all the methods available on Guava, so we only need to break "import" compatibility? --- If your project

[GitHub] spark pull request: [SPARK-12438][SQL] Add SQLUserDefinedType supp...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10390#issuecomment-168150125 **[Test build #48547 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48547/consoleFull)** for PR 10390 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168154343 **[Test build #48549 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48549/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-4819] [WIP] Remove Guava's "Optional" f...

2015-12-31 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10513#issuecomment-168158788 My theory was that it makes it more compatible with Java 8 Optional. Later, moving to its implementation is a smaller change. We're taking the bigger change now. Or at

[GitHub] spark pull request: [Spark-10625] [SQL] Spark SQL JDBC read/write ...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8785#issuecomment-168148505 **[Test build #2280 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2280/consoleFull)** for PR 8785 at commit

[GitHub] spark pull request: [SPARK-12039][SQL] Re-enable HiveSparkSubmitSu...

2015-12-31 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10533#issuecomment-168157967 I've merged this. --- 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-5673] [MLlib] Implement Streaming wrapp...

2015-12-31 Thread catap
Github user catap commented on the pull request: https://github.com/apache/spark/pull/4456#issuecomment-168152178 @rxin, ok, I will --- 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-4226][SQL]Add subquery (not) in/exists ...

2015-12-31 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/9055#issuecomment-168152168 Found a related HIVE JIRA to support the left anti join: https://issues.apache.org/jira/browse/HIVE-12519 However, their proposed solution has a hole.

[GitHub] spark pull request: [SPARK-12461] [SQL] Add ExpressionDescription ...

2015-12-31 Thread vectorijk
Github user vectorijk commented on a diff in the pull request: https://github.com/apache/spark/pull/10489#discussion_r48650445 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala --- @@ -279,6 +330,9 @@ object Factorial { )

[GitHub] spark pull request: [SPARK-7995][SPARK-6280][Core]Remove AkkaRpcEn...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10459#issuecomment-168148715 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-7995][SPARK-6280][Core]Remove AkkaRpcEn...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10459#issuecomment-168148675 **[Test build #48545 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48545/consoleFull)** for PR 10459 at commit

[GitHub] spark pull request: [SPARK-7995][SPARK-6280][Core]Remove AkkaRpcEn...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10459#issuecomment-168148714 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-12363][Mllib] Remove setRun and fix Pow...

2015-12-31 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10539#issuecomment-168149596 @yanboliang Because that test doesn't fail so this commit doesn't set `TripletFields.All` there. However, I tried it and it also passes the test. Do you think we

[GitHub] spark pull request: [SPARK-12438][SQL] Add SQLUserDefinedType supp...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10390#issuecomment-168153873 **[Test build #48548 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48548/consoleFull)** for PR 10390 at commit

[GitHub] spark pull request: [SPARK-6724] [MLlib] Support model save/load f...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9267#issuecomment-168150628 **[Test build #48546 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48546/consoleFull)** for PR 9267 at commit

[GitHub] spark pull request: [SPARK-12438][SQL] Add SQLUserDefinedType supp...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10390#issuecomment-168150652 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-12438][SQL] Add SQLUserDefinedType supp...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10390#issuecomment-168150653 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12363][Mllib] Remove setRun and fix Pow...

2015-12-31 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/10539#issuecomment-168150722 @viirya I'm not very familiar with GraphX and I just found they are do the same thing for different interface. Maybe we can hear others' opinions. --- If your

[GitHub] spark pull request: [SPARK-6724] [MLlib] Support model save/load f...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9267#issuecomment-168150715 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6724] [MLlib] Support model save/load f...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9267#issuecomment-168150714 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-12438][SQL] Add SQLUserDefinedType supp...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10390#issuecomment-168150645 **[Test build #48547 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48547/consoleFull)** for PR 10390 at commit

[GitHub] spark pull request: [SPARK-3873] [streaming] Import order fixes fo...

2015-12-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10532 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: [SPARK-12438][SQL] hiveContext.table() throws ...

2015-12-31 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10529#discussion_r48651665 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Catalog.scala --- @@ -104,7 +104,7 @@ class SimpleCatalog(val conf: CatalystConf)

[GitHub] spark pull request: [SPARK-12039][SQL] Re-enable HiveSparkSubmitSu...

2015-12-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10533 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: [SPARK-3873] [streaming] Import order fixes fo...

2015-12-31 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10532#issuecomment-168158419 I've merged this. --- 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-7995][SPARK-6280][Core]Remove AkkaRpcEn...

2015-12-31 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10459#issuecomment-168148942 I've merged it. 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 your project does not have

[GitHub] spark pull request: [SPARK-12592][SQL][TEST] Don't mute Spark logg...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10540#issuecomment-168181589 **[Test build #48551 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48551/consoleFull)** for PR 10540 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168184217 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168184213 **[Test build #48553 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48553/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168184215 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-12592][SQL][TEST] Don't mute Spark logg...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10540#issuecomment-168192401 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-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168194460 @JoshRosen these builds fail often and non-deterministically with a **fails build dependency** tests. The problem is that I cannot find the cause of this problem

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168208226 **[Test build #48555 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48555/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168210083 **[Test build #48557 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48557/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168182227 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

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168178455 **[Test build #48550 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48550/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168182029 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168182024 **[Test build #48552 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48552/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168182027 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-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168184101 **[Test build #48553 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48553/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168190691 When running any test suite that extends `HiveComparisonTest`, detail logs can be found in `sql/hive/target/unit-tests.log`. If a query string is convertible, we

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168191048 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

[GitHub] spark pull request: [SPARK-12438][SQL] hiveContext.table() throws ...

2015-12-31 Thread thomastechs
Github user thomastechs commented on a diff in the pull request: https://github.com/apache/spark/pull/10529#discussion_r48653675 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Catalog.scala --- @@ -104,7 +104,7 @@ class SimpleCatalog(val conf:

[GitHub] spark pull request: [SPARK-12438][SQL] hiveContext.table() throws ...

2015-12-31 Thread thomastechs
Github user thomastechs commented on the pull request: https://github.com/apache/spark/pull/10529#issuecomment-168173475 @rxin I have updated the style check difference ,you mentioned. and committed to my branch. And I see it is reflected here in this same pull request. Please

[GitHub] spark pull request: [SPARK-12592][SQL][TEST] Don't mute Spark logg...

2015-12-31 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/10540#issuecomment-168179906 Hit this issue while working on canonicalized native view support. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168179937 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

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168208273 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168208271 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-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168174676 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

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168174246 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168174243 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-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168178889 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-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168178887 **[Test build #48550 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48550/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168178890 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168191248 **[Test build #48554 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48554/consoleFull)** for PR 10541 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168191293 **[Test build #48555 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48555/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12592][SQL][TEST] Don't mute Spark logg...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10540#issuecomment-168192337 **[Test build #48551 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48551/consoleFull)** for PR 10540 at commit

[GitHub] spark pull request: [SPARK-12592][SQL][TEST] Don't mute Spark logg...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10540#issuecomment-168192402 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168209143 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

[GitHub] spark pull request: [SPARK-12438][SQL] Add SQLUserDefinedType supp...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10390#issuecomment-168165273 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-12438][SQL] Add SQLUserDefinedType supp...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10390#issuecomment-168165278 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12438][SQL] Add SQLUserDefinedType supp...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10390#issuecomment-168165009 **[Test build #48548 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48548/consoleFull)** for PR 10390 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168174120 **[Test build #48549 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48549/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168194609 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-12507][Streaming][Document]Expose close...

2015-12-31 Thread brkyvz
Github user brkyvz commented on the pull request: https://github.com/apache/spark/pull/10453#issuecomment-168194624 @zsxwing Thanks! LGTM --- 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-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168194592 **[Test build #48554 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48554/consoleFull)** for PR 10541 at commit

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168194610 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12470] [SQL] Fix size reduction calcula...

2015-12-31 Thread robbinspg
Github user robbinspg commented on the pull request: https://github.com/apache/spark/pull/10421#issuecomment-168172679 re-merged with latest master Please retest --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-12592][SQL][TEST] Don't mute Spark logg...

2015-12-31 Thread liancheng
GitHub user liancheng opened a pull request: https://github.com/apache/spark/pull/10540 [SPARK-12592][SQL][TEST] Don't mute Spark loggers in TestHive.reset() There's a hack done in `TestHive.reset()`, which intended to mute noisy Hive loggers. However, Spark testing loggers are

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168181912 **[Test build #48552 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48552/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread liancheng
GitHub user liancheng opened a pull request: https://github.com/apache/spark/pull/10541 [SPARK-12592][SQL][WIP] Converts resolved logical plan back to SQL This PR tries to enable Spark SQL to convert resolved logical plans back to SQL query strings. For now, the major use case is

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168191297 I found SQL generation in Slick can be a good reference for attacking limitations mentioned in the PR description. But the current approach should be enough for

[GitHub] spark pull request: [SPARK-12541] [SQL] support cube/rollup as fun...

2015-12-31 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10522#discussion_r48660161 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -203,15 +203,15 @@ class Analyzer( *

[GitHub] spark pull request: [SPARK-12541] [SQL] support cube/rollup as fun...

2015-12-31 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10522#discussion_r48660142 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -190,7 +190,7 @@ class Analyzer( *

[GitHub] spark pull request: [SPARK-6724][WIP][MLLIB]Model import/export fo...

2015-12-31 Thread FlytxtRnD
Github user FlytxtRnD closed the pull request at: https://github.com/apache/spark/pull/7320 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: [Spark-10625] [SQL] Spark SQL JDBC read/write ...

2015-12-31 Thread tribbloid
Github user tribbloid commented on the pull request: https://github.com/apache/spark/pull/8785#issuecomment-168211061 Looks like the removed qualifier of Map in JDBCRelation has caused build to fail (Map is cast to immutable.Map). I'm adding back the qualifier scala.collection.Map

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168211120 **[Test build #48558 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48558/consoleFull)** for PR 10541 at commit

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168211126 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168211125 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-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168234676 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168234675 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-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168234652 **[Test build #48557 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48557/consoleFull)** for PR 10525 at commit

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168220607 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168220278 **[Test build #48558 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48558/consoleFull)** for PR 10541 at commit

[GitHub] spark pull request: [SPARK-12592][SQL][WIP] Converts resolved logi...

2015-12-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10541#issuecomment-168220603 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-6724][WIP][MLLIB]Model import/export fo...

2015-12-31 Thread FlytxtRnD
Github user FlytxtRnD commented on the pull request: https://github.com/apache/spark/pull/7320#issuecomment-168222327 @jkbradley Thank you for the review. I will close this issue and will surely help in testing the other PR. Also it would be great if you could take a look at #6880

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168240969 I have a few hunches about what could be causing the dependency manifest checking problems. I'll keep investigating when I get home later tonight. I think I've

[GitHub] spark pull request: [SPARK-12470] [SQL] Fix size reduction calcula...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10421#issuecomment-168259354 **[Test build #2282 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2282/consoleFull)** for PR 10421 at commit

[GitHub] spark pull request: [SPARK-12438][SQL] hiveContext.table() throws ...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10529#issuecomment-168259243 **[Test build #2281 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2281/consoleFull)** for PR 10529 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168259081 Should we revert that dependency check thing first? --- 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-12594] [SQL] Outer Join Conversion: Out...

2015-12-31 Thread gatorsmile
GitHub user gatorsmile opened a pull request: https://github.com/apache/spark/pull/10542 [SPARK-12594] [SQL] Outer Join Conversion: Outer/Right/Left to Inner and Outer to Left/Right Conversion of outer joins, if the local predicates can restrict the result sets so that all

[GitHub] spark pull request: [SPARK-12594] [SQL] Outer Join Conversion: Out...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10542#issuecomment-168269686 **[Test build #48559 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48559/consoleFull)** for PR 10542 at commit

[GitHub] spark pull request: [SPARK-12362][SQL][WIP] Inline Hive Parser

2015-12-31 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10525#issuecomment-168278782 Aha! It appears that there _is_ a portability issue in `sort`, or at least in how we're using it in the dependency-checking script. When I run `echo -e

[GitHub] spark pull request: [SPARK-12504][SQL] Masking credentials in the ...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10452#issuecomment-168271940 **[Test build #2283 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2283/consoleFull)** for PR 10452 at commit

[GitHub] spark pull request: [SPARK-12504][SQL] Masking credentials in the ...

2015-12-31 Thread sureshthalamati
Github user sureshthalamati commented on the pull request: https://github.com/apache/spark/pull/10452#issuecomment-168260505 Thanks, Michael. Updated the PR. Please review. --- 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-12504][SQL] Masking credentials in the ...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10452#issuecomment-168259705 **[Test build #2283 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2283/consoleFull)** for PR 10452 at commit

[GitHub] spark pull request: [SPARK-12438][SQL] hiveContext.table() throws ...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10529#issuecomment-168261528 **[Test build #2281 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2281/consoleFull)** for PR 10529 at commit

[GitHub] spark pull request: [SPARK-12470] [SQL] Fix size reduction calcula...

2015-12-31 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10421#issuecomment-168268755 **[Test build #2282 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2282/consoleFull)** for PR 10421 at commit

[GitHub] spark pull request: [SPARK-10359][PROJECT-INFRA] Multiple fixes to...

2015-12-31 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10543#issuecomment-168279632 Actually, it occurred to me that I should really be setting `LC_ALL=C` at the top of the script to guard against future problems in case we add additional

  1   2   >