Skip to content

Refactor GrpcRemoteCache to use uploadFile and uploadBlob.#9039

Closed
buchgr wants to merge 1 commit intobazelbuild:masterfrom
buchgr:refactor-upload
Closed

Refactor GrpcRemoteCache to use uploadFile and uploadBlob.#9039
buchgr wants to merge 1 commit intobazelbuild:masterfrom
buchgr:refactor-upload

Conversation

@buchgr
Copy link
Copy Markdown
Contributor

@buchgr buchgr commented Aug 1, 2019

The methods have the same signature as the ones in SimpleBlobStore. This gets us one step closer to merging AbstractRemoteActionCache and SimpleBlobStoreActionCache.

@buchgr
Copy link
Copy Markdown
Contributor Author

buchgr commented Aug 1, 2019

@borkaehw

@borkaehw
Copy link
Copy Markdown

borkaehw commented Aug 1, 2019

I like the plan we discuss here #9028 (comment), just wondering are all the works going to happen in this single PR?

Copy link
Copy Markdown
Contributor

@ishikhman ishikhman left a comment

Choose a reason for hiding this comment

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

This one will be based on #9028, right?
Other than comments in #9028, LGTM

@irengrig irengrig added the WIP label Aug 2, 2019
The methods have the same signature as the ones in SimpleBlobStore.
This gets us one step closer to merging AbstractRemoteActionCache
and SimpleBlobStoreActionCache.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants