Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22050
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user LuGuangWang closed the pull request at:
https://github.com/apache/spark/pull/22050
---
-
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/22050
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user LuGuangWang commented on the issue:
https://github.com/apache/spark/pull/22050
pull
---
-
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/22040
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user LuGuangWang opened a pull request:
https://github.com/apache/spark/pull/22050
Merge pull request #1 from apache/master
pull code
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
## How was this
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22040
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94462/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22040
**[Test build #94462 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94462/testReport)**
for PR 22040 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94467/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
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/22009
**[Test build #94467 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94467/testReport)**
for PR 22009 at commit
Github user httfighter commented on the issue:
https://github.com/apache/spark/pull/21826
It failed again. I don't know what the problem is. Could you help me
trigger it again?@viirya
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22009#discussion_r208808788
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/ReadSupport.java
---
@@ -0,0 +1,53 @@
+/*
+ * Licensed to the Apache
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22009#discussion_r208808091
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Strategy.scala
---
@@ -76,41 +76,43 @@ object
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22041
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20636
**[Test build #94472 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94472/testReport)**
for PR 20636 at commit
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20636
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/22041
Thanks! merging to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20636
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/22044
**[Test build #94471 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94471/testReport)**
for PR 22044 at commit
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 kiszk commented on the issue:
https://github.com/apache/spark/pull/22044
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user sddyljsx commented on a diff in the pull request:
https://github.com/apache/spark/pull/21859#discussion_r208804032
--- Diff: core/src/main/scala/org/apache/spark/Partitioner.scala ---
@@ -166,7 +169,16 @@ class RangePartitioner[K : Ordering : ClassTag, V](
//
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22049
**[Test build #94470 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94470/testReport)**
for PR 22049 at commit
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 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 sddyljsx commented on a diff in the pull request:
https://github.com/apache/spark/pull/21859#discussion_r208803622
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ShuffleExchangeExec.scala
---
@@ -294,7 +296,12 @@ object ShuffleExchangeExec {
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/94461/
Test FAILed.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/22044
**[Test build #94461 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94461/testReport)**
for PR 22044 at commit
GitHub user maryannxue opened a pull request:
https://github.com/apache/spark/pull/22049
[SPARK-25063][SQL] Rename class KnowNotNull to KnownNotNull
## What changes were proposed in this pull request?
Correct the class name typo checked in through SPARK-24891
##
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22017#discussion_r208803198
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -442,3 +442,184 @@ case class
Github user maryannxue commented on the issue:
https://github.com/apache/spark/pull/22049
@gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22017#discussion_r208803217
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -442,3 +442,184 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22017#discussion_r208803239
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ExpressionEvalHelper.scala
---
@@ -225,7 +264,9 @@ trait
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22017#discussion_r208803179
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -442,3 +442,191 @@ case class
Github user sddyljsx commented on a diff in the pull request:
https://github.com/apache/spark/pull/21859#discussion_r208803055
--- Diff: core/src/main/scala/org/apache/spark/Partitioner.scala ---
@@ -166,7 +169,16 @@ class RangePartitioner[K : Ordering : ClassTag, V](
//
Github user sddyljsx commented on the issue:
https://github.com/apache/spark/pull/21859
This optimization is only for SQL, but other places also use
RangePartitioner. What it can affect other places?
The failed UTs are caused by
```
else if (sampleCacheEnabled
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22009#discussion_r208802631
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/BatchWriteSupportProvider.java
---
@@ -21,33 +21,39 @@
import
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22008#discussion_r208802495
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -158,8 +158,9 @@ abstract class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17586
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17586
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
Thanks @HyukjinKwon @hvanhovell @mgaido91 @kiszk
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21535
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user sddyljsx commented on a diff in the pull request:
https://github.com/apache/spark/pull/21859#discussion_r208801641
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ShuffleExchangeExec.scala
---
@@ -294,7 +296,12 @@ object ShuffleExchangeExec {
Github user sddyljsx commented on a diff in the pull request:
https://github.com/apache/spark/pull/21859#discussion_r208801492
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -2799,6 +2799,26 @@ class SQLQuerySuite extends QueryTest with
Github user sddyljsx commented on a diff in the pull request:
https://github.com/apache/spark/pull/21859#discussion_r208801520
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -2799,6 +2799,26 @@ class SQLQuerySuite extends QueryTest with
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21535
Merged to master.
---
-
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 #94468 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94468/testReport)**
for PR 21859 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21733
**[Test build #94469 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94469/testReport)**
for PR 21733 at commit
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21733
@tdas Addressed review comments. Please take a look again. Thanks in
advance.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22009
**[Test build #94467 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94467/testReport)**
for PR 22009 at commit
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/20636
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/20636
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94465/
Test FAILed.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/20636
**[Test build #94465 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94465/testReport)**
for PR 20636 at commit
Github user codeatri commented on the issue:
https://github.com/apache/spark/pull/22013
@hvanhovell @mn-mikke @mgaido91 Thanks for the review! I have addressed
all your comments and added appropriate test cases for the same.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20637
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94464/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20637
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/20637
**[Test build #94464 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94464/testReport)**
for PR 20637 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22048
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/22048
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/22048
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user xuejianbest opened a pull request:
https://github.com/apache/spark/pull/22048
Fix the show method to display the wide character alignment problem
before:
+---+---+-+
|id |ä¸å½ |s2 |
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21889
I hit the following error in my local environment.
```
sbt.ForkMain$ForkError: org.apache.spark.SparkException: Job aborted due to
stage failure: Task 0 in stage 220.0 failed 1 times,
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22036#discussion_r208795446
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/StatisticsCollectionSuite.scala ---
@@ -204,6 +204,24 @@ class StatisticsCollectionSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22009
**[Test build #94466 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94466/testReport)**
for PR 22009 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94466/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21859
Good point. [These
failures](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94431/testReport/)
may show that it affects other places.
---
Github user habren commented on the issue:
https://github.com/apache/spark/pull/21868
@maropu Thanks for your comments. ORC can also benefit from this change
since ORC is also columnar file format. Do you think I should add ORC support
by change the below line
`
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22009
**[Test build #94466 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94466/testReport)**
for PR 22009 at commit
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/22009
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21977
What about R, do we also need a similar setting for R? I was thinking that
with project hydrogen, more and more external processes will be run inside the
Spark's executor (MPP), all these
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20636
**[Test build #94465 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94465/testReport)**
for PR 20636 at commit
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20636
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21860
btw, we still need the impl. of the vectorized hash map (the comment says
this is for test and benchmark only) in future releases? @hvanhovell @cloud-fan
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22047
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/22047
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94459/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22047
**[Test build #94459 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94459/testReport)**
for PR 22047 at commit
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21860#discussion_r208788502
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/WholeStageCodegenSuite.scala
---
@@ -232,6 +232,25 @@ class WholeStageCodegenSuite extends
Github user habren commented on a diff in the pull request:
https://github.com/apache/spark/pull/22018#discussion_r208788059
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala
---
@@ -297,7 +297,7 @@ object InMemoryFileIndex
Github user habren commented on a diff in the pull request:
https://github.com/apache/spark/pull/22018#discussion_r208787523
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala
---
@@ -297,7 +297,7 @@ object InMemoryFileIndex
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/22034#discussion_r208786793
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -169,6 +171,19 @@ private[spark] class Executor(
Github user markhamstra commented on the issue:
https://github.com/apache/spark/pull/22039
> Yes, "quick hack", but, as opposed to what in these specific cases?
Yes, that is the key question. I'll admit, I haven't looked at all deeply
to try to figure out whether something
Github user LantaoJin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22034#discussion_r208784817
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -169,6 +171,19 @@ private[spark] class Executor(
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21868
Is this a parquet-specific issue? e.g., how about ORC?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user habren commented on a diff in the pull request:
https://github.com/apache/spark/pull/22018#discussion_r208784609
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala
---
@@ -297,7 +297,7 @@ object InMemoryFileIndex
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22018#discussion_r208784598
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala
---
@@ -297,7 +297,7 @@ object InMemoryFileIndex
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22018
Can you narrow down the title and description? I thinks the current one is
obscure..
---
-
To unsubscribe, e-mail:
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22018#discussion_r208783652
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala
---
@@ -297,7 +297,7 @@ object InMemoryFileIndex
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20637
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/20637
**[Test build #94464 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94464/testReport)**
for PR 20637 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20637
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/20636
**[Test build #94463 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94463/testReport)**
for PR 20636 at commit
1 - 100 of 680 matches
Mail list logo