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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/06/15 11:27:46 UTC

[GitHub] [arrow] assignUser commented on a diff in pull request #13379: ARROW-16803: [R][CI] Fix caching for R mingw build

assignUser commented on code in PR #13379:
URL: https://github.com/apache/arrow/pull/13379#discussion_r897862214


##########
ci/scripts/ccache_setup.sh:
##########
@@ -23,4 +23,4 @@ echo "ARROW_USE_CCACHE=ON" >> $GITHUB_ENV
 echo "CCACHE_COMPILERCHECK=content" >> $GITHUB_ENV
 echo "CCACHE_COMPRESS=1" >> $GITHUB_ENV
 echo "CCACHE_COMPRESSLEVEL=6" >> $GITHUB_ENV
-echo "CCACHE_MAXSIZE=500M" >> $GITHUB_ENV
+echo "CCACHE_MAXSIZE=2G" >> $GITHUB_ENV

Review Comment:
   We seem to have a special allowance (or its part of Enterprise) as we have 80G of active caches. Regardless of that, we are forced to create a unique key on each run to update the cache due to the shortcomings of `actions/cache` so we will have a bunch of unused caches that will get evicted anyway, which is not ideal but we currently have no better solution. 
   
   This is why I now think that we should replace ccache with sccache not only in crossbow but also in arrow. (given that it  works well, I am already working on getting a test setup in crossbow)



-- 
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.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

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