alamb commented on issue #7522: URL: https://github.com/apache/arrow-datafusion/issues/7522#issuecomment-1714265299
cc @comphead > I think there are two paths forward: > do NOT generate backtraces for errors > do NOT use errors for the happy path but rather Option or some other enum I agree with this assessment. What I propose we do is: 1. revert https://github.com/apache/arrow-datafusion/pull/7434 while we sort things out 2. Try and work on avoiding errors on the happy path (https://github.com/apache/arrow-datafusion/issues/5309 is related) which is a somewhat unfortunate pattern (as each error requires at least one string allocation!) among other things -- 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]
