Skip to content

Add buckets caching to GcsFS - #424

Closed
meandnano wants to merge 1 commit into
spf13:masterfrom
meandnano:cache-bucket
Closed

Add buckets caching to GcsFS#424
meandnano wants to merge 1 commit into
spf13:masterfrom
meandnano:cache-bucket

Conversation

@meandnano

Copy link
Copy Markdown

Without it each files interaction on GCS results in an unnecessary GetBucket request.

Without it each files interaction on GCS results in an unnecessary
GetBucket request.
@CLAassistant

CLAassistant commented Apr 29, 2024

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@mvrhov

mvrhov commented Mar 11, 2026

Copy link
Copy Markdown

@sagikazarmark can you please find time and review/merge gcsfs merge requests #424, #457, #568. They are pretty small but still significant. We are trying to get rid of various repo forks as they are getting annoying.

@spf13

spf13 commented May 7, 2026

Copy link
Copy Markdown
Owner

Closing — superseded by #568, which is a strict superset: same bucket caching, plus a proper sync.RWMutex protecting the cache map (avoiding the data race your PR would have introduced with concurrent access), plus configurable upload chunk size. Thanks for identifying the missing initialization — that was a real bug.

@spf13 spf13 closed this May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants