Re: [PR] FIX: Remove redundant repartition [datafusion]

2025-04-06 Thread via GitHub
getChan commented on PR #15604: URL: https://github.com/apache/datafusion/pull/15604#issuecomment-2781414127 close. it isn't really redundant. see issue comments -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] FIX: Remove redundant repartition [datafusion]

2025-04-06 Thread via GitHub
getChan closed pull request #15604: FIX: Remove redundant repartition URL: https://github.com/apache/datafusion/pull/15604 -- 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 un

Re: [PR] FIX: Remove redundant repartition [datafusion]

2025-04-06 Thread via GitHub
getChan commented on code in PR #15604: URL: https://github.com/apache/datafusion/pull/15604#discussion_r2030142110 ## datafusion/physical-optimizer/src/enforce_distribution.rs: ## @@ -1258,19 +1259,14 @@ pub fn ensure_distribution( child = add_spm_on_top(ch

Re: [PR] FIX: Remove redundant repartition [datafusion]

2025-04-06 Thread via GitHub
getChan commented on code in PR #15604: URL: https://github.com/apache/datafusion/pull/15604#discussion_r2030128874 ## datafusion/physical-plan/src/repartition/mod.rs: ## @@ -510,7 +510,7 @@ impl DisplayAs for RepartitionExec { writeln!(f, "partitioning_scheme={