Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6537
---
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 enab
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-109321889
Since #6405 and #6505 have already been merged. I'm merging this to master.
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-109270989
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-109270962
[Test build #34272 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34272/console)
for PR 6537 at commit
[`a3bf5fe`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-109252053
[Test build #34272 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34272/consoleFull)
for PR 6537 at commit
[`a3bf5fe`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-109251442
Merged build started.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-109251402
Merged build triggered.
---
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 h
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-108781043
[Test build #34169 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34169/consoleFull)
for PR 6537 at commit
[`6b1b915`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-108781049
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-108780127
[Test build #34169 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34169/consoleFull)
for PR 6537 at commit
[`6b1b915`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-108779906
Merged build started.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-108779873
Merged build triggered.
---
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 h
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107646145
[Test build #33898 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33898/consoleFull)
for PR 6537 at commit
[`188028f`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107646157
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107643240
[Test build #33896 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33896/consoleFull)
for PR 6537 at commit
[`3e06036`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107643250
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107635010
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107634977
[Test build #33893 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33893/consoleFull)
for PR 6537 at commit
[`17c5ea5`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107587807
[Test build #33898 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33898/consoleFull)
for PR 6537 at commit
[`188028f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107586352
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107586402
Merged build started.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107580614
[Test build #33896 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33896/consoleFull)
for PR 6537 at commit
[`3e06036`](https://gith
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31435190
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -216,12 +216,12 @@ trait HiveTypeCoercion {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107578591
Merged build started.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107578446
Merged build triggered.
---
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 h
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31434879
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -127,20 +127,20 @@ trait HiveTypeCoercion {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31434629
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -459,16 +444,16 @@ trait HiveTypeCoercion {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31434578
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -216,12 +216,12 @@ trait HiveTypeCoercion {
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107554274
[Test build #33893 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33893/consoleFull)
for PR 6537 at commit
[`17c5ea5`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107552082
Merged build started.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107551983
Merged build triggered.
---
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 h
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107322383
[Test build #33874 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33874/consoleFull)
for PR 6537 at commit
[`f83d282`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107322393
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 n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107301231
[Test build #33874 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33874/consoleFull)
for PR 6537 at commit
[`f83d282`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107301171
Build triggered.
---
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 thi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107301185
Build started.
---
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 f
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31399508
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -127,20 +127,20 @@ trait HiveTypeCoercion {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31399048
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -127,20 +127,20 @@ trait HiveTypeCoercion {
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31399027
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -127,20 +127,20 @@ trait HiveTypeCoercion {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31393926
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -127,20 +127,20 @@ trait HiveTypeCoercion {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31393883
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -127,20 +127,20 @@ trait HiveTypeCoercion {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31393869
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -127,20 +127,20 @@ trait HiveTypeCoercion {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107163243
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107163216
[Test build #33849 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33849/consoleFull)
for PR 6537 at commit
[`d4c48fb`](https://git
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31389137
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -127,20 +127,20 @@ trait HiveTypeCoercion {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31388873
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -127,20 +127,20 @@ trait HiveTypeCoercion {
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107151991
Ah, just realized that @cloud-fan also adds `BinaryComparison.unapply` in
#6405.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107151736
@yhuai @rxin If this looks good to you, we may merge this after #6405 and
#6505 to avoid unnecessary merging conflicts for @cloud-fan.
---
If your project is set up f
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107151506
[Test build #33849 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33849/consoleFull)
for PR 6537 at commit
[`d4c48fb`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107151417
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6537#issuecomment-107151420
Merged build started.
---
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 user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6537#discussion_r31388729
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -119,7 +119,7 @@ trait HiveTypeCoercion {
GitHub user liancheng opened a pull request:
https://github.com/apache/spark/pull/6537
[SQL] Simplifies binary node pattern matching
This PR is a simpler version of #2764, and adds `unapply` method for the
following binary nodes for simpler pattern matching:
- `BinaryExpres
53 matches
Mail list logo
