itholic commented on code in PR #39543:
URL: https://github.com/apache/spark/pull/39543#discussion_r1083266330
##
core/src/main/resources/error/error-classes.json:
##
@@ -1592,7 +1592,7 @@
},
"MUST_AGGREGATE_CORRELATED_SCALAR_SUBQUERY" : {
"message" : [
-
itholic commented on code in PR #39543:
URL: https://github.com/apache/spark/pull/39543#discussion_r1083266330
##
core/src/main/resources/error/error-classes.json:
##
@@ -1592,7 +1592,7 @@
},
"MUST_AGGREGATE_CORRELATED_SCALAR_SUBQUERY" : {
"message" : [
-
tedyu commented on PR #39654:
URL: https://github.com/apache/spark/pull/39654#issuecomment-1399242161
Test failures were not related to the PR.
https://github.com/tedyu/spark/actions/runs/3973317986/jobs/6811901738#step:9:23488
```
Error: Exception in thread
LuciferYang commented on code in PR #39682:
URL: https://github.com/apache/spark/pull/39682#discussion_r1083274700
##
sql/core/src/main/scala/org/apache/spark/status/protobuf/sql/SQLExecutionUIDataSerializer.scala:
##
@@ -23,17 +23,17 @@ import collection.JavaConverters._
yabola opened a new pull request, #39687:
URL: https://github.com/apache/spark/pull/39687
…uld assume InternalRow implements equals and hashCode
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR
LuciferYang opened a new pull request, #39688:
URL: https://github.com/apache/spark/pull/39688
### What changes were proposed in this pull request?
This pr aims refactor input parameter type of `Utils#setStringField`
function to make maven build pass when sql module use this functions.
LuciferYang commented on PR #39688:
URL: https://github.com/apache/spark/pull/39688#issuecomment-1399230015
GA failed case:
https://github.com/LuciferYang/spark/actions/runs/3973073352/jobs/6811519184
--
This is an automated message from the Apache Git Service.
To respond to the message,
LuciferYang commented on code in PR #39674:
URL: https://github.com/apache/spark/pull/39674#discussion_r1083262519
##
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala:
##
@@ -1005,26 +1005,6 @@ private[spark] class Client(
val tmpDir = new
kuwii commented on PR #39190:
URL: https://github.com/apache/spark/pull/39190#issuecomment-1399216960
Tried the example code in the
[JIRA](https://issues.apache.org/jira/browse/SPARK-24415), and it is not
affected by this change. Tasks showed in the stage are the same before and
after
yabola commented on PR #39687:
URL: https://github.com/apache/spark/pull/39687#issuecomment-1399219285
retest this please
--
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.
dongjoon-hyun opened a new pull request, #39689:
URL: https://github.com/apache/spark/pull/39689
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
###
gengliangwang commented on code in PR #39682:
URL: https://github.com/apache/spark/pull/39682#discussion_r1083262127
##
sql/core/src/main/scala/org/apache/spark/status/protobuf/sql/SQLExecutionUIDataSerializer.scala:
##
@@ -23,17 +23,17 @@ import collection.JavaConverters._
LuciferYang commented on code in PR #39682:
URL: https://github.com/apache/spark/pull/39682#discussion_r1083261688
##
sql/core/src/main/scala/org/apache/spark/status/protobuf/sql/SQLExecutionUIDataSerializer.scala:
##
@@ -23,17 +23,17 @@ import collection.JavaConverters._
LuciferYang commented on code in PR #39683:
URL: https://github.com/apache/spark/pull/39683#discussion_r1083261992
##
core/src/main/protobuf/org/apache/spark/status/protobuf/store_types.proto:
##
@@ -495,9 +495,10 @@ message RDDOperationGraphWrapper {
}
message
LuciferYang commented on code in PR #39682:
URL: https://github.com/apache/spark/pull/39682#discussion_r1083268932
##
sql/core/src/main/scala/org/apache/spark/status/protobuf/sql/SQLExecutionUIDataSerializer.scala:
##
@@ -23,17 +23,17 @@ import collection.JavaConverters._
HyukjinKwon commented on PR #39674:
URL: https://github.com/apache/spark/pull/39674#issuecomment-1399234450
I would defer to either @tgravescs or @mridulm
--
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 commented on PR #39638:
URL: https://github.com/apache/spark/pull/39638#issuecomment-1399243499
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 #39638:
[SPARK-42082][SPARK-41598][PYTHON][CONNECT] Introduce `PySparkValueError` and
`PySparkTypeError`
URL: https://github.com/apache/spark/pull/39638
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
LuciferYang commented on code in PR #39682:
URL: https://github.com/apache/spark/pull/39682#discussion_r1083262236
##
sql/core/src/main/scala/org/apache/spark/status/protobuf/sql/SQLExecutionUIDataSerializer.scala:
##
@@ -23,17 +23,17 @@ import collection.JavaConverters._
LuciferYang commented on code in PR #39682:
URL: https://github.com/apache/spark/pull/39682#discussion_r1083268583
##
sql/core/src/main/scala/org/apache/spark/status/protobuf/sql/SQLExecutionUIDataSerializer.scala:
##
@@ -23,17 +23,17 @@ import collection.JavaConverters._
yabola commented on PR #39687:
URL: https://github.com/apache/spark/pull/39687#issuecomment-1399232731
@sunchao @aokolnychyi Please take a look, thanks!
--
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
LuciferYang commented on code in PR #39682:
URL: https://github.com/apache/spark/pull/39682#discussion_r1083259668
##
sql/core/src/main/scala/org/apache/spark/status/protobuf/sql/SQLPlanMetricSerializer.scala:
##
@@ -19,18 +19,24 @@ package org.apache.spark.status.protobuf.sql
peter-toth commented on PR #39676:
URL: https://github.com/apache/spark/pull/39676#issuecomment-1399207914
Thanks for the quik review!
--
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
itholic commented on code in PR #39638:
URL: https://github.com/apache/spark/pull/39638#discussion_r1083264192
##
python/pyspark/sql/tests/test_functions.py:
##
@@ -763,25 +798,55 @@ def test_higher_order_function_failures(self):
from pyspark.sql.functions import col,
dcoliversun commented on PR #39306:
URL: https://github.com/apache/spark/pull/39306#issuecomment-1399216750
Thank you for the reviews @dongjoon-hyun , I believe I've addressed your
comments! Tomorrow is also the Chinese New Year, I wish you a happy Chinese New
Year.
--
This is an
LuciferYang commented on PR #39674:
URL: https://github.com/apache/spark/pull/39674#issuecomment-1399233117
Updated pr description
--
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
LuciferYang commented on PR #39684:
URL: https://github.com/apache/spark/pull/39684#issuecomment-1399275488
Yeah, this one GA passed
--
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
wangyum commented on PR #39691:
URL: https://github.com/apache/spark/pull/39691#issuecomment-1399281359
In fact databricks also supports this clause.
--
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
LuciferYang opened a new pull request, #39694:
URL: https://github.com/apache/spark/pull/39694
### What changes were proposed in this pull request?
This pr aims change to use `_` instead of `-` in `shadedPattern` for
relocation package name.
### Why are the changes needed?
LuciferYang commented on code in PR #39688:
URL: https://github.com/apache/spark/pull/39688#discussion_r1083302432
##
core/src/main/scala/org/apache/spark/status/protobuf/Utils.scala:
##
@@ -17,16 +17,18 @@
package org.apache.spark.status.protobuf
-import
LuciferYang commented on PR #39679:
URL: https://github.com/apache/spark/pull/39679#issuecomment-1399278365
late 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
wangyum commented on PR #39691:
URL: https://github.com/apache/spark/pull/39691#issuecomment-1399282308
cc @xinrong-meng @MaxGekk @gengliangwang @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
grundprinzip commented on PR #39692:
URL: https://github.com/apache/spark/pull/39692#issuecomment-1399287673
R: @HyukjinKwon @zhengruifeng
--
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
tedyu commented on PR #39654:
URL: https://github.com/apache/spark/pull/39654#issuecomment-1399319090
@srowen @mridulm
Tests passed.
--
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 #39689: [SPARK-42148][K8S][BUILD] Upgrade
`kubernetes-client` to 6.4.0
URL: https://github.com/apache/spark/pull/39689
--
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
dongjoon-hyun closed pull request #39668: [WIP] Test 3.4.0 tagging
URL: https://github.com/apache/spark/pull/39668
--
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
dongjoon-hyun opened a new pull request, #39690:
URL: https://github.com/apache/spark/pull/39690
### What changes were proposed in this pull request?
This PR aims to upgrade `Volcano` from 1.5.1 to 1.7.0.
### Why are the changes needed?
Volcano 1.7.0 finally provides
tedyu commented on PR #39654:
URL: https://github.com/apache/spark/pull/39654#issuecomment-1399319165
@dongjoon-hyun
Do you think this PR is in mergeable state ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
dongjoon-hyun commented on PR #39686:
URL: https://github.com/apache/spark/pull/39686#issuecomment-1399336316
Merged to master, @gengliangwang .
--
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
dongjoon-hyun commented on PR #39690:
URL: https://github.com/apache/spark/pull/39690#issuecomment-1399336416
Since this is a doc-only PR, GitHub action result is irrelevant.
cc @Yikun
--
This is an automated message from the Apache Git Service.
To respond to the message, please
tedyu commented on PR #39654:
URL: https://github.com/apache/spark/pull/39654#issuecomment-1399341276
@dongjoon-hyun @srowen @mridulm
Thanks for reviewing this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
wangyum opened a new pull request, #39691:
URL: https://github.com/apache/spark/pull/39691
### What changes were proposed in this pull request?
The `QUALIFY` clause is used to filter the results of [window
grundprinzip commented on code in PR #39585:
URL: https://github.com/apache/spark/pull/39585#discussion_r1083341487
##
connector/connect/common/src/main/protobuf/spark/connect/expressions.proto:
##
@@ -217,6 +218,28 @@ message Expression {
bool is_user_defined_function =
RyanBerti commented on PR #39678:
URL: https://github.com/apache/spark/pull/39678#issuecomment-1399332706
Hi @dtenedor and @huaxingao
Thanks for the input! I agree with you both that migrating Spark's existing
HLL++ implementation to use the Apache Datasketches library would be
dongjoon-hyun closed pull request #39685: [SPARK-42142][UI] Handle null string
values in CachedQuantile/ExecutorSummary/PoolData
URL: https://github.com/apache/spark/pull/39685
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
dongjoon-hyun commented on PR #39685:
URL: https://github.com/apache/spark/pull/39685#issuecomment-1399336202
Merged to master, @gengliangwang .
--
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
dongjoon-hyun closed pull request #39686: [SPARK-42143][UI] Handle null string
values in RDDStorageInfo/RDDDataDistribution/RDDPartitionInfo
URL: https://github.com/apache/spark/pull/39686
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
grundprinzip opened a new pull request, #39695:
URL: https://github.com/apache/spark/pull/39695
### What changes were proposed in this pull request?
To support retryable errors either produced by Spark directly or an
intermediate proxy, the Spark Connect client can now properly handle
gengliangwang commented on PR #39684:
URL: https://github.com/apache/spark/pull/39684#issuecomment-1399354886
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
gengliangwang commented on PR #39696:
URL: https://github.com/apache/spark/pull/39696#issuecomment-1399364595
cc @LuciferYang @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
dongjoon-hyun commented on PR #39690:
URL: https://github.com/apache/spark/pull/39690#issuecomment-1399364776
Could you review this, @gengliangwang ?
--
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
gengliangwang opened a new pull request, #39696:
URL: https://github.com/apache/spark/pull/39696
### What changes were proposed in this pull request?
Similar to #39666, this PR handles null string values in
PairStrings/RDDOperationNode/RDDOperationClusterWrapper
### Why are the
LuciferYang commented on PR #39683:
URL: https://github.com/apache/spark/pull/39683#issuecomment-1399397574
rebased
--
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
LuciferYang commented on PR #39682:
URL: https://github.com/apache/spark/pull/39682#issuecomment-1399397378
rebased
--
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
mridulm commented on code in PR #39674:
URL: https://github.com/apache/spark/pull/39674#discussion_r1083387253
##
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala:
##
@@ -1005,26 +1005,6 @@ private[spark] class Client(
val tmpDir = new
mridulm closed pull request #39674: [SPARK-42149][YARN] Remove the env
`SPARK_USE_CONC_INCR_GC` used to enable CMS GC for Yarn AM
URL: https://github.com/apache/spark/pull/39674
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
dongjoon-hyun commented on PR #39690:
URL: https://github.com/apache/spark/pull/39690#issuecomment-1399359006
Could you review this when you have some time, @viirya ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
dongjoon-hyun commented on code in PR #39690:
URL: https://github.com/apache/spark/pull/39690#discussion_r1083372057
##
resource-managers/kubernetes/integration-tests/README.md:
##
@@ -364,13 +360,5 @@ You can also specify `volcano` tag to only run Volcano
test:
## Cleanup
gengliangwang commented on code in PR #39690:
URL: https://github.com/apache/spark/pull/39690#discussion_r1083371912
##
resource-managers/kubernetes/integration-tests/README.md:
##
@@ -364,13 +360,5 @@ You can also specify `volcano` tag to only run Volcano
test:
## Cleanup
dongjoon-hyun commented on PR #39690:
URL: https://github.com/apache/spark/pull/39690#issuecomment-1399366517
Thank you so much, @gengliangwang . 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
dongjoon-hyun closed pull request #39690: [SPARK-42150][K8S][DOCS] Upgrade
`Volcano` to 1.7.0
URL: https://github.com/apache/spark/pull/39690
--
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
zhengruifeng commented on code in PR #39585:
URL: https://github.com/apache/spark/pull/39585#discussion_r1083380299
##
python/pyspark/sql/connect/udf.py:
##
@@ -0,0 +1,165 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license
zhengruifeng opened a new pull request, #39699:
URL: https://github.com/apache/spark/pull/39699
### What changes were proposed in this pull request?
Fix incorrect column name in `withField`'s doctest
```
pyspark.sql.connect.column.Column.withField
Failed example:
LuciferYang commented on PR #39688:
URL: https://github.com/apache/spark/pull/39688#issuecomment-1399396145
should we merge this one?I need rebase others
--
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
mridulm commented on PR #39190:
URL: https://github.com/apache/spark/pull/39190#issuecomment-1399397779
Late LGTM.
Thanks for fixing this @kuwii !
Thanks for merging it @srowen :-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
LuciferYang commented on PR #39642:
URL: https://github.com/apache/spark/pull/39642#issuecomment-1399397643
rebased
--
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
vinodkc commented on PR #39449:
URL: https://github.com/apache/spark/pull/39449#issuecomment-1399417725
@dtenedor @srielau , Could please review this PR?
--
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
srowen closed pull request #39190: [SPARK-41683][CORE] Fix issue of getting
incorrect property numActiveStages in jobs API
URL: https://github.com/apache/spark/pull/39190
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
srowen commented on PR #39190:
URL: https://github.com/apache/spark/pull/39190#issuecomment-1399271485
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 commented on PR #39654:
URL: https://github.com/apache/spark/pull/39654#issuecomment-1399271648
Yeah looks fine, just rerun tests
--
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
grundprinzip opened a new pull request, #39692:
URL: https://github.com/apache/spark/pull/39692
### What changes were proposed in this pull request?
This patch allows the planner and command plugins for Spark Connect to
access the Spark Session and let other consumers access the
itholic commented on code in PR #39693:
URL: https://github.com/apache/spark/pull/39693#discussion_r1083316715
##
python/pyspark/errors/exceptions.py:
##
@@ -288,7 +291,57 @@ class UnknownException(CapturedException):
class SparkUpgradeException(CapturedException):
"""
dongjoon-hyun commented on PR #39689:
URL: https://github.com/apache/spark/pull/39689#issuecomment-1399323248
Thank you. 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
gengliangwang commented on PR #39685:
URL: https://github.com/apache/spark/pull/39685#issuecomment-1399358483
@dongjoon-hyun @LuciferYang Thanks for the review!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
gengliangwang commented on PR #39686:
URL: https://github.com/apache/spark/pull/39686#issuecomment-1399358487
@dongjoon-hyun @LuciferYang Thanks for the review!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
vinodkc commented on code in PR #38419:
URL: https://github.com/apache/spark/pull/38419#discussion_r1083371035
##
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathExpressionsSuite.scala:
##
@@ -937,4 +937,135 @@ class MathExpressionsSuite extends
vinodkc commented on code in PR #38419:
URL: https://github.com/apache/spark/pull/38419#discussion_r1083371035
##
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathExpressionsSuite.scala:
##
@@ -937,4 +937,135 @@ class MathExpressionsSuite extends
dongjoon-hyun opened a new pull request, #39697:
URL: https://github.com/apache/spark/pull/39697
…
### What changes were proposed in this pull request?
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
HyukjinKwon commented on PR #39692:
URL: https://github.com/apache/spark/pull/39692#issuecomment-1399391123
These aren't API. Configuration is supposed to be internal, and
SparkConnectPlanner isn't also supposed to be exposed to the end users, and we
don't keep the binary compatibility
HyukjinKwon commented on code in PR #39585:
URL: https://github.com/apache/spark/pull/39585#discussion_r1083382917
##
python/pyspark/sql/connect/udf.py:
##
@@ -0,0 +1,165 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license
gengliangwang commented on PR #39696:
URL: https://github.com/apache/spark/pull/39696#issuecomment-1399399512
@dongjoon-hyun @LuciferYang Thanks for the review.
Merging this one to master
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
gengliangwang closed pull request #39696: [SPARK-42153][UI] Handle null string
values in PairStrings/RDDOperationNode/RDDOperationClusterWrapper
URL: https://github.com/apache/spark/pull/39696
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
gengliangwang closed pull request #39684: [SPARK-42140][CORE] Handle null
string values in ApplicationEnvironmentInfoWrapper/ApplicationInfoWrapper
URL: https://github.com/apache/spark/pull/39684
--
This is an automated message from the Apache Git Service.
To respond to the message, please
xinrong-meng commented on code in PR #39585:
URL: https://github.com/apache/spark/pull/39585#discussion_r1083373417
##
connector/connect/common/src/main/protobuf/spark/connect/expressions.proto:
##
@@ -217,6 +218,28 @@ message Expression {
bool is_user_defined_function =
zhengruifeng closed pull request #39692: [SPARK-41629][CONNECT][FOLLOW] Enable
access to SparkSession from Plugin
URL: https://github.com/apache/spark/pull/39692
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
zhengruifeng commented on PR #39692:
URL: https://github.com/apache/spark/pull/39692#issuecomment-1399382836
merged into 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
zhengruifeng opened a new pull request, #39698:
URL: https://github.com/apache/spark/pull/39698
### What changes were proposed in this pull request?
`array_append` was recently added in SQL and PySpark, this PR adds it to
Connect.
### Why are the changes needed?
For parity
LuciferYang commented on PR #39688:
URL: https://github.com/apache/spark/pull/39688#issuecomment-1399396999
Thanks @gengliangwang @srowen
--
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
gengliangwang commented on PR #39688:
URL: https://github.com/apache/spark/pull/39688#issuecomment-1399396937
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
gengliangwang closed pull request #39688: [SPARK-42146][CORE] Refactor
`Utils#setStringField` to make maven build pass when sql module use this method
URL: https://github.com/apache/spark/pull/39688
--
This is an automated message from the Apache Git Service.
To respond to the message,
mridulm commented on PR #39674:
URL: https://github.com/apache/spark/pull/39674#issuecomment-1399398921
Merged to master.
Thanks for working on this @LuciferYang !
Thanks for the review @tgravescs, and discussion @dongjoon-hyun,
@HyukjinKwon :-)
--
This is an automated message from
srowen commented on code in PR #39688:
URL: https://github.com/apache/spark/pull/39688#discussion_r1083302080
##
core/src/main/scala/org/apache/spark/status/protobuf/Utils.scala:
##
@@ -17,16 +17,18 @@
package org.apache.spark.status.protobuf
-import
itholic opened a new pull request, #39693:
URL: https://github.com/apache/spark/pull/39693
### What changes were proposed in this pull request?
This PR proposes to migrate the Spark Connect errors into PySpark error
framework.
Also introducing 5 exceptions to
dongjoon-hyun commented on PR #39654:
URL: https://github.com/apache/spark/pull/39654#issuecomment-1399336797
I'll leave this to the other committers, @tedyu .
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
srowen commented on PR #39654:
URL: https://github.com/apache/spark/pull/39654#issuecomment-1399340205
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 #39654: [MINOR][SHUFFLE] Include IOException in
warning log of finalizeShuffleMerge
URL: https://github.com/apache/spark/pull/39654
--
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
96 matches
Mail list logo