Skip to content
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

Enable prefix in output S3 files for unload_to_files #729

Merged
merged 2 commits into from
Jun 4, 2021

Conversation

jaidisido
Copy link
Contributor

Issue #, if available:
#717

Description of changes:
The UNLOAD Redshift operation takes "name-prefix" as a parameter, prepending the output S3 objects with a prefix.

This behavior can be enabled for the unload_to_files Wrangler method, but not the unload call. This is because unload makes a subsequent calls to read parquet files from S3 and expects a specific prefix to read from.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jaidisido jaidisido added the enhancement New feature or request label Jun 1, 2021
@jaidisido jaidisido added this to the 2.9.0 milestone Jun 1, 2021
@jaidisido jaidisido requested a review from kukushking June 1, 2021 11:11
@jaidisido jaidisido self-assigned this Jun 1, 2021
@jaidisido
Copy link
Contributor Author

AWS CodeBuild CI Report

  • CodeBuild project: GitHubCodeBuild8756EF16-sDRE8Pq0duHT
  • Commit ID: 6391608
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@jaidisido
Copy link
Contributor Author

AWS CodeBuild CI Report

  • CodeBuild project: GitHubCodeBuild8756EF16-sDRE8Pq0duHT
  • Commit ID: 39dbb55
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@jaidisido jaidisido merged commit 1e22878 into main Jun 4, 2021
@jaidisido jaidisido deleted the enhancement-717-unload-prefix branch June 4, 2021 10:43
@jaidisido jaidisido linked an issue Jun 4, 2021 that may be closed by this pull request
@jaidisido jaidisido added minor release Will be addressed in the next minor release ready to release labels Jun 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request minor release Will be addressed in the next minor release ready to release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow name-prefix when using awswrangler.redshift.unload_to_files
2 participants