Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-23 Thread via GitHub
jackylee-ch commented on PR #44661: URL: https://github.com/apache/spark/pull/44661#issuecomment-1905833521 Thanks for your review. @cloud-fan @beliefer @ulysses-you -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-23 Thread via GitHub
cloud-fan closed pull request #44661: [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces URL: https://github.com/apache/spark/pull/44661 -- 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

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-23 Thread via GitHub
cloud-fan commented on PR #44661: URL: https://github.com/apache/spark/pull/44661#issuecomment-1905495286 thanks, merging to master/3.5! -- 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 specif

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-23 Thread via GitHub
cloud-fan commented on code in PR #44661: URL: https://github.com/apache/spark/pull/44661#discussion_r1462863274 ## sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/CoalesceShufflePartitions.scala: ## @@ -146,13 +147,16 @@ case class CoalesceShufflePartitions(sess

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-21 Thread via GitHub
ulysses-you commented on code in PR #44661: URL: https://github.com/apache/spark/pull/44661#discussion_r1461281654 ## sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/CoalesceShufflePartitions.scala: ## @@ -146,13 +147,16 @@ case class CoalesceShufflePartitions(se

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-20 Thread via GitHub
jackylee-ch commented on code in PR #44661: URL: https://github.com/apache/spark/pull/44661#discussion_r1460718052 ## sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/CoalesceShufflePartitions.scala: ## @@ -146,14 +147,17 @@ case class CoalesceShufflePartitions(se

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-20 Thread via GitHub
cloud-fan commented on code in PR #44661: URL: https://github.com/apache/spark/pull/44661#discussion_r1460428714 ## sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/CoalesceShufflePartitions.scala: ## @@ -146,14 +147,17 @@ case class CoalesceShufflePartitions(sess

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-19 Thread via GitHub
cloud-fan commented on code in PR #44661: URL: https://github.com/apache/spark/pull/44661#discussion_r1458802466 ## sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/ShufflePartitionsUtil.scala: ## @@ -47,9 +47,7 @@ object ShufflePartitionsUtil extends Logging {

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-19 Thread via GitHub
jackylee-ch commented on code in PR #44661: URL: https://github.com/apache/spark/pull/44661#discussion_r1458785050 ## sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/ShufflePartitionsUtil.scala: ## @@ -47,9 +47,7 @@ object ShufflePartitionsUtil extends Logging {

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-19 Thread via GitHub
cloud-fan commented on code in PR #44661: URL: https://github.com/apache/spark/pull/44661#discussion_r1458683923 ## sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/ShufflePartitionsUtil.scala: ## @@ -47,9 +47,7 @@ object ShufflePartitionsUtil extends Logging {

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-18 Thread via GitHub
jackylee-ch commented on code in PR #44661: URL: https://github.com/apache/spark/pull/44661#discussion_r1458389613 ## sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/ShufflePartitionsUtil.scala: ## @@ -47,9 +47,7 @@ object ShufflePartitionsUtil extends Logging {

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-18 Thread via GitHub
cloud-fan commented on code in PR #44661: URL: https://github.com/apache/spark/pull/44661#discussion_r1458353853 ## sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/ShufflePartitionsUtil.scala: ## @@ -47,9 +47,7 @@ object ShufflePartitionsUtil extends Logging {

Re: [PR] [SPARK-46590][SQL] Fix coalesce failed with unexpected partition indeces [spark]

2024-01-18 Thread via GitHub
cloud-fan commented on code in PR #44661: URL: https://github.com/apache/spark/pull/44661#discussion_r1458353228 ## sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/ShufflePartitionsUtil.scala: ## @@ -128,14 +126,18 @@ object ShufflePartitionsUtil extends Logging