Re: [PR] Refactor error handling to use boxed errors for DataFusionError variants [datafusion]

2025-07-04 Thread via GitHub 


kosiew merged PR #16672:
URL: https://github.com/apache/datafusion/pull/16672


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Re: [PR] Refactor error handling to use boxed errors for DataFusionError variants [datafusion]

2025-07-03 Thread via GitHub 


kosiew commented on PR #16672:
URL: https://github.com/apache/datafusion/pull/16672#issuecomment-3034678043

   @alamb 
   
   > Adding a test to ensure the size of DataFusionError does not get larger 
(similar to the one added by @crepererum in #16653 )
   > Add a note in the upgrade guide (in #16673)
   
   Added test and upgrade note.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Re: [PR] Refactor error handling to use boxed errors for DataFusionError variants [datafusion]

2025-07-03 Thread via GitHub 


comphead commented on PR #16672:
URL: https://github.com/apache/datafusion/pull/16672#issuecomment-3032828771

   cc @crepererum
   There is another PR pending for `SchemaError` 
https://github.com/apache/datafusion/pull/16653


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]