gz opened a new issue, #14080:
URL: https://github.com/apache/datafusion/issues/14080

   ### Describe the bug
   
   I'm getting errors in the form of 
DataFusionError::External(External(External(External(External(External(xyz)))) 
when throwing them from 
https://docs.rs/datafusion/latest/datafusion/physical_plan/trait.ExecutionPlan.html#tymethod.execute
   
   ### To Reproduce
   
   Implement a TableProvider that throws an error and call it
   
   ```
       fn execute(
           &self,
           partition: usize,
           _context: Arc<TaskContext>,
       ) -> datafusion::common::Result<SendableRecordBatchStream> {
               return Err(DataFusionError::Execution(
                   format!("error"),
               ));
       }
   ```
   
   
   ### Expected behavior
   
   I'd expect the `External(External(External(External(External` flattened to 
just one `External`
   
   ### Additional context
   
   - This seems to be a regression as it wasn't happening in e.g., 42
   - Similar fixes have been done for other parts of the code 
https://github.com/apache/datafusion/pull/14000
   


-- 
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.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