You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@buildstream.apache.org by GitBox <gi...@apache.org> on 2021/03/04 13:39:28 UTC

[GitHub] [buildstream] juergbi commented on pull request #1431: Draft: Add remote cache support

juergbi commented on pull request #1431:
URL: https://github.com/apache/buildstream/pull/1431#issuecomment-790624509


   I've rebased this on top of PR #1450 and changed the cache query strategy. This branch now schedules cache queries as jobs for parallelization only if a remote cache is configured. If only a local cache is used and thus, the job overhead may not be worth it, cache query remains serialized in the main thread. Both cases are handled by `Stream.query_cache()`. This change is in a separate commit and it could be deferred to a follow-up (i.e. until we reduce the overhead to a point where we can enable it unconditionally).
   
   Remote cache tests still need to be added to the test suite / CI.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org