SparkQA commented on pull request #32298:
URL: https://github.com/apache/spark/pull/32298#issuecomment-870826390
**[Test build #140386 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140386/testReport)**
for PR 32298 at commit
AmplabJenkins commented on pull request #33100:
URL: https://github.com/apache/spark/pull/33100#issuecomment-870826301
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44896/
--
SparkQA commented on pull request #32286:
URL: https://github.com/apache/spark/pull/32286#issuecomment-870826313
**[Test build #140387 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140387/testReport)**
for PR 32286 at commit
AmplabJenkins commented on pull request #33106:
URL: https://github.com/apache/spark/pull/33106#issuecomment-870826056
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44893/
--
SparkQA commented on pull request #32552:
URL: https://github.com/apache/spark/pull/32552#issuecomment-870825644
**[Test build #140384 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140384/testReport)**
for PR 32552 at commit
SparkQA commented on pull request #32365:
URL: https://github.com/apache/spark/pull/32365#issuecomment-870825574
**[Test build #140385 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140385/testReport)**
for PR 32365 at commit
SparkQA commented on pull request #32753:
URL: https://github.com/apache/spark/pull/32753#issuecomment-870825440
**[Test build #140382 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140382/testReport)**
for PR 32753 at commit
SparkQA commented on pull request #32610:
URL: https://github.com/apache/spark/pull/32610#issuecomment-870825506
**[Test build #140383 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140383/testReport)**
for PR 32610 at commit
SparkQA commented on pull request #32793:
URL: https://github.com/apache/spark/pull/32793#issuecomment-870825358
**[Test build #140380 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140380/testReport)**
for PR 32793 at commit
SparkQA commented on pull request #32787:
URL: https://github.com/apache/spark/pull/32787#issuecomment-870825377
**[Test build #140381 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140381/testReport)**
for PR 32787 at commit
SparkQA commented on pull request #32832:
URL: https://github.com/apache/spark/pull/32832#issuecomment-870825342
**[Test build #140379 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140379/testReport)**
for PR 32832 at commit
SparkQA commented on pull request #32875:
URL: https://github.com/apache/spark/pull/32875#issuecomment-870825234
**[Test build #140377 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140377/testReport)**
for PR 32875 at commit
SparkQA commented on pull request #32902:
URL: https://github.com/apache/spark/pull/32902#issuecomment-870825203
**[Test build #140375 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140375/testReport)**
for PR 32902 at commit
AmplabJenkins removed a comment on pull request #33078:
URL: https://github.com/apache/spark/pull/33078#issuecomment-868924160
Can one of the admins verify this patch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
SparkQA commented on pull request #32850:
URL: https://github.com/apache/spark/pull/32850#issuecomment-870825238
**[Test build #140378 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140378/testReport)**
for PR 32850 at commit
SparkQA commented on pull request #32928:
URL: https://github.com/apache/spark/pull/32928#issuecomment-870825069
**[Test build #140374 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140374/testReport)**
for PR 32928 at commit
SparkQA commented on pull request #32883:
URL: https://github.com/apache/spark/pull/32883#issuecomment-870825166
**[Test build #140376 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140376/testReport)**
for PR 32883 at commit
SparkQA commented on pull request #33078:
URL: https://github.com/apache/spark/pull/33078#issuecomment-870825128
**[Test build #140368 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140368/testReport)**
for PR 33078 at commit
AmplabJenkins commented on pull request #33115:
URL: https://github.com/apache/spark/pull/33115#issuecomment-870824971
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44890/
--
SparkQA commented on pull request #32934:
URL: https://github.com/apache/spark/pull/32934#issuecomment-870824970
**[Test build #140372 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140372/testReport)**
for PR 32934 at commit
SparkQA commented on pull request #32933:
URL: https://github.com/apache/spark/pull/32933#issuecomment-870824903
**[Test build #140373 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140373/testReport)**
for PR 32933 at commit
AmplabJenkins commented on pull request #33117:
URL: https://github.com/apache/spark/pull/33117#issuecomment-870824861
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44888/
--
SparkQA commented on pull request #32943:
URL: https://github.com/apache/spark/pull/32943#issuecomment-870824868
**[Test build #140371 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140371/testReport)**
for PR 32943 at commit
SparkQA commented on pull request #32980:
URL: https://github.com/apache/spark/pull/32980#issuecomment-870824788
**[Test build #140369 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140369/testReport)**
for PR 32980 at commit
AmplabJenkins commented on pull request #33119:
URL: https://github.com/apache/spark/pull/33119#issuecomment-870824723
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44887/
--
SparkQA commented on pull request #32958:
URL: https://github.com/apache/spark/pull/32958#issuecomment-870824743
**[Test build #140370 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140370/testReport)**
for PR 32958 at commit
AmplabJenkins removed a comment on pull request #33101:
URL: https://github.com/apache/spark/pull/33101#issuecomment-869080424
Can one of the admins verify this patch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
AmplabJenkins removed a comment on pull request #33114:
URL: https://github.com/apache/spark/pull/33114#issuecomment-870196000
Can one of the admins verify this patch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
AmplabJenkins removed a comment on pull request #33100:
URL: https://github.com/apache/spark/pull/33100#issuecomment-870824450
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140365/
SparkQA commented on pull request #33091:
URL: https://github.com/apache/spark/pull/33091#issuecomment-870824612
**[Test build #140367 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140367/testReport)**
for PR 33091 at commit
AmplabJenkins commented on pull request #33120:
URL: https://github.com/apache/spark/pull/33120#issuecomment-870824681
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44886/
--
SparkQA commented on pull request #33095:
URL: https://github.com/apache/spark/pull/33095#issuecomment-870824506
**[Test build #140366 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140366/testReport)**
for PR 33095 at commit
SparkQA commented on pull request #33114:
URL: https://github.com/apache/spark/pull/33114#issuecomment-870824396
**[Test build #140362 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140362/testReport)**
for PR 33114 at commit
SparkQA commented on pull request #33101:
URL: https://github.com/apache/spark/pull/33101#issuecomment-870824488
**[Test build #140364 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140364/testReport)**
for PR 33101 at commit
viirya commented on pull request #32867:
URL: https://github.com/apache/spark/pull/32867#issuecomment-870824460
Ur, this might be what I said before, how we make sure the tests are run...
@Yikun Can you check it again?
--
This is an automated message from the Apache Git Service.
AmplabJenkins commented on pull request #33100:
URL: https://github.com/apache/spark/pull/33100#issuecomment-870824450
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140365/
--
This
SparkQA commented on pull request #33105:
URL: https://github.com/apache/spark/pull/33105#issuecomment-870824402
**[Test build #140363 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140363/testReport)**
for PR 33105 at commit
SparkQA commented on pull request #33116:
URL: https://github.com/apache/spark/pull/33116#issuecomment-870824311
**[Test build #140361 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140361/testReport)**
for PR 33116 at commit
SparkQA commented on pull request #33121:
URL: https://github.com/apache/spark/pull/33121#issuecomment-870824270
**[Test build #140360 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140360/testReport)**
for PR 33121 at commit
SparkQA commented on pull request #33136:
URL: https://github.com/apache/spark/pull/33136#issuecomment-870824155
**[Test build #140356 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140356/testReport)**
for PR 33136 at commit
SparkQA commented on pull request #33133:
URL: https://github.com/apache/spark/pull/33133#issuecomment-870824122
**[Test build #140357 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140357/testReport)**
for PR 33133 at commit
SparkQA commented on pull request #33137:
URL: https://github.com/apache/spark/pull/33137#issuecomment-870824055
**[Test build #140355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140355/testReport)**
for PR 33137 at commit
SparkQA commented on pull request #33128:
URL: https://github.com/apache/spark/pull/33128#issuecomment-870824118
**[Test build #140358 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140358/testReport)**
for PR 33128 at commit
AmplabJenkins commented on pull request #33126:
URL: https://github.com/apache/spark/pull/33126#issuecomment-870824143
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140359/
--
This
SparkQA commented on pull request #33138:
URL: https://github.com/apache/spark/pull/33138#issuecomment-870823920
**[Test build #140354 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/140354/testReport)**
for PR 33138 at commit
cloud-fan commented on a change in pull request #30245:
URL: https://github.com/apache/spark/pull/30245#discussion_r660870034
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/EquivalentExpressions.scala
##
@@ -65,11 +65,82 @@ class
dongjoon-hyun commented on pull request #33131:
URL: https://github.com/apache/spark/pull/33131#issuecomment-870816136
Maven build also looks good to me.
```
$ build/mvn package -pl common/unsafe --am
...
[INFO] Spark Project Parent POM ... SUCCESS [
dongjoon-hyun closed pull request #33126:
URL: https://github.com/apache/spark/pull/33126
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
ueshin edited a comment on pull request #32867:
URL: https://github.com/apache/spark/pull/32867#issuecomment-870809700
I'm sorry the the late review, but is this triggering unit tests? I don't
see any test cases running.
- https://github.com/apache/spark/runs/2940354315
```
dongjoon-hyun commented on pull request #33139:
URL: https://github.com/apache/spark/pull/33139#issuecomment-870812081
Thank you for working on this, @xinrong-databricks .
cc @HyukjinKwon
--
This is an automated message from the Apache Git Service.
To respond to the message,
viirya commented on pull request #29326:
URL: https://github.com/apache/spark/pull/29326#issuecomment-870811211
try this again.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific
viirya opened a new pull request #29326:
URL: https://github.com/apache/spark/pull/29326
### What changes were proposed in this pull request?
This PR upgrades Guava to newer 27.0-jre and the dependency version of
Hadoop 3.2 line to 3.2.1.
### Why are the changes
ueshin commented on pull request #32867:
URL: https://github.com/apache/spark/pull/32867#issuecomment-870809700
Is this triggering unit tests? I don't see any test cases running.
https://github.com/apache/spark/runs/2940354315
```
viirya commented on a change in pull request #32980:
URL: https://github.com/apache/spark/pull/32980#discussion_r660853410
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
##
@@ -1049,17 +1103,25 @@ class
xinrong-databricks opened a new pull request #33139:
URL: https://github.com/apache/spark/pull/33139
### What changes were proposed in this pull request?
Add deprecation warning for Python 3.6.
### Why are the changes needed?
According to
otterc commented on a change in pull request #33078:
URL: https://github.com/apache/spark/pull/33078#discussion_r660843950
##
File path:
common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/RemoteBlockPushResolver.java
##
@@ -567,7 +598,8 @@ public void
dongjoon-hyun commented on pull request #33100:
URL: https://github.com/apache/spark/pull/33100#issuecomment-870806007
cc @cloud-fan
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the
dongjoon-hyun closed pull request #33100:
URL: https://github.com/apache/spark/pull/33100
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
dongjoon-hyun commented on pull request #33100:
URL: https://github.com/apache/spark/pull/33100#issuecomment-870804816
Thank you, @wangyum , @HyukjinKwon , @maropu , @huaxingao .
For the above comment
(https://github.com/apache/spark/pull/33100#discussion_r659455547), we can
discuss
viirya commented on a change in pull request #32928:
URL: https://github.com/apache/spark/pull/32928#discussion_r660848165
##
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/RocksDB.scala
##
@@ -0,0 +1,451 @@
+/*
+ * Licensed to the Apache
SaurabhChawla100 commented on pull request #32972:
URL: https://github.com/apache/spark/pull/32972#issuecomment-870803046
> #33040 was merged so this needs to be reworked based off that now
Thanks for sharing the details. Will update the PR as per the new change done
--
This is an
otterc commented on a change in pull request #33078:
URL: https://github.com/apache/spark/pull/33078#discussion_r660843950
##
File path:
common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/RemoteBlockPushResolver.java
##
@@ -567,7 +598,8 @@ public void
ueshin closed pull request #33117:
URL: https://github.com/apache/spark/pull/33117
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
ueshin commented on pull request #33117:
URL: https://github.com/apache/spark/pull/33117#issuecomment-870796934
Thanks! merging to master.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the
viirya commented on a change in pull request #32921:
URL: https://github.com/apache/spark/pull/32921#discussion_r660834539
##
File path:
sql/catalyst/src/main/java/org/apache/spark/sql/connector/read/SupportsDynamicFiltering.java
##
@@ -0,0 +1,55 @@
+/*
+ * Licensed to the
viirya commented on a change in pull request #32921:
URL: https://github.com/apache/spark/pull/32921#discussion_r660834539
##
File path:
sql/catalyst/src/main/java/org/apache/spark/sql/connector/read/SupportsDynamicFiltering.java
##
@@ -0,0 +1,55 @@
+/*
+ * Licensed to the
srowen commented on pull request #33135:
URL: https://github.com/apache/spark/pull/33135#issuecomment-870790464
I just don't know enough about YARN to review this one --
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
srowen commented on pull request #33130:
URL: https://github.com/apache/spark/pull/33130#issuecomment-870789567
I'm probably missing something - we don't have / need Jenkins tests, just
the Github Actions? I just couldn't see test results here, or for the Chill
change.
--
This is an
sunchao commented on a change in pull request #32921:
URL: https://github.com/apache/spark/pull/32921#discussion_r660321387
##
File path:
sql/catalyst/src/main/java/org/apache/spark/sql/connector/read/SupportsRuntimeFiltering.java
##
@@ -0,0 +1,61 @@
+/*
+ * Licensed to the
dongjoon-hyun closed pull request #33130:
URL: https://github.com/apache/spark/pull/33130
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
dongjoon-hyun commented on a change in pull request #33130:
URL: https://github.com/apache/spark/pull/33130#discussion_r660822494
##
File path: pom.xml
##
@@ -2858,6 +2864,18 @@
org.apache.maven.plugins
maven-shade-plugin
3.2.4
+
+
dongjoon-hyun commented on pull request #33130:
URL: https://github.com/apache/spark/pull/33130#issuecomment-870781939
Thank you, @viirya .
Merged to master.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
xuanyuanking commented on a change in pull request #32928:
URL: https://github.com/apache/spark/pull/32928#discussion_r660815271
##
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/RocksDB.scala
##
@@ -0,0 +1,451 @@
+/*
+ * Licensed to the
gengliangwang opened a new pull request #33138:
URL: https://github.com/apache/spark/pull/33138
### What changes were proposed in this pull request?
Add a new ANSI type coercion rule: when getting a date field from a
Timestamp column, cast the column as Date type.
akshatb1 commented on pull request #33135:
URL: https://github.com/apache/spark/pull/33135#issuecomment-870772962
@srowen @HyukjinKwon @tgravescs Could you kindly help in reviewing this PR?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
viirya commented on a change in pull request #33130:
URL: https://github.com/apache/spark/pull/33130#discussion_r660801520
##
File path: pom.xml
##
@@ -2858,6 +2864,18 @@
org.apache.maven.plugins
maven-shade-plugin
3.2.4
+
+
otterc commented on a change in pull request #32140:
URL: https://github.com/apache/spark/pull/32140#discussion_r660751432
##
File path:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
##
@@ -386,40 +415,53 @@ final class
MaxGekk opened a new pull request #33137:
URL: https://github.com/apache/spark/pull/33137
### What changes were proposed in this pull request?
In the PR, I propose to catch all non-fatal exceptions coming
`refreshTable()` at the final stage of table repairing, and output an error
xinrong-databricks commented on a change in pull request #32139:
URL: https://github.com/apache/spark/pull/32139#discussion_r660786103
##
File path: dev/sparktestsupport/modules.py
##
@@ -611,43 +611,47 @@ def __hash__(self):
"pyspark.pandas.spark.utils",
xinrong-databricks commented on pull request #32955:
URL: https://github.com/apache/spark/pull/32955#issuecomment-870743798
Thanks @ueshin! I will file follow-up tickets.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
dongjoon-hyun edited a comment on pull request #33131:
URL: https://github.com/apache/spark/pull/33131#issuecomment-870742689
FYI, in the master branch, I can test the module without this PR. Given
that, `unsafe` module is not broken at least. Which module did you hit the
failure?
```
dongjoon-hyun commented on pull request #33131:
URL: https://github.com/apache/spark/pull/33131#issuecomment-870742689
FYI, in the master branch, I can test the module without this PR.
```
$ build/sbt "unsafe/test"
...
[info] Run completed in 1 second, 56 milliseconds.
[info]
dongjoon-hyun commented on a change in pull request #33131:
URL: https://github.com/apache/spark/pull/33131#discussion_r660777507
##
File path: common/unsafe/pom.xml
##
@@ -56,6 +56,11 @@
chill_${scala.binary.version}
+
+ com.esotericsoftware
+
dongjoon-hyun commented on a change in pull request #33131:
URL: https://github.com/apache/spark/pull/33131#discussion_r660776280
##
File path: pom.xml
##
@@ -3317,9 +3322,9 @@
scala-2.12
-
dongjoon-hyun commented on a change in pull request #33131:
URL: https://github.com/apache/spark/pull/33131#discussion_r660776009
##
File path: core/pom.xml
##
@@ -33,7 +33,7 @@
core
-
+
Review comment:
Let's not touch irrelevant file.
--
This is an
dongjoon-hyun commented on pull request #33131:
URL: https://github.com/apache/spark/pull/33131#issuecomment-870739505
Thank you for making a PR. The GitHub Action works fine. Could you add how
to reproduce the failure into the PR description?
> I found that the build failed when I
otterc commented on a change in pull request #32140:
URL: https://github.com/apache/spark/pull/32140#discussion_r660774462
##
File path:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
##
@@ -767,6 +878,83 @@ final class
dongjoon-hyun commented on pull request #33130:
URL: https://github.com/apache/spark/pull/33130#issuecomment-870736924
Could you review this please, @srowen and @viirya ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
otterc commented on a change in pull request #32140:
URL: https://github.com/apache/spark/pull/32140#discussion_r660771829
##
File path:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
##
@@ -767,6 +878,83 @@ final class
viirya commented on a change in pull request #32980:
URL: https://github.com/apache/spark/pull/32980#discussion_r660768427
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
##
@@ -76,24 +76,34 @@ object ExprCode {
Ngone51 commented on a change in pull request #32140:
URL: https://github.com/apache/spark/pull/32140#discussion_r660766903
##
File path:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
##
@@ -767,6 +878,83 @@ final class
otterc commented on a change in pull request #32140:
URL: https://github.com/apache/spark/pull/32140#discussion_r660751432
##
File path:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
##
@@ -386,40 +415,53 @@ final class
Ngone51 commented on a change in pull request #33034:
URL: https://github.com/apache/spark/pull/33034#discussion_r660750549
##
File path:
common/network-common/src/main/java/org/apache/spark/network/client/TransportClient.java
##
@@ -222,7 +223,7 @@ public void
otterc commented on a change in pull request #33078:
URL: https://github.com/apache/spark/pull/33078#discussion_r660749709
##
File path:
common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/protocol/PushBlockStream.java
##
@@ -99,10 +110,11 @@ public void
viirya commented on a change in pull request #32980:
URL: https://github.com/apache/spark/pull/32980#discussion_r660748186
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
##
@@ -76,24 +76,34 @@ object ExprCode {
AmplabJenkins commented on pull request #33131:
URL: https://github.com/apache/spark/pull/33131#issuecomment-870712325
Can one of the admins verify this patch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
AmplabJenkins commented on pull request #33132:
URL: https://github.com/apache/spark/pull/33132#issuecomment-870712291
Can one of the admins verify this patch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
viirya commented on a change in pull request #32980:
URL: https://github.com/apache/spark/pull/32980#discussion_r660745976
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
##
@@ -1049,17 +1103,25 @@ class
viirya commented on a change in pull request #32980:
URL: https://github.com/apache/spark/pull/32980#discussion_r660744333
##
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
##
@@ -1071,14 +1133,13 @@ class
itholic edited a comment on pull request #33128:
URL: https://github.com/apache/spark/pull/33128#issuecomment-870706328
Thanks, @HyukjinKwon . Just fixed the PR description!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
601 - 700 of 973 matches
Mail list logo