[GitHub] spark pull request: [SPARK-12734][BUILD] Backport Netty exclusion ...

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

[GitHub] spark pull request: [SPARK-12734][BUILD] Backport Netty exclusion ...

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

[GitHub] spark pull request: [SPARK-12734][BUILD] Backport Netty exclusion ...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10690#issuecomment-170457916 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-12734][BUILD] Backport Netty exclusion ...

2016-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10690#issuecomment-170457858 [Test build #49106 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49106/console) for PR 10690 at commit

[GitHub] spark pull request: [SPARK-12734][BUILD] Backport Netty exclusion ...

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

[GitHub] spark pull request: [HOTFIX][test-maven][SPARK-12734] Fix bug in N...

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

[GitHub] spark pull request: [HOTFIX][test-maven][SPARK-12734] Fix bug in N...

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

[GitHub] spark pull request: [SPARK-12734][BUILD] Backport Netty exclusion ...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10691#issuecomment-170454472 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: [HOTFIX][test-maven][SPARK-12734] Fix bug in N...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10693#issuecomment-170456629 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-12734][BUILD] Backport Netty exclusion ...

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

[GitHub] spark pull request: [SPARK-12536] [SQL] Added "[]" in Explain Outp...

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

[GitHub] spark pull request: [SPARK-12745] [SQL] Hive Parser: Limit is not ...

2016-01-10 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/10689#discussion_r49290681 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/CatalystQlSuite.scala --- @@ -49,4 +49,11 @@ class CatalystQlSuite extends PlanTest

[GitHub] spark pull request: [SPARK-12536] [SQL] Added "[]" in Explain Outp...

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

[GitHub] spark pull request: [SPARK-12536] [SQL] Added "[]" in Explain Outp...

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

[GitHub] spark pull request: [SPARK-12536] [SQL] Added "[]" in Explain Outp...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10646#issuecomment-170434507 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-12610][SQL][WIP] Add Anti join operator...

2016-01-10 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/10563#discussion_r49290887 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala --- @@ -25,17 +25,26 @@ import

[GitHub] spark pull request: [Spark-12260][wip][Streaming]Graceful Shutdown...

2016-01-10 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/10252#issuecomment-170436089 After talk with @mwws offline, we probably need to serialize the `accumulator` variables as well. I personally agree with @srowen, we need to think

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

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

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10480#issuecomment-170436166 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-12224][SPARKR] R support for JDBC sourc...

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

[GitHub] spark pull request: [SPARK-12610][SQL][WIP] Add Anti join operator...

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

[GitHub] spark pull request: [SPARK-12610][SQL][WIP] Add Anti join operator...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10563#issuecomment-170436782 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-12610][SQL][WIP] Add Anti join operator...

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

[GitHub] spark pull request: [SPARK-12610][SQL][WIP] Add Anti join operator...

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

[GitHub] spark pull request: [SPARK-12536] [SQL] Added "[]" in Explain Outp...

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

[GitHub] spark pull request: [SPARK-12734][BUILD] Backport Netty exclusion ...

2016-01-10 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10690#issuecomment-170439459 This is NOT ready to merge; it looks like these exclusions have somehow catastrophically broken tests. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-12734][BUILD] Backport Netty exclusion ...

2016-01-10 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10691#issuecomment-170439442 This is NOT ready to merge; it looks like these exclusions have somehow catastrophically broken tests. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-4628][BUILD] Add a resolver to MiMaBuil...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10688#issuecomment-170413848 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-4628][BUILD] Add a resolver to MiMaBuil...

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

[GitHub] spark pull request: [SPARK-4628][BUILD] Add a resolver to MiMaBuil...

2016-01-10 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10688#discussion_r49285986 --- Diff: project/MimaBuild.scala --- @@ -91,11 +91,16 @@ object MimaBuild { def mimaSettings(sparkHome: File, projectRef: ProjectRef) = {

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

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

[GitHub] spark pull request: [SPARK-4628][BUILD] Add a resolver to MiMaBuil...

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

[GitHub] spark pull request: [SPARK-12692][BUILD][STREAMING] Scala style: F...

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

[GitHub] spark pull request: [SPARK-12692][BUILD][STREAMING] Scala style: F...

2016-01-10 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/10685#issuecomment-170419723 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-12692][BUILD][STREAMING] Scala style: F...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10685#issuecomment-170422188 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-12692][BUILD][STREAMING] Scala style: F...

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

[GitHub] spark pull request: [SPARK-12692][BUILD][STREAMING] Scala style: F...

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

[GitHub] spark pull request: [SPARK-12745] [SQL] Hive Parser: Limit is not ...

