Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9770#discussion_r45238084
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1063,6 +1065,34 @@ class Analyzer(
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9770
---
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 user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157810190
Looks great! Merging 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157410913
**[Test build #46090 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46090/consoleFull)**
for PR 9770 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157401188
**[Test build #46088 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46088/consoleFull)**
for PR 9770 at commit
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/9770
[SPARK-11725][SQL] correctly handle null inputs for UDF
If user use primitive parameters in UDF, there is no way for him to do the
null-check for primitive inputs, so we are assuming the
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157398240
cc @marmbrus
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157442850
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157442620
**[Test build #46088 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46088/consoleFull)**
for PR 9770 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157442853
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9770#discussion_r45104665
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -719,6 +719,12 @@ trait ScalaReflection {
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157560144
**[Test build #46130 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46130/consoleFull)**
for PR 9770 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157578462
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157578365
**[Test build #46130 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46130/consoleFull)**
for PR 9770 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157578464
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157602661
**[Test build #46169 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46169/consoleFull)**
for PR 9770 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9770#discussion_r45159277
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1063,6 +1065,34 @@ class Analyzer(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157602893
**[Test build #46169 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46169/consoleFull)**
for PR 9770 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157602897
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157604962
**[Test build #46171 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46171/consoleFull)**
for PR 9770 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157602895
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157489181
**[Test build #46090 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46090/consoleFull)**
for PR 9770 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157489316
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157489311
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 user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9770#discussion_r45104293
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1063,6 +1065,29 @@ class Analyzer(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157626798
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157626716
**[Test build #46171 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46171/consoleFull)**
for PR 9770 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9770#issuecomment-157626801
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/9770#discussion_r45166023
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1063,6 +1065,34 @@ class Analyzer(
29 matches
Mail list logo