Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21733
**[Test build #94469 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94469/testReport)**
for PR 21733 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21859
**[Test build #94468 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94468/testReport)**
for PR 21859 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22044
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22044
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/21889
> @mallman, while we wait for the go-no-go, do you have the changes for the
next PR ready? Is there anything you need help with?
I have the hack I used originally, but I haven't tried
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21320
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user sddyljsx commented on the issue:
https://github.com/apache/spark/pull/21859
retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21320
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20636
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21847
**[Test build #94478 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94478/testReport)**
for PR 21847 at commit
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/18143
@brkyvz and @zsxwing No comments on whether this will be useful or not, so
far. Should I consider closing it ?
---
-
To
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22029#discussion_r208835030
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
---
@@ -240,20 +242,25 @@ case class In(value:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22029
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22029
**[Test build #94480 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94480/testReport)**
for PR 22029 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22051
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22049
**[Test build #94481 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94481/testReport)**
for PR 22049 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21847
LGTM except one concern about error checking for fixed type. Thanks for
working on it!
---
-
To unsubscribe, e-mail:
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22044#discussion_r208855525
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3410,6 +3410,28 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22044#discussion_r208854615
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3517,56 +3510,24 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22044#discussion_r208856510
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3517,56 +3510,24 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22044#discussion_r208849675
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3410,6 +3410,28 @@ case class
Github user mn-mikke commented on a diff in the pull request:
https://github.com/apache/spark/pull/22017#discussion_r208868838
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ExpressionEvalHelper.scala
---
@@ -225,7 +264,9 @@ trait
Github user mn-mikke commented on the issue:
https://github.com/apache/spark/pull/21121
Sure, closing ...
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mn-mikke closed the pull request at:
https://github.com/apache/spark/pull/21121
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22029
@mgaido91 do you mean the current behavior is same with Hive and Presto?
---
-
To unsubscribe, e-mail:
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22038#discussion_r208833322
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -486,9 +486,17 @@ object TypeCoercion {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22049
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22029
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22029
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22029
**[Test build #94482 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94482/testReport)**
for PR 22029 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/21121
@mn-mikke I think we can close this since we've added `transform` which can
take the index argument as suggested.
---
-
To
GitHub user eatoncys opened a pull request:
https://github.com/apache/spark/pull/22053
[SPARK-25069][Core]Using UnsafeAlignedOffset to make the entire record of 8
byte Items aligned like which is used in UnsafeExternalSorter
## What changes were proposed in this pull request?
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22044
**[Test build #94487 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94487/testReport)**
for PR 22044 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22044
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22044
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user mn-mikke commented on a diff in the pull request:
https://github.com/apache/spark/pull/22017#discussion_r208872928
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -442,3 +442,191 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22044
**[Test build #94487 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94487/testReport)**
for PR 22044 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22044
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user priyankagargnitk opened a pull request:
https://github.com/apache/spark/pull/22054
[SPARK-24703][SQL]: To add support to multiply CalendarInterval with
Integral Type.
## What changes were proposed in this pull request?
This change adds capability to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22044
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22044
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22051
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22029
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22051
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21847#discussion_r208841399
--- Diff:
external/avro/src/main/scala/org/apache/spark/sql/avro/AvroSerializer.scala ---
@@ -87,10 +87,18 @@ class AvroSerializer(rootCatalystType:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22052
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22052
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/21859
Jenkins, retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22053
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22053
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mn-mikke commented on a diff in the pull request:
https://github.com/apache/spark/pull/22017#discussion_r208871779
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -442,3 +442,184 @@ case class
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21505
gentle ping @ssonker
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21320
**[Test build #94476 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94476/testReport)**
for PR 21320 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22049
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22049
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22052
cc @hvanhovell @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22052
**[Test build #94483 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94483/testReport)**
for PR 22052 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22009
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22009
**[Test build #94484 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94484/testReport)**
for PR 22009 at commit
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22017#discussion_r208874195
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -442,3 +442,184 @@ case class
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22017#discussion_r208878022
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -442,3 +442,191 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22054
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22054
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22029
@cloud-fan if we consider only the expression IN with literals, yes, the
behavior is very similar, with the following difference: Presto throws
exception when null is present on both sides.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22044
**[Test build #94488 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94488/testReport)**
for PR 22044 at commit
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22029#discussion_r208835392
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -1476,6 +1476,16 @@ object SQLConf {
"are performed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21847
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94478/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21847
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21847#discussion_r208842200
--- Diff:
external/avro/src/test/scala/org/apache/spark/sql/avro/AvroSuite.scala ---
@@ -725,6 +744,158 @@ class AvroSuite extends QueryTest with
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/22052
[SPARK-25068][SQL] Add exists function.
## What changes were proposed in this pull request?
This pr adds `exists` function which tests whether a predicate holds for
one or more elements in
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22053
**[Test build #94486 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94486/testReport)**
for PR 22053 at commit
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22044#discussion_r208862917
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3410,6 +3410,28 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22044#discussion_r208873100
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3442,17 +3464,15 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22054
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21320
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94476/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21320
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user stanzhai opened a pull request:
https://github.com/apache/spark/pull/22051
[SPARK-25064][WEBUI] Add killed tasks count info to WebUI
## What changes were proposed in this pull request?
Add missing killed tasks to WebUI.
Total tasks = Active + Failed +
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20636
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21859
**[Test build #94485 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94485/testReport)**
for PR 21859 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22044
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94487/
Test FAILed.
---
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/21847#discussion_r208818671
--- Diff:
external/avro/src/test/scala/org/apache/spark/sql/avro/AvroSuite.scala ---
@@ -78,6 +79,25 @@ class AvroSuite extends QueryTest with
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21994#discussion_r20881
--- Diff: pom.xml ---
@@ -2609,6 +2609,28 @@
+
+com.github.spotbugs
+
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21847
**[Test build #94478 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94478/testReport)**
for PR 21847 at commit
Github user praetp commented on a diff in the pull request:
https://github.com/apache/spark/pull/21518#discussion_r208830232
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/UnsafeRowSerializerSuite.scala
---
@@ -45,6 +44,14 @@ class
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22008#discussion_r208831933
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -158,8 +158,9 @@ abstract class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21931#discussion_r208832383
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -1437,6 +1437,16 @@ object SQLConf {
.intConf
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22038
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22038
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22038
@mgaido91 what do you think about it?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21993#discussion_r208822706
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -631,19 +631,26 @@ object ColumnPruning extends
Github user sddyljsx commented on the issue:
https://github.com/apache/spark/pull/21859
@ueshin
please retest it, an unkown error occurred.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22038
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22036#discussion_r208831510
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/StatisticsCollectionSuite.scala ---
@@ -204,6 +204,24 @@ class StatisticsCollectionSuite extends
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22013#discussion_r208815130
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -2117,6 +2117,198 @@ class DataFrameFunctionsSuite extends
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22013#discussion_r208812207
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/HigherOrderFunctionsSuite.scala
---
@@ -230,4 +236,56 @@ class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22013#discussion_r208811169
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -442,3 +442,65 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22038
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22038
**[Test build #94473 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94473/testReport)**
for PR 22038 at commit
1 - 100 of 647 matches
Mail list logo