[PR] Implement tree explain for CoalescePartitionsExec [datafusion]

2025-03-15 Thread via GitHub
Shreyaskr1409 opened a new pull request, #15225: URL: https://github.com/apache/datafusion/pull/15225 ## Which issue does this PR close? - Closes #15195 . ## Rationale for this change ## What changes are included in this PR? Changes explain_tree.slt and coalesce_pa

Re: [PR] Implement tree explain for CoalescePartitionsExec [datafusion]

2025-03-15 Thread via GitHub
alamb commented on PR #15225: URL: https://github.com/apache/datafusion/pull/15225#issuecomment-2726479559 Thanks again @Shreyaskr1409 ! -- 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] Implement tree explain for CoalescePartitionsExec [datafusion]

2025-03-15 Thread via GitHub
alamb merged PR #15225: URL: https://github.com/apache/datafusion/pull/15225 -- 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: github-unsubscr...@datafusi

Re: [PR] Implement tree explain for CoalescePartitionsExec [datafusion]

2025-03-14 Thread via GitHub
alamb commented on PR #15225: URL: https://github.com/apache/datafusion/pull/15225#issuecomment-2725653649 I took the liberty of applying @Weijun-H 's suggestion and updating the expected output in this PR -- This is an automated message from the Apache Git Service. To respond to the mess

Re: [PR] Implement tree explain for CoalescePartitionsExec [datafusion]

2025-03-14 Thread via GitHub
Weijun-H commented on code in PR #15225: URL: https://github.com/apache/datafusion/pull/15225#discussion_r1995330384 ## datafusion/physical-plan/src/coalesce_partitions.rs: ## @@ -92,10 +92,12 @@ impl DisplayAs for CoalescePartitionsExec { } Non