2016-01-10 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10689#issuecomment-170422086 @hvanhovell @rxin Could you take a look? Thank you! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-9858][SPARK-9859][SPARK-9861][SQL] Add ...

2016-01-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/9276#discussion_r49287822 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/ExchangeCoordinator.scala --- @@ -0,0 +1,260 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

2016-01-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/10480#issuecomment-170424371 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

[GitHub] spark pull request: [SPARK-12744] [SQL] Change parsing JSON intege...

2016-01-10 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10687#issuecomment-170424304 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-12744] [SQL] Change parsing JSON intege...

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

[GitHub] spark pull request: [SPARK-10359][PROJECT-INFRA] Backport dev/test...

2016-01-10 Thread JoshRosen
Github user JoshRosen closed the pull request at: https://github.com/apache/spark/pull/10679 --- 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-12745] [SQL] Hive Parser: Limit is not ...

2016-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10689#discussion_r49288754 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/CatalystQlSuite.scala --- @@ -49,4 +49,11 @@ class CatalystQlSuite extends PlanTest {

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10612 --- 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-4628][BUILD] Add a resolver to MiMaBuil...

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

[GitHub] spark pull request: [SPARK-12558][SQL][WIP] AnalysisException when...

2016-01-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/10520#discussion_r49289367 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveShim.scala --- @@ -123,6 +126,24 @@ private[hive] object HiveShim { // for

[GitHub] spark pull request: [SPARK-12582][Test] IndexShuffleBlockResolverS...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10526#issuecomment-170428263 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-12582][Test] IndexShuffleBlockResolverS...

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

[GitHub] spark pull request: [SPARK-12582][Test] IndexShuffleBlockResolverS...

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

[GitHub] spark pull request: [SPARK-12582][Test] IndexShuffleBlockResolverS...

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

[GitHub] spark pull request: [SPARK-12734][BUILD] Backport Netty exclusion ...

2016-01-10 Thread JoshRosen
GitHub user JoshRosen opened a pull request: https://github.com/apache/spark/pull/10690 [SPARK-12734][BUILD] Backport Netty exclusion + Maven enforcer fixes to branch-1.5 This patch backports the Netty exclusion fixes from #10672 to branch-1.5. You can merge this pull request into

[GitHub] spark pull request: [SPARK-4628][BUILD] Add a resolver to MiMaBuil...

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

[GitHub] spark pull request: [SPARK-12734][BUILD] Backport Netty exclusion ...

2016-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10690#issuecomment-170429827 [Test build #49089 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49089/consoleFull) for PR 10690 at commit

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

2016-01-10 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/10480#discussion_r49286033 --- Diff: R/pkg/R/DataFrame.R --- @@ -2272,3 +2260,40 @@ setMethod("with", newEnv <- assignNewEnv(data)

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

2016-01-10 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/10480#discussion_r49286036 --- Diff: R/pkg/R/DataFrame.R --- @@ -2272,3 +2260,40 @@ setMethod("with", newEnv <- assignNewEnv(data)

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

2016-01-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/10480#issuecomment-170415749 rebased and updated. 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

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

2016-01-10 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/10480#discussion_r49286045 --- Diff: R/pkg/R/SQLContext.R --- @@ -556,3 +556,61 @@ createExternalTable <- function(sqlContext, tableName, path = NULL, source = NUL sdf <-

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

2016-01-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/10480#issuecomment-170418270 ``` spark-mllib: found 0 potential binary incompatibilities (filtered 10) sbt.ResolveException: unresolved dependency:

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10480#issuecomment-170418047 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-12224][SPARKR] R support for JDBC sourc...

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

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

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

[GitHub] spark pull request: [SPARK-12692][BUILD][STREAMING] Scala style: F...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10685#issuecomment-170419331 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-12692][BUILD][STREAMING] Scala style: F...

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

[GitHub] spark pull request: [SPARK-12582][Test] IndexShuffleBlockResolverS...

2016-01-10 Thread yucai
Github user yucai commented on the pull request: https://github.com/apache/spark/pull/10526#issuecomment-170419313 Sorry, I miss the conversation! I will update it right now. --- 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-4628][BUILD] Add a resolver to MiMaBuil...

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

[GitHub] spark pull request: SPARK-12637 Print stage info of finished stage...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10585#issuecomment-170421549 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-12637 Print stage info of finished stage...

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

[GitHub] spark pull request: SPARK-12637 Print stage info of finished stage...

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

[GitHub] spark pull request: [SPARK-12745] [SQL] Hive Parser: Limit is not ...

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

[GitHub] spark pull request: [SPARK-12745] [SQL] Hive Parser: Limit is not ...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10689#issuecomment-170421463 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-12744] [SQL] Change parsing JSON intege...

