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