[I] I think we need to update several map_err statements to propagate inner error messages rather than ignore them. E.g. [datafusion]

[via GitHub]

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

                 I think we need to update several map_err statements to 
propagate inner error messages rather than ignore them. E.g. 
   
   
https://github.com/apache/datafusion/blob/b96186fdef1ff410663ec8fce41186c018f8e09a/datafusion/core/src/datasource/file_format/parquet.rs#L880-L884
   
   change to something like
   
   ```rust
               col_array_channels[next_channel]
                   .send(c)
                   .await
                   .map_err(|e| internal_datafusion_err!("Unable to send array 
to writer due to error {e}"))
   ```
   
   _Originally posted by @devinjdangelo in 
https://github.com/apache/datafusion/pull/11345#discussion_r1672594976_
               


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