hundreds-father-404
03/16/2020, 11:32 PM...we should probably just re-enable the pex cache, and trust it to be concurrency safe.We should probably take this to the new channel #performance , but we’ve been discussing this proposal at Toolchain and think it’s likely a good change to re-enable. At a minimum, re-enable when we know it’s local execution. Pex has sane caching afaict (John put a lot of sweat into ensuring that)
witty-crayon-22786
03/16/2020, 11:33 PMcoursier
, for example.hundreds-father-404
03/16/2020, 11:35 PMwitty-crayon-22786
03/16/2020, 11:36 PMre-enable when we know it’s local executioni'm not sure this differentiation is worth it... in the remote case, the cost of writing to the cache should be minimal. and depending on the remoting implementation, it might not be in docker, which would allow it to actually share caches across invokes.
hundreds-father-404
03/16/2020, 11:37 PMwitty-crayon-22786
03/16/2020, 11:38 PMhundreds-father-404
03/16/2020, 11:39 PMwitty-crayon-22786
03/16/2020, 11:39 PMhundreds-father-404
03/16/2020, 11:40 PMwitty-crayon-22786
03/16/2020, 11:41 PMhundreds-father-404
03/16/2020, 11:46 PMenough-analyst-54434
03/16/2020, 11:48 PMwitty-crayon-22786
03/16/2020, 11:49 PMenough-analyst-54434
03/16/2020, 11:50 PMwitty-crayon-22786
03/16/2020, 11:50 PMhundreds-father-404
03/16/2020, 11:50 PMunsafe_local_only_files_because_we_favor_speed_over_correctness_for_this_rule
?enough-analyst-54434
03/16/2020, 11:50 PMwitty-crayon-22786
03/16/2020, 11:50 PMhundreds-breakfast-49010
03/16/2020, 11:51 PMhappy-kitchen-89482
03/17/2020, 5:00 PMwitty-crayon-22786
03/17/2020, 5:01 PMhundreds-father-404
03/17/2020, 5:03 PMwitty-crayon-22786
03/17/2020, 5:04 PMhundreds-father-404
03/17/2020, 5:04 PMwitty-crayon-22786
03/17/2020, 5:04 PM