Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20374#discussion_r163431340
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaContinuousSourceSuite.scala
---
@@ -91,6 +91,7 @@ class
KafkaCon
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20169#discussion_r163431536
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -82,8 +83,9 @@ import org.apache.spark.util.{CircularBuff
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20169#discussion_r163431452
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -82,8 +83,9 @@ import org.apache.spark.util.{CircularBuff
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20169
@vanzin Could we revert this PR since this is not well reviewed?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apach
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20374
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20374
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/164/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20369
**[Test build #86549 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86549/testReport)**
for PR 20369 at commit
[`d722bbf`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20369
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86549/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20369
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19285
Thanks for your valuable suggestion, the code has been updated.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86557 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86557/testReport)**
for PR 19285 at commit
[`c988762`](https://github.com/apache/spark/commit/c9
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20169#discussion_r163434545
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -82,8 +83,9 @@ import org.apache.spark.util.{CircularBuffer,
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/20374
LGTM. Thanks for catching this!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20169
> Could you please ping me or @cloud-fan when changing the codes in SQL?
I've had really spotty results pinging people. How about you guys make an
effort to monitor changes in SQL if you real
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20169#discussion_r163434724
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -82,8 +83,9 @@ import org.apache.spark.util.{CircularBuffer,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20374
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86556/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20374
**[Test build #86556 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86556/testReport)**
for PR 20374 at commit
[`fb7fa9e`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20374
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20355#discussion_r163434524
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/FileBasedDataSourceSuite.scala ---
@@ -78,4 +78,20 @@ class FileBasedDataSourceSuite extends Quer
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20355#discussion_r163434591
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/FileBasedDataSourceSuite.scala ---
@@ -23,6 +23,7 @@ class FileBasedDataSourceSuite extends Query
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20355
Let's don't forget to fix the PR title to `[SPARK-23148][SQL] ... `. This
style is actually documented to be encouraged in this project contributing
guide.
---
--
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20374
Thank you for review, @jose-torres !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20338
Ah, seems this one doesn't resolve SPARK-11222 too. I think that JIRA
describes to add `pydocstyle` check in our project too given my reading of
@rekhajoshm's issue link. Will double check and t
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20338
@rekhajoshm, mind updating variable names and one nit above? Will merge
this one as soon as they are fixed.
---
-
To unsubsc
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/20373
Thank you for going through and documentation the related changes back to
the cloudpickle changes :)
---
-
To unsubscribe, e-mai
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20373
Whoa, nice efforts! Will take a close look within few days.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.or
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20373#discussion_r163437299
--- Diff: python/pyspark/cloudpickle.py ---
@@ -344,24 +367,21 @@ def save_function(self, obj, name=None):
return self.save_global(obj,
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20373#discussion_r163436769
--- Diff: python/pyspark/cloudpickle.py ---
@@ -237,28 +262,14 @@ def dump(self, obj):
if 'recursion' in e.args[0]:
msg
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20373#discussion_r163436973
--- Diff: python/pyspark/cloudpickle.py ---
@@ -237,28 +262,14 @@ def dump(self, obj):
if 'recursion' in e.args[0]:
msg
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/17702
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17702
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17702
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/165/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17702
**[Test build #86558 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86558/testReport)**
for PR 17702 at commit
[`dc373ae`](https://github.com/apache/spark/commit/dc
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20169
A late LGTM. @vanzin I'm sorry that we sometimes missed your pinging, but I
think it's still good to ping more SQL people for SQL changes, for notification
purpose. A post-hoc review is better tha
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20360
**[Test build #86551 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86551/testReport)**
for PR 20360 at commit
[`74684a7`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20360
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86551/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20360
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20360
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20360
@viirya, mind if I ask to open a backport to branch-2.3?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
F
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20360
---
-
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/20355
**[Test build #86552 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86552/testReport)**
for PR 20355 at commit
[`740def4`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20355
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20355
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86552/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20355
**[Test build #86554 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86554/testReport)**
for PR 20355 at commit
[`1b9420a`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20355
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20355
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86554/
Test FAILed.
---
Github user heary-cao closed the pull request at:
https://github.com/apache/spark/pull/19693
---
-
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 a diff in the pull request:
https://github.com/apache/spark/pull/20368#discussion_r163442656
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala
---
@@ -126,6 +126,22 @@ class BroadcastJoinSuite extends
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20360
@HyukjinKwon Ok. I will open a backport later.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional c
GitHub user heary-cao opened a pull request:
https://github.com/apache/spark/pull/20375
[SPARK-23199][SQL]improved Removes repetition from group expressions in
Aggregate
## What changes were proposed in this pull request?
Currently, all Aggregate operations will go into
Re
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20375
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/20338
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20338
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/166/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20338
**[Test build #86559 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86559/testReport)**
for PR 20338 at commit
[`d10fbb4`](https://github.com/apache/spark/commit/d1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20375
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20169
It might be _optionally_ good to leave cc who are related with the proposal
itself for notification purpose but I don't think it's quite useful to leave cc
to few specific committers as a requir
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20338
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20338
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/167/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20338
**[Test build #86560 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86560/testReport)**
for PR 20338 at commit
[`34a8590`](https://github.com/apache/spark/commit/34
Github user rekhajoshm commented on the issue:
https://github.com/apache/spark/pull/20338
@HyukjinKwon @ueshin was bit hesitant to change variable/references as pep8
is very clear within python circles as python style checker.however after
pondering, have done the needful. please chec
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/20376
[SPARK-23020][CORE][FOLLOWUP] Fix Java style check issues.
## What changes were proposed in this pull request?
This is a follow-up of #20297 which broke lint-java checks.
This pr fixes t
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20376
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/168/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20376
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20376
**[Test build #86561 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86561/testReport)**
for PR 20376 at commit
[`dfd828b`](https://github.com/apache/spark/commit/df
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20224#discussion_r163449646
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -365,6 +388,26 @@ case class WholeStageCodegenExec(ch
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20224#discussion_r163449472
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/WholeStageCodegenSuite.scala
---
@@ -228,4 +229,21 @@ class WholeStageCodegenSuite exten
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20376
LGTM, cc @vanzin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20169
> but I don't think it's quite useful to leave cc to few specific
committers as a requirement
I agree with that, we should not be bound to a few specific committers. But
I think it's goo
Github user rednaxelafx commented on a diff in the pull request:
https://github.com/apache/spark/pull/20224#discussion_r163453639
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/WholeStageCodegenSuite.scala
---
@@ -228,4 +229,21 @@ class WholeStageCodegenSuite ext
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86557 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86557/testReport)**
for PR 19285 at commit
[`c988762`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86557/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19285
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/20377
[SPARK-17088] [FOLLOW-UP] Fix 'sharesHadoopClasses' option when creating
client
## What changes were proposed in this pull request?
This PR is to remove useless `warehouseDir`, which is alr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20377
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/169/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20377
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20377
**[Test build #86562 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86562/testReport)**
for PR 20377 at commit
[`0574ec7`](https://github.com/apache/spark/commit/05
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20169
Let me explain my point here. I am not trying to offend anybody. Spark is
becoming more and more complex. Peer review and test cases are the major
methods we used for quality control. Spark is an
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19285#discussion_r163455326
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -702,6 +645,76 @@ private[spark] class MemoryStore(
}
}
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20377
cc @vanzin @cloud-fan @felixcheung @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additi
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/20355#discussion_r163455727
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/FileBasedDataSourceSuite.scala ---
@@ -78,4 +78,20 @@ class FileBasedDataSourceSuite extends QueryTest
GitHub user rekhajoshm opened a pull request:
https://github.com/apache/spark/pull/20378
[SPARK-11222][Build][Python] Python document style checker added
## What changes were proposed in this pull request?
Using pydocstyle for python document style checker
https://github.com/
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20224#discussion_r163456017
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/WholeStageCodegenSuite.scala
---
@@ -228,4 +229,21 @@ class WholeStageCodegenSuite exten
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20378
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/170/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20378
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20378
**[Test build #86563 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86563/testReport)**
for PR 20378 at commit
[`e69827e`](https://github.com/apache/spark/commit/e6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20378
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86563/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20378
**[Test build #86563 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86563/testReport)**
for PR 20378 at commit
[`e69827e`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20378
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20169
To be clear, I think I personally usually cc related guys. Yup, if I were
fixing this codes, I would have cc'ed you @gatorsmile because I agree that it's
good to do. Sure, more eyes are better.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17702
**[Test build #86558 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86558/testReport)**
for PR 17702 at commit
[`dc373ae`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17702
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86558/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17702
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20355
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/171/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20355
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20355
**[Test build #86564 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86564/testReport)**
for PR 20355 at commit
[`51b0db5`](https://github.com/apache/spark/commit/51
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20368
Hi, All.
This seems to break both `master` and `branch-2.3`. It might be correlated
to the sister PR,
[SPARK-23192](https://github.com/apache/spark/commit/613c290336e382664c24319f
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20368
As a result, this seems to block SparkPullRequestBuilder, too.
-
https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test%20(Dashboard)/job/SparkPullRequestBuilder/86557/
---
--
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20355
Please fix the PR title.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19285
**[Test build #86565 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86565/testReport)**
for PR 19285 at commit
[`f392217`](https://github.com/apache/spark/commit/f3
Github user rednaxelafx commented on a diff in the pull request:
https://github.com/apache/spark/pull/20224#discussion_r163461361
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -365,6 +388,26 @@ case class WholeStageCodegenExec(
401 - 500 of 568 matches
Mail list logo
