Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-117049782
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-117049845
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-117050175
[Test build #36116 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36116/consoleFull)
for PR 6982 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-117094255
[Test build #36116 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36116/console)
for PR 6982 at commit
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-117218256
LGTM, merging this into master, thanks!
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/6982
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-117094764
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116558337
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116590951
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116652861
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33461668
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala
---
@@ -245,13 +254,26 @@ abstract class UnaryExpression
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116654128
[Test build #35984 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35984/consoleFull)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116660971
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116661524
[Test build #35987 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35987/consoleFull)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116652902
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116661050
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116591859
[Test build #35979 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35979/consoleFull)
for PR 6982 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116621677
[Test build #35979 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35979/console)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116621696
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116731548
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/6982#issuecomment-116731509
[Test build #35987 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35987/console)
for PR 6982 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116720487
[Test build #35984 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35984/console)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116720529
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 davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33496840
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ExtractValue.scala
---
@@ -195,6 +240,20 @@ case class GetArrayItem(child:
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116794010
@cloud-fan This PR looks pretty good to me. Since we're replying on
`StructField.nullable` to skip nullablity check in codegen, that could be kind
of risk, because we
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116463015
[Test build #35959 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35959/console)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116463130
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116508250
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116508214
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116515551
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/6982#issuecomment-116515541
[Test build #35973 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35973/console)
for PR 6982 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116509482
[Test build #35973 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35973/consoleFull)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116510065
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116510108
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116511277
[Test build #35974 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35974/consoleFull)
for PR 6982 at commit
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33436295
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ExtractValue.scala
---
@@ -195,6 +205,11 @@ case class GetArrayItem(child:
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116518107
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/6982#issuecomment-116518075
[Test build #35974 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35974/console)
for PR 6982 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33536204
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ExtractValue.scala
---
@@ -209,4 +257,16 @@ case class GetMapValue(child:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-11694
[Test build #36073 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36073/consoleFull)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116921630
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116921687
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116919313
[Test build #36071 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36071/consoleFull)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-11697
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116918599
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116918501
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
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33536380
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ExtractValue.scala
---
@@ -209,4 +257,16 @@ case class GetMapValue(child:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116933327
[Test build #36071 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36071/console)
for PR 6982 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116954993
[Test build #36073 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36073/console)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116955036
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 davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33427443
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/BoundAttribute.scala
---
@@ -47,7 +47,7 @@ case class BoundReference(ordinal:
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33427457
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -127,23 +127,24 @@ class CodeGenContext {
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33427538
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/BoundAttribute.scala
---
@@ -47,7 +47,7 @@ case class BoundReference(ordinal:
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116424813
Hi @davies , I have updated this PR, could you review it again? Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116425955
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33432796
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ExtractValue.scala
---
@@ -195,6 +205,10 @@ case class GetArrayItem(child:
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116425990
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116426996
[Test build #35953 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35953/consoleFull)
for PR 6982 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116427377
[Test build #35953 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35953/console)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116427384
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33435932
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ExtractValue.scala
---
@@ -195,6 +205,11 @@ case class GetArrayItem(child:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116452096
[Test build #35959 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35959/consoleFull)
for PR 6982 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33435789
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ExtractValue.scala
---
@@ -195,6 +205,11 @@ case class GetArrayItem(child:
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33434908
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ExtractValue.scala
---
@@ -195,6 +205,10 @@ case class GetArrayItem(child:
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6982#discussion_r33434873
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ComplexTypeSuite.scala
---
@@ -40,51 +40,36 @@ class ComplexTypeSuite extends
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116451177
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-116451242
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-114809447
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/6982#issuecomment-114809444
[Test build #35669 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35669/console)
for PR 6982 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-114808928
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-114808976
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6982#issuecomment-114809181
[Test build #35669 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35669/consoleFull)
for PR 6982 at commit
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/6982
[SPARK-8590][SQL] add code gen for ExtractValue
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/cloud-fan/spark extract-value
Alternatively
73 matches
Mail list logo