cloud-fan commented on code in PR #36934:
URL: https://github.com/apache/spark/pull/36934#discussion_r902185170
##
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala:
##
@@ -1272,8 +1272,11 @@ object InferFiltersFromConstraints extends
lsm1 opened a new pull request, #36937:
URL: https://github.com/apache/spark/pull/36937
### What changes were proposed in this pull request?
### Why are the changes needed?
When convert the millis to micro value, it will raise Overflow Error
```scala
ulysses-you opened a new pull request, #36936:
URL: https://github.com/apache/spark/pull/36936
### What changes were proposed in this pull request?
- Add session catalog name in identifiers, then all identifiers will be 3
part name
- Make `CatalogImpl#refreshTable` support
zhengruifeng commented on PR #36926:
URL: https://github.com/apache/spark/pull/36926#issuecomment-1161256434
since it is only exposed in the `.mllib` side, and it is not listed in the
[official documents
Ngone51 commented on PR #36512:
URL: https://github.com/apache/spark/pull/36512#issuecomment-1161238375
Thanks, 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 above to go to the specific
Ngone51 closed pull request #36512: [SPARK-39152][CORE] Deregistering disk
persisted local blocks in case of IO related errors
URL: https://github.com/apache/spark/pull/36512
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
Ngone51 commented on code in PR #36716:
URL: https://github.com/apache/spark/pull/36716#discussion_r902140262
##
docs/job-scheduling.md:
##
@@ -83,6 +83,10 @@ This feature is disabled by default and available on all
coarse-grained cluster
[Mesos coarse-grained
srowen commented on PR #36926:
URL: https://github.com/apache/spark/pull/36926#issuecomment-1161221959
Right, it was still public in the binary code. If there's an easy way to
retain source compatibility, that's great, otherwise OK
--
This is an automated message from the Apache Git
srowen commented on PR #36925:
URL: https://github.com/apache/spark/pull/36925#issuecomment-1161222543
Other things were removed, like the override of union(). May be fine, just
now seems to be worth another look
--
This is an automated message from the Apache Git Service.
To respond to
zhengruifeng commented on PR #36926:
URL: https://github.com/apache/spark/pull/36926#issuecomment-1161214871
@srowen hmm, I just think it is an internal variable, should not be exposed
to end users, like other priavte variables in `MulticlassMetrics` and so on.
However, it had been public
Ngone51 commented on code in PR #36162:
URL: https://github.com/apache/spark/pull/36162#discussion_r902126766
##
core/src/main/scala/org/apache/spark/internal/config/package.scala:
##
@@ -2073,6 +2073,37 @@ package object config {
.timeConf(TimeUnit.MILLISECONDS)
Ngone51 commented on code in PR #36162:
URL: https://github.com/apache/spark/pull/36162#discussion_r902100661
##
core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala:
##
@@ -1218,6 +1248,70 @@ private[spark] class TaskSetManager(
def executorAdded(): Unit = {
zhengruifeng commented on PR #36927:
URL: https://github.com/apache/spark/pull/36927#issuecomment-1161195094
thanks @HyukjinKwon for reviewing
--
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
cloud-fan closed pull request #36641: [SPARK-39263][SQL] Make GetTable,
TableExists and DatabaseExists be compatible with 3 layer namespace
URL: https://github.com/apache/spark/pull/36641
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
cloud-fan commented on PR #36641:
URL: https://github.com/apache/spark/pull/36641#issuecomment-1161188261
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 specific
HyukjinKwon opened a new pull request, #36935:
URL: https://github.com/apache/spark/pull/36935
### What changes were proposed in this pull request?
After SPARK-39521, we don't need such information anymore because they are
all classified under Actions as below:
LuciferYang commented on PR #36925:
URL: https://github.com/apache/spark/pull/36925#issuecomment-1161176292
> Merged to master. It looks like this wasn't a problem in 3.3.x; this
change removed the overrides:
>
> https://issues.apache.org/jira/browse/SPARK-38836 #36121
>
> I
beliefer opened a new pull request, #36934:
URL: https://github.com/apache/spark/pull/36934
### What changes were proposed in this pull request?
Currently, `InferFiltersFromConstraints` construct a new `Filter` which
condition without the same order in scala 2.12 and 2.13
It seems
HyukjinKwon commented on PR #36930:
URL: https://github.com/apache/spark/pull/36930#issuecomment-1161163389
--
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
HyukjinKwon commented on PR #36930:
URL: https://github.com/apache/spark/pull/36930#issuecomment-1161151658
Pushed another fix just now. Mind trying agian?
--
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
HyukjinKwon closed pull request #36931: [DO-NOT-MERGE] Test PR
URL: https://github.com/apache/spark/pull/36931
--
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,
HyukjinKwon closed pull request #36933: [SPARK-39521][INFRA][FOLLOW-UP] Update
step name to "Run / Check changes" to detect the workflow
URL: https://github.com/apache/spark/pull/36933
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
HyukjinKwon commented on PR #36933:
URL: https://github.com/apache/spark/pull/36933#issuecomment-1161147277
Merged to master.
To recover the build.
--
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
HyukjinKwon opened a new pull request, #36933:
URL: https://github.com/apache/spark/pull/36933
### What changes were proposed in this pull request?
This PR changes the step name that's renamed in previous PR.
### Why are the changes needed?
To recover the build.
###
HyukjinKwon closed pull request #36931: [DO-NOT-MERGE] Test PR
URL: https://github.com/apache/spark/pull/36931
--
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,
zinking commented on code in PR #36918:
URL: https://github.com/apache/spark/pull/36918#discussion_r902107022
##
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/BatchScanExec.scala:
##
@@ -55,16 +57,27 @@ case class BatchScanExec(
@transient private
pan3793 commented on PR #36930:
URL: https://github.com/apache/spark/pull/36930#issuecomment-1161120159
Thanks @HyukjinKwon, let me try.
--
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
HyukjinKwon closed pull request #36931: [DO-NOT-MERGE] Test PR
URL: https://github.com/apache/spark/pull/36931
--
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,
HyukjinKwon commented on PR #36930:
URL: https://github.com/apache/spark/pull/36930#issuecomment-1161117081
Just fixed. mind syncing 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
HyukjinKwon commented on PR #36932:
URL: https://github.com/apache/spark/pull/36932#issuecomment-1161116603
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 above to go to the specific
HyukjinKwon closed pull request #36932: [SPARK-39521][INFRA][FOLLOW-UP] Fix
notify workload to detect the main build, and readme link
URL: https://github.com/apache/spark/pull/36932
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
HyukjinKwon opened a new pull request, #36932:
URL: https://github.com/apache/spark/pull/36932
### What changes were proposed in this pull request?
This PR fixes the notify_test_workflow.yml to detect build_main.yml that is
for PR builds.
In addition, this PR fixes the link of
HyukjinKwon commented on PR #36932:
URL: https://github.com/apache/spark/pull/36932#issuecomment-1161116097
I am merging this to recover ASAP.
--
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
pan3793 commented on PR #36930:
URL: https://github.com/apache/spark/pull/36930#issuecomment-1161102852
I think my master branch has been synced. https://github.com/pan3793/spark
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
HyukjinKwon commented on PR #36930:
URL: https://github.com/apache/spark/pull/36930#issuecomment-1161099595
@pan3793 mind rebasing your master branch in your forked repo, to the latest
master branch in Apache Spark?
--
This is an automated message from the Apache Git Service.
To respond
pan3793 opened a new pull request, #36930:
URL: https://github.com/apache/spark/pull/36930
### What changes were proposed in this pull request?
Convert `CaseInsensitiveStringMap#logger` to static, it's safe since the
`org.slf4j.Logger` is thread-safe.
### Why are the
chenjunbiao001 commented on PR #35721:
URL: https://github.com/apache/spark/pull/35721#issuecomment-1160981399
这是来自QQ邮箱的假期自动回复邮件。
您好，邮件已收到，感谢您的来信！
--
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
github-actions[bot] commented on PR #35721:
URL: https://github.com/apache/spark/pull/35721#issuecomment-1160981158
We're closing this PR because it hasn't been updated in a while. This isn't
a judgement on the merit of the PR in any way. It's just a way of keeping the
PR queue manageable.
dongjoon-hyun commented on PR #36922:
URL: https://github.com/apache/spark/pull/36922#issuecomment-1160972770
+1, LGTM!
--
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
HyukjinKwon commented on code in PR #36787:
URL: https://github.com/apache/spark/pull/36787#discussion_r902056954
##
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcQuerySuite.scala:
##
@@ -832,6 +832,18 @@ abstract class OrcQuerySuite extends
HyukjinKwon commented on code in PR #36883:
URL: https://github.com/apache/spark/pull/36883#discussion_r902056893
##
sql/core/src/test/scala/org/apache/spark/sql/GeneratorFunctionSuite.scala:
##
@@ -389,6 +389,31 @@ class GeneratorFunctionSuite extends QueryTest with
HyukjinKwon closed pull request #36927: [SPARK-39534][PS] Series.argmax only
needs single pass
URL: https://github.com/apache/spark/pull/36927
--
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
HyukjinKwon commented on PR #36927:
URL: https://github.com/apache/spark/pull/36927#issuecomment-1160966557
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 above to go to the specific
HyukjinKwon commented on PR #36928:
URL: https://github.com/apache/spark/pull/36928#issuecomment-1160964597
and .. seems like actually what this PR does is similar with
https://github.com/apache/spark/pull/36922 in fact :-).
--
This is an automated message from the Apache Git Service.
To
HyukjinKwon commented on PR #36928:
URL: https://github.com/apache/spark/pull/36928#issuecomment-1160963905
Thanks for taking a look, @EnricoMi . Mind rebasing this please? I think it
conflicted with https://github.com/apache/spark/pull/36922
--
This is an automated message from the
dependabot[bot] commented on PR #36929:
URL: https://github.com/apache/spark/pull/36929#issuecomment-1160961481
OK, I won't notify you again about this release, but will get in touch when
a new version is available. If you'd rather skip all updates until the next
major or minor version,
HyukjinKwon closed pull request #36929: Bump mysql-connector-java from 8.0.27
to 8.0.28
URL: https://github.com/apache/spark/pull/36929
--
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
HyukjinKwon closed pull request #36922: [SPARK-39521][INFRA] Separate scheduled
jobs to each workflow
URL: https://github.com/apache/spark/pull/36922
--
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
HyukjinKwon commented on PR #36922:
URL: https://github.com/apache/spark/pull/36922#issuecomment-1160960656
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 above to go to the specific
uch commented on code in PR #36920:
URL: https://github.com/apache/spark/pull/36920#discussion_r902047346
##
mllib/src/main/scala/org/apache/spark/mllib/evaluation/RankingMetrics.scala:
##
@@ -154,54 +151,68 @@ class RankingMetrics[T: ClassTag] @Since("3.4.0") (
uch commented on code in PR #36920:
URL: https://github.com/apache/spark/pull/36920#discussion_r902047346
##
mllib/src/main/scala/org/apache/spark/mllib/evaluation/RankingMetrics.scala:
##
@@ -154,54 +151,68 @@ class RankingMetrics[T: ClassTag] @Since("3.4.0") (
dependabot[bot] opened a new pull request, #36929:
URL: https://github.com/apache/spark/pull/36929
Bumps [mysql-connector-java](https://github.com/mysql/mysql-connector-j)
from 8.0.27 to 8.0.28.
Changelog
Sourced from
uch commented on code in PR #36920:
URL: https://github.com/apache/spark/pull/36920#discussion_r902047346
##
mllib/src/main/scala/org/apache/spark/mllib/evaluation/RankingMetrics.scala:
##
@@ -154,54 +151,68 @@ class RankingMetrics[T: ClassTag] @Since("3.4.0") (
srowen commented on PR #36499:
URL: https://github.com/apache/spark/pull/36499#issuecomment-1160937525
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 above to go to the specific comment.
To
srowen closed pull request #36499: [SPARK-38846][SQL] Add explicit data mapping
between Teradata Numeric Type and Spark DecimalType
URL: https://github.com/apache/spark/pull/36499
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
HyukjinKwon commented on PR #36922:
URL: https://github.com/apache/spark/pull/36922#issuecomment-1160922013
Guys I'll merge this soon and monitor today (it's morning my time) so I can
follow up and fix some more if there are something going wrong
--
This is an automated message from the
srowen commented on PR #36925:
URL: https://github.com/apache/spark/pull/36925#issuecomment-1160811940
Merged to master. It looks like this wasn't a problem in 3.3.x; this change
removed the overrides:
https://issues.apache.org/jira/browse/SPARK-38836
srowen closed pull request #36925: [SPARK-39520][SQL] Override `--` method for
`ExpressionSet` in Scala 2.13
URL: https://github.com/apache/spark/pull/36925
--
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
gengliangwang closed pull request #36896: [SPARK-39497][SQL] Improve the
analysis exception of missing map key column
URL: https://github.com/apache/spark/pull/36896
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
gengliangwang commented on PR #36896:
URL: https://github.com/apache/spark/pull/36896#issuecomment-1160696817
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 specific
bersprockets commented on code in PR #36883:
URL: https://github.com/apache/spark/pull/36883#discussion_r901871005
##
sql/core/src/test/scala/org/apache/spark/sql/GeneratorFunctionSuite.scala:
##
@@ -389,6 +389,31 @@ class GeneratorFunctionSuite extends QueryTest with
Eugene-Mark commented on code in PR #36499:
URL: https://github.com/apache/spark/pull/36499#discussion_r901863572
##
docs/sql-migration-guide.md:
##
@@ -22,6 +22,11 @@ license: |
* Table of contents
{:toc}
+## Upgrading from Spark SQL 3.3 to 3.4
+
Review Comment:
srowen commented on code in PR #36499:
URL: https://github.com/apache/spark/pull/36499#discussion_r901825209
##
docs/sql-migration-guide.md:
##
@@ -22,6 +22,11 @@ license: |
* Table of contents
{:toc}
+## Upgrading from Spark SQL 3.3 to 3.4
+
Review Comment:
Is this
dongjoon-hyun commented on PR #36923:
URL: https://github.com/apache/spark/pull/36923#issuecomment-1160597253
Thank you, @wangyum !
--
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
Eugene-Mark commented on PR #36499:
URL: https://github.com/apache/spark/pull/36499#issuecomment-1160577823
Document updated, thanks for previous valuable comments!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
cxzl25 commented on code in PR #36787:
URL: https://github.com/apache/spark/pull/36787#discussion_r901783862
##
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcQuerySuite.scala:
##
@@ -832,6 +832,18 @@ abstract class OrcQuerySuite extends OrcQueryTest
EnricoMi commented on PR #36888:
URL: https://github.com/apache/spark/pull/36888#issuecomment-1160553251
@HyukjinKwon here is an improved version:
https://github.com/G-Research/spark/pull/9/files
It is based on https://github.com/apache/spark/pull/36928.
--
This is an automated
xiuzhu9527 commented on code in PR #36784:
URL: https://github.com/apache/spark/pull/36784#discussion_r901761793
##
sql/hive-thriftserver/src/main/java/org/apache/hive/service/auth/LdapAuthenticationProviderImpl.java:
##
@@ -30,21 +35,19 @@ public class
AmplabJenkins commented on PR #36920:
URL: https://github.com/apache/spark/pull/36920#issuecomment-1160520565
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 above to
wangyum commented on PR #36842:
URL: https://github.com/apache/spark/pull/36842#issuecomment-1160511334
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 above to go to the specific comment.
wangyum closed pull request #36842: [SPARK-39445][SQL] Remove the window if
windowExpressions is empty in column pruning
URL: https://github.com/apache/spark/pull/36842
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
LuciferYang commented on code in PR #36925:
URL: https://github.com/apache/spark/pull/36925#discussion_r901720849
##
sql/catalyst/src/main/scala-2.13/org/apache/spark/sql/catalyst/expressions/ExpressionSet.scala:
##
@@ -132,6 +132,12 @@ class ExpressionSet protected(
wangyum commented on code in PR #36925:
URL: https://github.com/apache/spark/pull/36925#discussion_r901709796
##
sql/catalyst/src/main/scala-2.13/org/apache/spark/sql/catalyst/expressions/ExpressionSet.scala:
##
@@ -132,6 +132,12 @@ class ExpressionSet protected(
newSet
wangyum commented on PR #36923:
URL: https://github.com/apache/spark/pull/36923#issuecomment-1160494791
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 above to go to the specific comment.
wangyum closed pull request #36923: [SPARK-39530][SS][TESTS] Fix
`KafkaTestUtils` to support IPv6
URL: https://github.com/apache/spark/pull/36923
--
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
EnricoMi commented on PR #36928:
URL: https://github.com/apache/spark/pull/36928#issuecomment-1160466102
@HyukjinKwon here is some simplification.
--
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
EnricoMi opened a new pull request, #36928:
URL: https://github.com/apache/spark/pull/36928
### What changes were proposed in this pull request?
CI jobs `configure-jobs` and `precondition` can be combined into a single
job, skip logic can be moved into one place.
### Why are the
zhengruifeng opened a new pull request, #36927:
URL: https://github.com/apache/spark/pull/36927
### What changes were proposed in this pull request?
compute `Series.argmax ` with one pass
### Why are the changes needed?
existing implemation of `Series.argmax` needs two pass
LuciferYang commented on PR #36925:
URL: https://github.com/apache/spark/pull/36925#issuecomment-1160395973
run
```
dev/change-scala-version.sh 2.13
mvn clean install -DskipTests -pl sql/catalyst -am -Pscala-2.13
mvn test -pl sql/catalyst -Pscala-2.13
```
All test
zhengruifeng opened a new pull request, #36926:
URL: https://github.com/apache/spark/pull/36926
### What changes were proposed in this pull request?
Remove `scoreLabelsWeight` in `BinaryClassificationMetrics`
### Why are the changes needed?
`scoreLabelsWeight` in
pan3793 commented on code in PR #36784:
URL: https://github.com/apache/spark/pull/36784#discussion_r901603588
##
sql/hive-thriftserver/src/main/java/org/apache/hive/service/auth/LdapAuthenticationProviderImpl.java:
##
@@ -30,21 +35,19 @@ public class
zhengruifeng commented on code in PR #36920:
URL: https://github.com/apache/spark/pull/36920#discussion_r901586489
##
mllib/src/main/scala/org/apache/spark/mllib/evaluation/RankingMetrics.scala:
##
@@ -70,9 +62,14 @@ class RankingMetrics[T: ClassTag] @Since("3.4.0") (
LuciferYang commented on PR #36925:
URL: https://github.com/apache/spark/pull/36925#issuecomment-1160357491
cc @HyukjinKwon
--
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.
LuciferYang opened a new pull request, #36925:
URL: https://github.com/apache/spark/pull/36925
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How was
uch commented on code in PR #36920:
URL: https://github.com/apache/spark/pull/36920#discussion_r901533321
##
mllib/src/main/scala/org/apache/spark/mllib/evaluation/RankingMetrics.scala:
##
@@ -70,9 +62,14 @@ class RankingMetrics[T: ClassTag] @Since("3.4.0") (
EnricoMi commented on code in PR #36924:
URL: https://github.com/apache/spark/pull/36924#discussion_r901524495
##
.github/actions/sync-fork-with-upstream/action.yml:
##
@@ -0,0 +1,19 @@
+name: 'Sync sources'
+author: 'Apache Spark'
+description: 'A composite GitHub Action that
HyukjinKwon commented on PR #36924:
URL: https://github.com/apache/spark/pull/36924#issuecomment-1160286264
uhoh, right. thanks for explanation though :-).
--
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
EnricoMi closed pull request #36924: [SPARK-39532][INFRA] Move checkout and
sync steps into re-usable action
URL: https://github.com/apache/spark/pull/36924
--
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
EnricoMi commented on PR #36924:
URL: https://github.com/apache/spark/pull/36924#issuecomment-1160285430
This is not applicable to Apache Spark CI as it first checksout Spark repo,
then the fork repo, while this action (until merged into upstream repo) only
becomes available after the sync
HyukjinKwon commented on code in PR #36924:
URL: https://github.com/apache/spark/pull/36924#discussion_r901519239
##
.github/actions/sync-fork-with-upstream/action.yml:
##
@@ -0,0 +1,19 @@
+name: 'Sync sources'
+author: 'Apache Spark'
+description: 'A composite GitHub Action
AmplabJenkins commented on PR #36924:
URL: https://github.com/apache/spark/pull/36924#issuecomment-1160283324
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 above to
EnricoMi commented on PR #36924:
URL: https://github.com/apache/spark/pull/36924#issuecomment-1160250439
@HyukjinKwon here is some first simplification of the `build-and-test`
workflow.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
EnricoMi opened a new pull request, #36924:
URL: https://github.com/apache/spark/pull/36924
### What changes were proposed in this pull request?
Steps "Checkout Spark repository" and "Sync the current branch with the
latest in Apache Spark" are defined 9 times in `build-and-test.yml`.
HyukjinKwon commented on PR #36922:
URL: https://github.com/apache/spark/pull/36922#issuecomment-1160206599
Seems working fine as expected.
We can merge this once
https://github.com/HyukjinKwon/spark/actions/runs/2527981545 gets passed.
After this PR, developers should fetch
HyukjinKwon commented on PR #36922:
URL: https://github.com/apache/spark/pull/36922#issuecomment-1160198370
Seems working fine as expected.
We can merge this once
https://github.com/HyukjinKwon/spark/actions/runs/2527942990 gets passed.
After this PR, developers should fetch
HyukjinKwon commented on PR #36922:
URL: https://github.com/apache/spark/pull/36922#issuecomment-1160193622
Seems working fine as expected.
We can merge this once
https://github.com/HyukjinKwon/spark/actions/runs/2527898372 gets passed.
After this PR, developers should fetch
dongjoon-hyun commented on PR #36923:
URL: https://github.com/apache/spark/pull/36923#issuecomment-1160175814
Thank you, @HyukjinKwon !
--
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 commented on PR #36922:
URL: https://github.com/apache/spark/pull/36922#issuecomment-1160152206
Thank you for this refactoring.
--
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
LuciferYang commented on PR #36876:
URL: https://github.com/apache/spark/pull/36876#issuecomment-1160150926
thanks @dongjoon-hyun
--
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
dongjoon-hyun closed pull request #36876: [SPARK-39464][CORE][TESTS][FOLLOWUP]
Use Utils.localHostNameForURI instead of Utils.localCanonicalHostName in tests
URL: https://github.com/apache/spark/pull/36876
--
This is an automated message from the Apache Git Service.
To respond to the
1 - 100 of 124 matches
Mail list logo
