Vanja Pejovic

Currently when flagd is not configured, we use `openfeature.NoopProvider`, which always returns the default value. Since this is nil, `config` always returns an error. When flagd is configured, but the...

Don't add lookaside cache entries with invalid sizes

1

If the size doesn't match the expected size, delete it.

After adding more tiers to the gRPC buffer pool (https://github.com/buildbuddy-io/buildbuddy/pull/10873), it's now ~10% faster to use buffer sizes that are slightly smaller than the pool tiers. This allows the whole...

Presize the map when we know how many entries will be in it.

It always returns the same value. This simplifies the code a bit and avoids path joining on every reader and writer call.

I created 2 test functions, one which uses our helpers in disk.go, and another which just uses Go stdlib functions. Both fail most of the time with `bazel test //server/util/disk/vfs_test:vfs_test_test...

Move `min_bytes_auto_zstd_compression` to compression.go

1

This allows a few other places to use it. It's never set in configs, so I renamed it as well.

Enable dynamic execution for GoLink actions

12

This races a local execution against the remote one. In my experiments, the local GoLink is 3x faster. In incremental builds, this often makes the whole build 3x faster. Compare...

Remove an unused leaser.DB call in pebble_cache.Write

1

Benchmark results: ``` │ setbase5 │ setremovelease5 │ │ sec/op │ sec/op vs base │ Set/LocalPebble10/AC-24 9.892µ ± 1% 9.349µ ± 1% -5.49% (p=0.000 n=30) Set/LocalPebble10/CAS-24 8.559µ ± 1% 8.047µ...