-
Notifications
You must be signed in to change notification settings - Fork 699
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
Add Athena UNLOAD #1038
Add Athena UNLOAD #1038
Conversation
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
ae8192a
to
7ed9227
Compare
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
7ed9227
to
8f88eb8
Compare
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Hi! Really excited this is getting worked on. Would it make sense to change the error messages here to suggest either a ctas or unload approach? https:/awslabs/aws-data-wrangler/blob/42b46f67c3e651d6a9847d216a0257fca4eabe7c/awswrangler/athena/_utils.py#L211 |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Thanks @lucasmo - corrected the error message. Feel free to try out the unload by:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great overall, just a few nitpicks. Excited about this one, thank you!
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
*Issue #1015 *
Description of changes:
Add UNLOAD
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.