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 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 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 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 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 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 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 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 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 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 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 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 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/20375
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/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/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 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 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 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 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 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 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 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 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 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 #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 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 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 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 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 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 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 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 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 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 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 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 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 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/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 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/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 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 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 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 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 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 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 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 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 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 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 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/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 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 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 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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/20374
**[Test build #86556 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86556/testReport)**
for PR 20374 at commit
[`fb7fa9e`](https://github.com/apache/spark/commit/fb
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/20374
[SPARK-XXX][SS][TEST] Fix
KafkaContinuousSourceStressForDontFailOnDataLossSuite to test
ContinuousExecution
## What changes were proposed in this pull request?
Currently, `KafkaCont
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20169
Will review this more carefully later.
@vanzin Could you please ping me or @cloud-fan when changing the codes in
SQL? I believe we are more familiar about the code base in this area comp
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20169#discussion_r163429951
--- 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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20169#discussion_r163429758
--- 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_r163429626
--- 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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13599
Merged build finished. Test FAILed.
---
-
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/13599
**[Test build #86555 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86555/testReport)**
for PR 13599 at commit
[`5efcb43`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13599
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86555/
Test FAILed.
---
Github user zjffdu commented on a diff in the pull request:
https://github.com/apache/spark/pull/13599#discussion_r163427975
--- Diff: python/pyspark/context.py ---
@@ -1023,6 +1032,42 @@ def getConf(self):
conf.setAll(self._conf.getAll())
return conf
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13599
**[Test build #86555 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86555/testReport)**
for PR 13599 at commit
[`5efcb43`](https://github.com/apache/spark/commit/5e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13599
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/13599
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/163/
Test
Github user glentakahashi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20372#discussion_r163427261
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala
---
@@ -445,16 +445,25 @@ case class FileSourceScanExec(
Github user glentakahashi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20372#discussion_r163426554
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala
---
@@ -445,16 +445,25 @@ case class FileSourceScanExec(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20361
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/20361
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86547/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20361
**[Test build #86547 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86547/testReport)**
for PR 20361 at commit
[`38debd7`](https://github.com/apache/spark/commit/3
Github user zuotingbing commented on the issue:
https://github.com/apache/spark/pull/20025
@gatorsmile @liufengdb Anything else? Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addition
Github user ash211 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20372#discussion_r163419745
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileSourceStrategySuite.scala
---
@@ -142,15 +142,16 @@ class FileSourceStrateg
Github user ash211 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20372#discussion_r163424784
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala
---
@@ -445,16 +445,25 @@ case class FileSourceScanExec(
Github user ash211 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20372#discussion_r163419675
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileSourceStrategySuite.scala
---
@@ -142,15 +142,16 @@ class FileSourceStrateg
Github user ash211 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20372#discussion_r163424415
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala
---
@@ -445,16 +445,25 @@ case class FileSourceScanExec(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20373
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86553/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20373
**[Test build #86553 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86553/testReport)**
for PR 20373 at commit
[`c362df8`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20373
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/17234
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20371
---
-
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 #86554 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86554/testReport)**
for PR 20355 at commit
[`1b9420a`](https://github.com/apache/spark/commit/1b
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/20373
@holdenk @HyukjinKwon it seemed like mostly straightforward fixes/cleanups
to match cloudpickle 0.4.2 but you two are way more experienced here than me.
Are there any concerns over these update
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/162/
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
101 - 200 of 568 matches
Mail list logo