Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21754#discussion_r207189437
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ExchangeCoordinator.scala
---
@@ -117,10 +118,6 @@ class ExchangeCoordinator(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21943
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21943
**[Test build #93969 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93969/testReport)**
for PR 21943 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21754#discussion_r207188971
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ExchangeCoordinator.scala
---
@@ -83,16 +83,17 @@ import
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21754#discussion_r207188779
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ExchangeCoordinator.scala
---
@@ -83,16 +83,17 @@ import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21915
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21915
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93971/
Test FAILed.
---
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/21952
Hi @dbtsai , nice catch!
I think we can also check the nullability here:
https://github.com/apache/spark/pull/21952/files#diff-01fea32e6ec6bcf6f34d06282e08705aR160
If the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21964
**[Test build #93995 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93995/testReport)**
for PR 21964 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21109
I think range join is a very important feature, but the current
implementation is not robust enough(no spilling to disk). Have you explored
other simpler solutions? I vaguely remember that some
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21915
**[Test build #93971 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93971/testReport)**
for PR 21915 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21754
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93980/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21964
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/21964
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/21754
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21754
**[Test build #93980 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93980/testReport)**
for PR 21754 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21964
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21931
What does the benchmark result suggest? The result should be `1048576` by
default?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21967
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93992/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21967
**[Test build #93992 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93992/testReport)**
for PR 21967 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21967
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/21954
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21954
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93973/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21954
**[Test build #93973 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93973/testReport)**
for PR 21954 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21964
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21963
**[Test build #93994 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93994/testReport)**
for PR 21963 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21964
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93975/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21963
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/21963
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/21964
**[Test build #93975 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93975/testReport)**
for PR 21964 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/21963
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21944#discussion_r207182155
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1367,6 +1367,22 @@ class Dataset[T] private[sql](
}: _*)
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21963
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21963
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93972/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21963
**[Test build #93972 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93972/testReport)**
for PR 21963 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21967
**[Test build #93992 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93992/testReport)**
for PR 21967 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21754
**[Test build #93993 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93993/testReport)**
for PR 21754 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21754
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/21754
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/21967
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/21967
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21967
cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/21967
[SPARK-24598][DOCS] State in the documentation the behavior when arithmetic
operations cause overflow
## What changes were proposed in this pull request?
According to the discussion in
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21754
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21754#discussion_r207178639
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ExchangeCoordinator.scala
---
@@ -83,16 +83,17 @@ import
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #93991 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93991/testReport)**
for PR 21895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21957
**[Test build #93990 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93990/testReport)**
for PR 21957 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
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/21941
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/21957
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/21957
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/21941
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user sddyljsx commented on the issue:
https://github.com/apache/spark/pull/21859
@felixcheung
Thanks for review.
**1. How small is 'small':**
This optimazition works when the sampled data of the RangePartitioner
covers all the data to sort.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21941
**[Test build #93989 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93989/testReport)**
for PR 21941 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/21941
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21599
@cloud-fan I think this is just a wrong behavior (being against SQL
standard is not the only issue), so I am not sure that makes sense to have it
controlled by a config. Anyway, I agree that as it
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21954#discussion_r207172497
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -0,0 +1,212 @@
+/*
+ * Licensed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21754
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93967/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21754
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21754
**[Test build #93967 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93967/testReport)**
for PR 21754 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/21954#discussion_r207171941
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -0,0 +1,212 @@
+/*
+ *
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21954#discussion_r207169967
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -0,0 +1,212 @@
+/*
+ * Licensed
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21599
I don't think we can change behavior like this without introducing a
config. In Spark 3.0 we will add a "strict mode" and closely follow SQL
standard. But for Spark 2.4, how about we keep the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21962
**[Test build #93988 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93988/testReport)**
for PR 21962 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21962
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/21962
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21962
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21754#discussion_r207164799
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ExchangeCoordinator.scala
---
@@ -83,16 +83,17 @@ import
Github user mahmoudmahdi24 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21944#discussion_r207164547
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1367,6 +1367,22 @@ class Dataset[T] private[sql](
}: _*)
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21733
@tdas
I found the spare time to run performance tests though I've run only one
app for now... I couldn't run the tests concurrently. Please let me know if you
are not confident with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21958
**[Test build #93987 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93987/testReport)**
for PR 21958 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21958
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/21958
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21954#discussion_r207162371
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -0,0 +1,212 @@
+/*
+ * Licensed
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21954#discussion_r207162167
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -0,0 +1,212 @@
+/*
+ * Licensed
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21958
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/21966
**[Test build #93986 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93986/testReport)**
for PR 21966 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21966
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/21966
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21944#discussion_r207160479
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1367,6 +1367,22 @@ class Dataset[T] private[sql](
}: _*)
}
Github user mahmoudmahdi24 commented on the issue:
https://github.com/apache/spark/pull/21961
Hello @Satyajitv, please rename the title of this PR properly.
The PR title should be of the form [SPARK-][COMPONENT] Title, where
SPARK- is the relevant JIRA number, COMPONENT
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21966
cc @kiszk @ueshin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/21966
simplify codegen of array_except
## What changes were proposed in this pull request?
simplify the codegen:
1. only do real codegen if the type can be specialized by the hash set
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21935#discussion_r207159637
--- Diff:
external/avro/src/test/scala/org/apache/spark/sql/avro/AvroSuite.scala ---
@@ -41,6 +41,7 @@ import org.apache.spark.sql.types._
class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/21954#discussion_r207158636
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -0,0 +1,212 @@
+/*
+ *
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21935
**[Test build #93985 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93985/testReport)**
for PR 21935 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21935
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/21935
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/21958
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93976/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21958
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21958
**[Test build #93976 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93976/testReport)**
for PR 21958 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21941
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93966/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21941
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21941
**[Test build #93966 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93966/testReport)**
for PR 21941 at commit
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/21935
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mahmoudmahdi24 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21944#discussion_r207156078
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -1367,6 +1367,22 @@ class Dataset[T] private[sql](
}: _*)
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21957#discussion_r207155086
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -436,8 +436,9 @@ object DataSourceStrategy {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21087
**[Test build #93984 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93984/testReport)**
for PR 21087 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21965
**[Test build #93983 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93983/testReport)**
for PR 21965 at commit
901 - 1000 of 1238 matches
Mail list logo