alamb opened a new pull request, #4572: URL: https://github.com/apache/arrow-datafusion/pull/4572
# Which issue does this PR close? Closes https://github.com/apache/arrow-datafusion/issues/4571 # Rationale for this change I think #4530 made error messages look much crappier. For example: ``` Table Bare \{ table: "user" \} doesn't exist. ``` I noticed this while adding errors in https://github.com/apache/arrow-datafusion/pull/4535 I would like the messages to only have the table name (not the Debug version of OwnedTableReference). # What changes are included in this PR? 1. Improve error messages to not have Bare 2. Update tests 2. Drive by improvement to change an InternalError to an execution error # Are these changes tested? Yes # Are there any user-facing changes? Better error messages -- 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]
