[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-02-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10964 --- 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-13056][SQL] map column would throw NPE ...

2016-02-02 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-178776773 Merged to master and 1.6 --- 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-13056][SQL] map column would throw NPE ...

2016-02-01 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-177865841 LGTM, pending 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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-02-01 Thread adrian-wang
Github user adrian-wang commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51392316 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -2046,6 +2046,15 @@ class SQLQuerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-02-01 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51391066 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -2046,6 +2046,15 @@ class SQLQuerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-02-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-177872346 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-13056][SQL] map column would throw NPE ...

2016-02-01 Thread adrian-wang
Github user adrian-wang commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-177877914 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-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-02-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-177920132 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-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-177810918 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-13056][SQL] map column would throw NPE ...

2016-01-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-177824349 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-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-31 Thread adrian-wang
Github user adrian-wang commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51381272 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -2056,4 +2056,11 @@ class SQLQuerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-15877 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-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-31 Thread tejasapatil
Github user tejasapatil commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51380304 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -2056,4 +2056,11 @@ class SQLQuerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-31 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51380297 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -2056,4 +2056,11 @@ class SQLQuerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-31 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51379478 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -2056,4 +2056,11 @@ class SQLQuerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-31 Thread tejasapatil
Github user tejasapatil commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51380186 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -2056,4 +2056,11 @@ class SQLQuerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-176683786 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-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-29 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51239924 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala --- @@ -1463,4 +1463,11 @@ class SQLQuerySuite extends

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-29 Thread adrian-wang
Github user adrian-wang commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51240308 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala --- @@ -1463,4 +1463,11 @@ class SQLQuerySuite extends

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-29 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51292487 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala --- @@ -1463,4 +1463,11 @@ class SQLQuerySuite extends

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-176599391 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-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-28 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51190079 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeExtractors.scala --- @@ -278,7 +278,7 @@ case class

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-28 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51189982 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeExtractors.scala --- @@ -307,7 +307,7 @@ case class

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-28 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51225993 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala --- @@ -1463,4 +1463,11 @@ class SQLQuerySuite extends

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-28 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51226065 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeExtractors.scala --- @@ -307,8 +310,8 @@ case class

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-28 Thread adrian-wang
Github user adrian-wang commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51226292 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeExtractors.scala --- @@ -218,7 +218,7 @@ case class

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-28 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10964#discussion_r51226510 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeExtractors.scala --- @@ -218,7 +218,7 @@ case class

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-27 Thread adrian-wang
GitHub user adrian-wang opened a pull request: https://github.com/apache/spark/pull/10964 [SPARK-13056][SQL] map column would throw NPE if value is null Jira: https://issues.apache.org/jira/browse/SPARK-13056 Create a map like { "a": "somestring", "b": null}

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

2016-01-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10964#issuecomment-176037481 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-13056][SQL] map column would throw NPE ...

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

[GitHub] spark pull request: [SPARK-13056][SQL] map column would throw NPE ...

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