-
Notifications
You must be signed in to change notification settings - Fork 179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Handle non-English Snowflake error message for non-existing schemas #840
Conversation
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. CLA has not been signed by users: @pei0804 |
Confirmation of differences in messages. MY_USER#[email protected]>alter USER MY_USER set LANGUAGE = 'ja';
+----------------------------------+
| status |
|----------------------------------|
| Statement executed successfully. |
+----------------------------------+
1 Row(s) produced. Time Elapsed: 0.096s
MY_USER#[email protected]>show terse objects in db.schema;
002043 (02000): SQLコンパイルエラー:
オブジェクトは存在しないか、操作を実行できません。
MY_USER#[email protected]>alter USER MY_USER set LANGUAGE = 'en';
+----------------------------------------+
| status |
|----------------------------------------|
| ステートメントは正常に実行されました。 |
+----------------------------------------+
1 Row(s) produced. Time Elapsed: 0.126s
MY_USER#[email protected]>show terse objects in db.schema;
002043 (02000): SQL compilation error:
Object does not exist, or operation cannot be performed. |
@dbeatty10 |
Co-authored-by: Anders <[email protected]>
As for the other methods, do I fix another PR? |
@dbeatty10 fixed. |
@pei0804 thanks for the change! I'm adding this PR to the queue to be reviewed by an engineer. |
Co-authored-by: Doug Beatty <[email protected]>
This PR has been marked as Stale because it has been open with no activity as of late. If you would like the PR to remain open, please comment on the PR or else it will be closed in 7 days. |
resolves #834
Problem
Handling is failing because the error message is different between Japanese and English.
Solution
Handle the error message regardless of language by using the error code instead.
Checklist