2016-01-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/10687#discussion_r49288092 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala --- @@ -1465,4 +1465,15 @@ class JsonSuite extends

[GitHub] spark pull request: [SPARK-12744] [SQL] Change parsing JSON intege...

2016-01-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/10687#discussion_r49288079 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala --- @@ -1465,4 +1465,15 @@ class JsonSuite extends

[GitHub] spark pull request: [SPARK-12744] [SQL] Change parsing JSON intege...

2016-01-10 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10687#issuecomment-170424902 @antlypls Thank you for the PR! Overall it looks good. Just left two comments about the test. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-10359][PROJECT-INFRA] Backport dev/test...

2016-01-10 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10679#issuecomment-170426165 Test failure is unrelated (it failed almost at the very end of the build with a timeout), so I'm going to merge this now in order to unblock another backport. ---

[GitHub] spark pull request: [SPARK-12558][SQL][WIP] AnalysisException when...

2016-01-10 Thread dilipbiswal
Github user dilipbiswal commented on a diff in the pull request: https://github.com/apache/spark/pull/10520#discussion_r49288690 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveShim.scala --- @@ -125,6 +125,12 @@ private[hive] object HiveShim { // for

[GitHub] spark pull request: [SPARK-12734][BUILD] Fix Netty exclusion and u...

2016-01-10 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/10672#issuecomment-170426398 Thanks for reviewing. I'm going to merge this into master and will open separate PRs to test the backports, which might possibly require a different set of exclusion

[GitHub] spark pull request: [SPARK-12744] [SQL] Change parsing JSON intege...

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

[GitHub] spark pull request: [SPARK-12744] [SQL] Change parsing JSON intege...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10687#issuecomment-170426462 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-12744] [SQL] Change parsing JSON intege...

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

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170426738 I've merged this. 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-4628][BUILD] Add a resolver to MiMaBuil...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10688#issuecomment-170427636 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-4628][BUILD] Add a resolver to MiMaBuil...

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

[GitHub] spark pull request: [SPARK-4628][BUILD] Add a resolver to MiMaBuil...

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

[GitHub] spark pull request: [SPARK-4628][BUILD] Add a resolver to MiMaBuil...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10688#issuecomment-170428533 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-12734][BUILD] Backport Netty exclusion ...

2016-01-10 Thread JoshRosen
GitHub user JoshRosen opened a pull request: https://github.com/apache/spark/pull/10691 [SPARK-12734][BUILD] Backport Netty exclusion + Maven enforcer fixes to branch-1.6 This patch backports the Netty exclusion fixes from #10672 to branch-1.6. You can merge this pull request into

[GitHub] spark pull request: [SPARK-12558][SQL][WIP] AnalysisException when...

2016-01-10 Thread dilipbiswal
Github user dilipbiswal commented on a diff in the pull request: https://github.com/apache/spark/pull/10520#discussion_r49289994 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveShim.scala --- @@ -123,6 +126,24 @@ private[hive] object HiveShim { // for

[GitHub] spark pull request: [SPARK-10359][PROJECT-INFRA] Backport dev/test...

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

[GitHub] spark pull request: [SPARK-10359][PROJECT-INFRA] Backport dev/test...

2016-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10679#issuecomment-170416071 **[Test build #49076 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49076/console)** for PR 10679 at commit

[GitHub] spark pull request: [SPARK-10359][PROJECT-INFRA] Backport dev/test...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10679#issuecomment-170416096 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-9858][SPARK-9859][SPARK-9861][SQL] Add ...

2016-01-10 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/9276#discussion_r49286654 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/ExchangeCoordinator.scala --- @@ -0,0 +1,260 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-12692][BUILD][STREAMING] Scala style: F...

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

[GitHub] spark pull request: [SPARK-12745] [SQL] Hive Parser: Limit is not ...

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

[GitHub] spark pull request: [SPARK-12705] [SQL] AnalysisException: Sorting...

2016-01-10 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10678#issuecomment-170421900 @davies Could you take a look if the fix covers the analysis resolution issue in TPCD? Thank you! --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-12638] [API DOC] Parameter explanation ...

2016-01-10 Thread Wenpei
Github user Wenpei commented on the pull request: https://github.com/apache/spark/pull/10587#issuecomment-170423933 Thank you @srowen, I refine parameter explanation base on your suggestion. Pls take a look. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-12224][SPARKR] R support for JDBC sourc...

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

[GitHub] spark pull request: [SPARK-12734][BUILD] Fix Netty exclusion and u...

2016-01-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10672 --- 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] [build] Enable import ordering er...

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

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

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

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170426632 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

  1   2   3   >