[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11410 --- 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-13456][SQL] fix creating encoders for c...

2016-03-21 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-199393749 Thanks. Merging to master. --- 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-13456][SQL] fix creating encoders for c...

2016-03-21 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-199238467 cc @rxin --- 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-13456][SQL] fix creating encoders for c...

2016-03-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-199212226 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-21 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-199163066 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-13456][SQL] fix creating encoders for c...

2016-03-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-199157021 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-20 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-199135927 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-199126314 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-20 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-199090260 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-13456][SQL] fix creating encoders for c...

2016-03-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-199089510 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-198283125 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-19 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-198242915 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-198457435 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-13456][SQL] fix creating encoders for c...

2016-03-19 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-198240951 ping @retronym again --- 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-13456][SQL] fix creating encoders for c...

2016-03-18 Thread retronym
Github user retronym commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-198330775 So long as you've got some defence from a test case, your current solution seems okay. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-15 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-196690093 Hi @retronym , your idea looks good, but in our situation, we only have a class name string and it's not that easy to get the line object. BTW the REPL classes are

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-14 Thread retronym
Github user retronym commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-196676313 Since offering review, I've found a more direct way to get to the object. ``` Welcome to Scala 2.11.8 (Java HotSpot(TM) 64-Bit Server VM, Java 1.8.0_71).

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-196675770 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-14 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-196642468 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-13456][SQL] fix creating encoders for c...

2016-03-10 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-195174175 @retronym Thank you for the review! I'd like to merge this fix. Sounds good? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-10 Thread retronym
Github user retronym commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r55785695 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/OuterScopes.scala --- @@ -39,4 +41,47 @@ object OuterScopes { def

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-10 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r55781359 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/OuterScopes.scala --- @@ -39,4 +41,47 @@ object OuterScopes { def

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-194108194 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-08 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-193982927 @retronym Will have time to take another round of review? Thank you! --- 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-13456][SQL] fix creating encoders for c...

2016-03-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-193744957 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-08 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-193660924 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-193653822 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-13456][SQL] fix creating encoders for c...

2016-03-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r55296075 --- Diff: repl/scala-2.11/src/test/scala/org/apache/spark/repl/ReplSuite.scala --- @@ -254,6 +254,30 @@ class ReplSuite extends SparkFunSuite {

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-191153683 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-01 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-191098934 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-191079873 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-01 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54678025 --- Diff: repl/scala-2.11/src/test/scala/org/apache/spark/repl/ReplSuite.scala --- @@ -296,6 +320,31 @@ class ReplSuite extends SparkFunSuite { }

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-01 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54672837 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/OuterScopes.scala --- @@ -39,4 +41,35 @@ object OuterScopes { def

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-01 Thread retronym
Github user retronym commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54647099 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/OuterScopes.scala --- @@ -39,4 +41,35 @@ object OuterScopes { def

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-01 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54570439 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/OuterScopes.scala --- @@ -39,4 +41,35 @@ object OuterScopes { def

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-01 Thread retronym
Github user retronym commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54557328 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/OuterScopes.scala --- @@ -39,4 +41,35 @@ object OuterScopes { def

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-01 Thread ScrapCodes
Github user ScrapCodes commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54555457 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/OuterScopes.scala --- @@ -39,4 +41,35 @@ object OuterScopes { def

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-190653742 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-03-01 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54535121 --- Diff: repl/scala-2.11/src/test/scala/org/apache/spark/repl/ReplSuite.scala --- @@ -254,6 +254,30 @@ class ReplSuite extends SparkFunSuite {

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-29 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54452439 --- Diff: repl/scala-2.11/src/test/scala/org/apache/spark/repl/ReplSuite.scala --- @@ -254,6 +254,30 @@ class ReplSuite extends SparkFunSuite {

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-29 Thread ScrapCodes
Github user ScrapCodes commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54401926 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/OuterScopes.scala --- @@ -39,4 +41,35 @@ object OuterScopes { def

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-189813853 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-27 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-189784873 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-13456][SQL] fix creating encoders for c...

2016-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-189687161 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-189686635 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-27 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54334648 --- Diff: repl/scala-2.10/src/test/scala/org/apache/spark/repl/ReplSuite.scala --- @@ -288,7 +288,7 @@ class ReplSuite extends SparkFunSuite {

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-27 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54334612 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/OuterScopes.scala --- @@ -39,4 +41,35 @@ object OuterScopes { def

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-27 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11410#discussion_r54334598 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/OuterScopes.scala --- @@ -39,4 +41,31 @@ object OuterScopes { def

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-27 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-189660014 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-189656038 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-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

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

[GitHub] spark pull request: [SPARK-13456][SQL] fix creating encoders for c...

2016-02-27 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11410#issuecomment-189632314 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

  1   2   >