Re: [PR] implement tree explain for GlobalLimitExec [datafusion]

2025-03-11 Thread via GitHub
alamb merged PR #15100: URL: https://github.com/apache/datafusion/pull/15100 -- 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 GlobalLimitExec [datafusion]

2025-03-10 Thread via GitHub
alamb commented on PR #15100: URL: https://github.com/apache/datafusion/pull/15100#issuecomment-2710170906 I also ran the tests locally and made sure they still passed after a merge from main -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] implement tree explain for GlobalLimitExec [datafusion]

2025-03-09 Thread via GitHub
zjregee commented on code in PR #15100: URL: https://github.com/apache/datafusion/pull/15100#discussion_r1986344402 ## datafusion/physical-plan/src/limit.rs: ## @@ -109,8 +109,12 @@ impl DisplayAs for GlobalLimitExec { ) } DisplayFormat

Re: [PR] implement tree explain for GlobalLimitExec [datafusion]

2025-03-09 Thread via GitHub
alamb commented on code in PR #15100: URL: https://github.com/apache/datafusion/pull/15100#discussion_r1986316154 ## datafusion/physical-plan/src/limit.rs: ## @@ -109,8 +109,12 @@ impl DisplayAs for GlobalLimitExec { ) } DisplayFormatTy

Re: [PR] implement tree explain for GlobalLimitExec [datafusion]

2025-03-08 Thread via GitHub
irenjj commented on code in PR #15100: URL: https://github.com/apache/datafusion/pull/15100#discussion_r1986188763 ## datafusion/physical-plan/src/limit.rs: ## @@ -109,8 +109,12 @@ impl DisplayAs for GlobalLimitExec { ) } DisplayFormatT

[PR] implement tree explain for GlobalLimitExec [datafusion]

2025-03-08 Thread via GitHub
zjregee opened a new pull request, #15100: URL: https://github.com/apache/datafusion/pull/15100 ## Which issue does this PR close? - Closes #15026. ## Rationale for this change ## What changes are included in this PR? Implement tree explain for `GlobalLimitExec`. ## Are these c