-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
CI: Fix botocore Error #34087
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
CI: Fix botocore Error #34087
Conversation
Hello @alimcmaster1! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2020-05-09 11:48:53 UTC |
also need to catch in test_write_s3_parquet_fails |
https://github.com/pandas-dev/pandas/pull/34087/checks?check_run_id=658833815 Fixed up parquet test too and added boto docs on dynamic errors here |
thanks @alimcmaster1 for the quick response |
* Update Error Msg * Fix botocore error * Add match pattern * black * Add AWS Docs & Fix Parquet Test * Docs * Docs
* Update Error Msg * Fix botocore error * Add match pattern * black * Add AWS Docs & Fix Parquet Test * Docs * Docs
Co-authored-by: alimcmaster1 <[email protected]>
AWS Service Exceptions are not statically defined in boto3. Easiest to just catch a ClientError here.
Ref Docs:
https://boto3.amazonaws.com/v1/documentation/api/latest/guide/error-handling.html#parsing-error-responses-and-catching-exceptions-from-aws-services