[GitHub] spark pull request: [SPARK-12415] Do not use closure serializer to...

2015-12-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10368#issuecomment-167044664 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-12476][SQL] Implement JdbcRelation#unha...

2015-12-23 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10427#issuecomment-167045790 @HyukjinKwon Anyway, I added a test by reflecting this discussion. thanks. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: Branch 1.6

2015-12-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10467#issuecomment-167047758 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11394][SQL] Throw IllegalArgumentExcept...

2015-12-23 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/9350#issuecomment-167056743 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

2015-12-23 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10330#discussion_r48399567 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala --- @@ -200,6 +204,10 @@ case class TakeOrderedAndProject(

[GitHub] spark pull request: [SPARK-12476][SQL] Implement JdbcRelation#unha...

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

[GitHub] spark pull request: [SPARK-8641][SPARK-12455][SQL] Native Spark Wi...

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

[GitHub] spark pull request: [SPARK-12415] Do not use closure serializer to...

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

[GitHub] spark pull request: [SPARK-12409][SPARK-12387][SPARK-12391][SQL] S...

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

[GitHub] spark pull request: [SPARK-12476][SQL] Implement JdbcRelation#unha...

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

[GitHub] spark pull request: [SPARK-12481] [CORE] [STREAMING] [SQL] Remove ...

2015-12-23 Thread srowen
GitHub user srowen opened a pull request: https://github.com/apache/spark/pull/10446 [SPARK-12481] [CORE] [STREAMING] [SQL] Remove usage of Hadoop deprecated APIs and reflection that supported 1.x Remove use of deprecated Hadoop APIs now that 2.2+ is required You can merge this

[GitHub] spark pull request: [SPARK-12311][CORE] Restore previous value of ...

2015-12-23 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10289#issuecomment-166881220 I'm going to go ahead and merge this today --- 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-12396][Core] Modify the function schedu...

2015-12-23 Thread echoTomei
GitHub user echoTomei opened a pull request: https://github.com/apache/spark/pull/10447 [SPARK-12396][Core] Modify the function scheduleAtFixedRate to schedule. Instead of just cancel the registrationRetryTimer to avoid driver retry connect to master, change the function to

[GitHub] spark pull request: [SPARK-12439][SQL] Fix toCatalystArray and Map...

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

[GitHub] spark pull request: [SPARK-12396][Core] Modify the function schedu...

2015-12-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10447#issuecomment-166881505 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-12481] [CORE] [STREAMING] [SQL] Remove ...

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

[GitHub] spark pull request: [SPARK-12480][SQL] add Hash expression that ca...

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

[GitHub] spark pull request: [SPARK-12481] [CORE] [STREAMING] [SQL] Remove ...

2015-12-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10446#issuecomment-166888672 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-12481] [CORE] [STREAMING] [SQL] Remove ...

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

[GitHub] spark pull request: [SPARK-12481] [CORE] [STREAMING] [SQL] Remove ...

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

[GitHub] spark pull request: [SPARK-12480][SQL] add Hash expression that ca...

2015-12-23 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10435#discussion_r48342878 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/stat/StatFunctions.scala --- @@ -114,6 +114,7 @@ private[sql] object StatFunctions

[GitHub] spark pull request: [SPARK-6624][SQL]Add CNF Normalization as part...

2015-12-23 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/8200#issuecomment-166860512 Just realized that [one of the common factor elimination rules][1] defined in `BooleanSimplification` actually reverts CNF normalization, thus we may never reach

[GitHub] spark pull request: [CORE] Refactoring: Use pattern matching and d...

2015-12-23 Thread jaceklaskowski
Github user jaceklaskowski closed the pull request at: https://github.com/apache/spark/pull/10433 --- 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

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

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

[GitHub] spark pull request: [SPARK-6624][WIP] Another alternative version ...

2015-12-23 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/10444#discussion_r48331968 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -583,6 +583,12 @@ object BooleanSimplification

[GitHub] spark pull request: [SPARK-6624][SQL]Add CNF Normalization as part...

2015-12-23 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/8200#issuecomment-166844885 Made a draft PR #10444 based on the idea described in [another comment of mine][1] above to workaround exponential expansion issue of CNF normalization. [1]:

[GitHub] spark pull request: [SPARK-6624][SQL]Add CNF Normalization as part...

2015-12-23 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/8200#discussion_r48331134 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -464,6 +465,24 @@ object OptimizeIn extends

[GitHub] spark pull request: [SPARK-12495][SQL] use true as default value f...

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

[GitHub] spark pull request: [SPARK-12495][SQL] use true as default value f...

2015-12-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10443#issuecomment-166843816 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-12495][SQL] use true as default value f...

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

[GitHub] spark pull request: [SPARK-6624][WIP] Another alternative version ...

2015-12-23 Thread liancheng
GitHub user liancheng opened a pull request: https://github.com/apache/spark/pull/10444 [SPARK-6624][WIP] Another alternative version of CNF normalization This PR is a draft version of another alternative of CNF normalization based on [comment][1] in PR #8200. This PR doesn't

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2015-12-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10411#issuecomment-166845460 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-6624][WIP] Another alternative version ...

2015-12-23 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/10444#discussion_r48332063 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala --- @@ -47,6 +48,34 @@ trait Predicate extends

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

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

[GitHub] spark pull request: [SPARK-12495][SQL] use true as default value f...

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

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

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

[GitHub] spark pull request: [SPARK-6624][SQL]Add CNF Normalization as part...

2015-12-23 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/8200#discussion_r48328448 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -464,6 +465,24 @@ object OptimizeIn extends

<    1   2   3   4   5