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

when writing to local disk cache, open files later in order to avoid "too many open files" #13574

Closed
wants to merge 1 commit into from

Conversation

jablin
Copy link
Contributor

@jablin jablin commented Jun 14, 2021

Re-use the existing "LazyFileOutputStream" in DiskAndRemoteCacheClient.java
in order to avoid "Too many open files".

Resolves #13435

Re-use the existing "LazyFileOutputStream" in DiskAndRemoteCacheClient.java
in order to avoid "Too many open files".

Resolves bazelbuild#13435
@google-cla google-cla bot added the cla: yes label Jun 14, 2021
@oquenchil oquenchil added the team-Local-Exec Issues and PRs for the Execution (Local) team label Jun 14, 2021
@meisterT meisterT added team-Remote-Exec Issues and PRs for the Execution (Remote) team and removed team-Local-Exec Issues and PRs for the Execution (Local) team labels Jun 14, 2021
@meisterT meisterT requested a review from coeuvre June 14, 2021 14:00
Copy link
Member

@coeuvre coeuvre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@bazel-io bazel-io closed this in ba5b2a7 Jun 17, 2021
coeuvre pushed a commit to coeuvre/bazel that referenced this pull request Jul 15, 2021
…"too many open files"

Re-use the existing "LazyFileOutputStream" in DiskAndRemoteCacheClient.java
in order to avoid "Too many open files".

Resolves bazelbuild#13435

Closes bazelbuild#13574.

PiperOrigin-RevId: 379892227
coeuvre pushed a commit to coeuvre/bazel that referenced this pull request Jul 15, 2021
…"too many open files"

Re-use the existing "LazyFileOutputStream" in DiskAndRemoteCacheClient.java
in order to avoid "Too many open files".

Resolves bazelbuild#13435

Closes bazelbuild#13574.

PiperOrigin-RevId: 379892227
coeuvre pushed a commit to coeuvre/bazel that referenced this pull request Jul 15, 2021
…"too many open files"

Re-use the existing "LazyFileOutputStream" in DiskAndRemoteCacheClient.java
in order to avoid "Too many open files".

Resolves bazelbuild#13435

Closes bazelbuild#13574.

PiperOrigin-RevId: 379892227
coeuvre pushed a commit to coeuvre/bazel that referenced this pull request Jul 15, 2021
…"too many open files"

Re-use the existing "LazyFileOutputStream" in DiskAndRemoteCacheClient.java
in order to avoid "Too many open files".

Resolves bazelbuild#13435

Closes bazelbuild#13574.

PiperOrigin-RevId: 379892227
coeuvre pushed a commit to coeuvre/bazel that referenced this pull request Jul 16, 2021
…"too many open files"

Re-use the existing "LazyFileOutputStream" in DiskAndRemoteCacheClient.java
in order to avoid "Too many open files".

Resolves bazelbuild#13435

Closes bazelbuild#13574.

PiperOrigin-RevId: 379892227
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes team-Remote-Exec Issues and PRs for the Execution (Remote) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

file handle exhaustion when fetching from remote cache
4 participants