You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The way exceptions are raised often the only way to distinguish them when catching is to match on the string message. Updating wording of exceptions is a frequent community-driven improvement but has high risk of changing our exception handling anywhere in the system.
User-facing string formatting is decentralized and often would require painstaking string parsing to extract important values if downstream code wanted to render the exception differently. Here is an example of this problem with ansi characters: (Issue) (Originating Exception)
Catching exceptions from warehouse drivers often uses the string matching strategy as well. This leaves our exception handling strategy dependent on warehouse language settings being set to English. Exceptions are not properly handled for other languages. Example for Japanese.
Goals:
near zero risk of modifying exception messages
catching errors works across all warehouse language settings
Possible Solution:
leverage the event subsystem for exceptions as well
This issue has been marked as Stale because it has been open for 180 days with no activity. If you would like the issue to remain open, please comment on the issue or else it will be closed in 7 days.
Although we are closing this issue as stale, it's not gone forever. Issues can be reopened if there is renewed community interest. Just add a comment to notify the maintainers.
Problem:
Goals:
Possible Solution:
Relevant chain of tickets for dbt-core
The text was updated successfully, but these errors were encountered: