Re: [PR] Move back schema not matching check and woraround [datafusion]

2025-04-07 Thread via GitHub
LiaCastaneda commented on code in PR #15580: URL: https://github.com/apache/datafusion/pull/15580#discussion_r2031002407 ## datafusion/core/src/physical_planner.rs: ## @@ -2061,6 +2066,35 @@ fn tuple_err(value: (Result, Result)) -> Result<(T, R)> { } } +// Handle the ca

Re: [PR] Move back schema not matching check and woraround [datafusion]

2025-04-07 Thread via GitHub
berkaysynnada commented on code in PR #15580: URL: https://github.com/apache/datafusion/pull/15580#discussion_r2030998452 ## datafusion/core/src/physical_planner.rs: ## @@ -2061,6 +2066,35 @@ fn tuple_err(value: (Result, Result)) -> Result<(T, R)> { } } +// Handle the c

Re: [PR] Move back schema not matching check and woraround [datafusion]

2025-04-07 Thread via GitHub
LiaCastaneda commented on code in PR #15580: URL: https://github.com/apache/datafusion/pull/15580#discussion_r2030945282 ## datafusion/core/src/physical_planner.rs: ## @@ -2061,6 +2066,35 @@ fn tuple_err(value: (Result, Result)) -> Result<(T, R)> { } } +// Handle the ca

Re: [PR] Move back schema not matching check and woraround [datafusion]

2025-04-07 Thread via GitHub
LiaCastaneda commented on code in PR #15580: URL: https://github.com/apache/datafusion/pull/15580#discussion_r2030945282 ## datafusion/core/src/physical_planner.rs: ## @@ -2061,6 +2066,35 @@ fn tuple_err(value: (Result, Result)) -> Result<(T, R)> { } } +// Handle the ca

Re: [PR] Move back schema not matching check and woraround [datafusion]

2025-04-04 Thread via GitHub
LiaCastaneda commented on PR #15580: URL: https://github.com/apache/datafusion/pull/15580#issuecomment-2779448611 Related PRs: - #10088 - adds the check, the workaround for aggregate was changing the physical name with `create_function_physical_name`, before there was also `physical_name

Re: [PR] Move back schema not matching check and woraround [datafusion]

2025-04-04 Thread via GitHub
LiaCastaneda commented on code in PR #15580: URL: https://github.com/apache/datafusion/pull/15580#discussion_r2029242588 ## datafusion/core/src/physical_planner.rs: ## @@ -2061,6 +2066,36 @@ fn tuple_err(value: (Result, Result)) -> Result<(T, R)> { } } +// Handle the ca

[PR] Move back schema not matching check and woraround [datafusion]

2025-04-04 Thread via GitHub
LiaCastaneda opened a new pull request, #15580: URL: https://github.com/apache/datafusion/pull/15580 ## Which issue does this PR close? related to #15439 ## Rationale for this change We want to keep the names mismatch check ## What changes are included in this