Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Modified the GCS Backend to use transferManager package for Downloads in Parallel#367

Open
gkeesh7 wants to merge 3 commits into
uber:masteruber/kraken:masterfrom
gkeesh7:gcs_backend_transfermanagergkeesh7/kraken:gcs_backend_transfermanagerCopy head branch name to clipboard
Open

Modified the GCS Backend to use transferManager package for Downloads in Parallel#367
gkeesh7 wants to merge 3 commits into
uber:masteruber/kraken:masterfrom
gkeesh7:gcs_backend_transfermanagergkeesh7/kraken:gcs_backend_transfermanagerCopy head branch name to clipboard

Conversation

@gkeesh7

@gkeesh7 gkeesh7 commented Oct 1, 2024

Copy link
Copy Markdown
Collaborator
  • There was an issue with Large Blob Downloads failing because they were exceeding the buffer Guard config in the GCSBackend.
  • This change addressed that by doing the following modifications

Modifications

  1. Added the dependency of Transfer Manager package.
  2. Updated the GCSImpl to have a *transfermanager.Downloaderfield
  3. Modified the GCSImpl download function to make use of the Downloader.

@gkeesh7 gkeesh7 added the bug Something isn't working label Oct 1, 2024
@gkeesh7 gkeesh7 self-assigned this Oct 1, 2024
@gkeesh7
gkeesh7 requested a review from zachcheu October 1, 2024 11:29
@Anton-Kalpakchiev

Copy link
Copy Markdown
Collaborator

Should we wait until we have properly tested this internally before making the change to the external repo? We can then make all our changes with confidence.

On another note, this CICD is extremely flaky, we should fix this soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.