Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[8.1.0] Use Single.using instead of onError/onSuccess for async uploads #25251

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

bazel-io
Copy link
Member

Users have reported hangs in Bazel's asynchronous remote cache uploads that may be happening because neither onSuccess nor onError is called on the observer.

Work towards #25232

Closes #25231.

PiperOrigin-RevId: 725235495
Change-Id: I20c3aaa2ee57a52041dea0b3c17356445f2bbc34

Commit d4c9b92

Users have reported hangs in Bazel's asynchronous remote cache uploads that may be happening because neither `onSuccess` nor `onError` is called on the observer.

Work towards bazelbuild#25232

Closes bazelbuild#25231.

PiperOrigin-RevId: 725235495
Change-Id: I20c3aaa2ee57a52041dea0b3c17356445f2bbc34
@bazel-io bazel-io added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Feb 11, 2025
@bazel-io bazel-io requested a review from a team as a code owner February 11, 2025 18:54
@bazel-io bazel-io requested a review from coeuvre February 11, 2025 18:55
@iancha1992 iancha1992 enabled auto-merge February 11, 2025 18:56
@iancha1992 iancha1992 requested a review from Wyverald February 11, 2025 18:57
@iancha1992 iancha1992 added this pull request to the merge queue Feb 11, 2025
Merged via the queue into bazelbuild:release-8.1.0 with commit cf3ebdf Feb 11, 2025
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Feb 11, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
team-Remote-Exec Issues and PRs for the Execution (Remote) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants