[GitHub] spark pull request: [SPARK-14990][SQL] Fix checkForSameTypeInputEx...

2016-05-19 Thread asfgit 
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/13208


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14990][SQL] Fix checkForSameTypeInputEx...

2016-05-19 Thread rxin 
Github user rxin commented on the pull request:

https://github.com/apache/spark/pull/13208#issuecomment-220518940
  
Merging in master/2.0.



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14990][SQL] Fix checkForSameTypeInputEx...

2016-05-19 Thread cloud-fan 
Github user cloud-fan commented on the pull request:

https://github.com/apache/spark/pull/13208#issuecomment-220518610
  
LGTM


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14990][SQL] Fix checkForSameTypeInputEx...

2016-05-19 Thread AmplabJenkins 
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/13208#issuecomment-220507260
  
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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14990][SQL] Fix checkForSameTypeInputEx...

2016-05-19 Thread AmplabJenkins 
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/13208#issuecomment-220507261
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58927/
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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14990][SQL] Fix checkForSameTypeInputEx...

2016-05-19 Thread SparkQA 
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/13208#issuecomment-220507143
  
**[Test build #58927 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58927/consoleFull)**
 for PR 13208 at commit 
[`2ad10ee`](https://github.com/apache/spark/commit/2ad10ee6cd96edf4d7f719ab7cd46a1cafa4be4e).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14990][SQL] Fix checkForSameTypeInputEx...

2016-05-19 Thread SparkQA 
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/13208#issuecomment-220496476
  
**[Test build #58927 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58927/consoleFull)**
 for PR 13208 at commit 
[`2ad10ee`](https://github.com/apache/spark/commit/2ad10ee6cd96edf4d7f719ab7cd46a1cafa4be4e).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14990][SQL] Fix checkForSameTypeInputEx...

2016-05-19 Thread rxin 
Github user rxin commented on the pull request:

https://github.com/apache/spark/pull/13208#issuecomment-220496051
  
cc @yhuai for review


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14990][SQL] Fix checkForSameTypeInputEx...

2016-05-19 Thread rxin 
Github user rxin commented on the pull request:

https://github.com/apache/spark/pull/13208#issuecomment-220496022
  
cc @dosoft 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-14990][SQL] Fix checkForSameTypeInputEx...

2016-05-19 Thread rxin 
GitHub user rxin opened a pull request:

https://github.com/apache/spark/pull/13208

[SPARK-14990][SQL] Fix checkForSameTypeInputExpr (ignore nullability)

## What changes were proposed in this pull request?
This patch fixes a bug in TypeUtils.checkForSameTypeInputExpr. Previously 
the code was testing on strict equality, which does not taking nullability into 
account.

This is based on https://github.com/apache/spark/pull/12768. This patch 
fixed a bug there (with empty expression) and added a test case.

## How was this patch tested?
Added a new test suite and test case.

Closes #12768.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/rxin/spark SPARK-14990

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/13208.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #13208


commit 0e4464cd332c056f2798d31663804144723db26b
Author: Oleg Danilov 
Date:   2016-04-28T22:00:26Z

[SPARK-14990][SQL] nvl, coalesce, array functions with parameter of type 
'array'

commit 7011995fa70e54abeb253615d38fb75886b57f05
Author: Reynold Xin 
Date:   2016-05-20T01:20:52Z

Merge pull request #12768 from dosoft/SPARK-14990

[SPARK-14990][SQL] nvl, coalesce, array with parameter of type 'array'

commit 2ad10ee6cd96edf4d7f719ab7cd46a1cafa4be4e
Author: Reynold Xin 
Date:   2016-05-20T01:29:40Z

[SPARK-14990][SQL] Fix checkForSameTypeInputExpr by ignoring nullability.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org