Friede80 commented on PR #15135:
URL: https://github.com/apache/datafusion/pull/15135#issuecomment-2711777758

   I'm not sure if there are still valid uses of `coerce_union_schema` given 
only the set of logical plans, but if we can't change the api of a public 
function, it would be easy enough to accomplish this with a wrapper. i.e.
   ```rust
   pub fn coerce_union_schema(inputs: &[Arc<LogicalPlan>]) -> Result<DFSchema> {
       coerce_union_schema_with_schema(inputs, inputs[0].schema())
   }
   
   pub fn coerce_union_schema_with_schema(inputs: &[Arc<LogicalPlan>], 
base_schema: &DFSchemaRef) -> Result<DFSchema> {
   ...
   }
   ```
   
   
   


-- 
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...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to