Ricky Stewart

Inefficiency in Java implementation

2

I notice the Java implementation of the `set` method uses a `finally` block to ensure the file object is always closed. So in effect we are opening and closing a...

building with race detection (`@io_bazel_rules_go//go/config:race`) is very slow especially if using Go binaries to generate code

12

Over at Cockroach we are having [this issue](cockroachdb/cockroach#81314) where building a `go_test` with `--@io_bazel_rules_go//go/config:race` takes a very long time. The fact that the stdlib needs to be recompiled with race...

### What version of gazelle are you using? 0.25 ### What version of rules_go are you using? 0.33 (plus a few custom patches) ### What version of Bazel are you...

workflows: run examples-orms tests in GitHub Actions

2

A relevant difference between this and TC CI is I have pinned to a specific commit in the `examples-orms` repo. Epic: [CRDB-8308](https://cockroachlabs.atlassian.net/browse/CRDB-8308) Release note: None

lint: make runnable under `bazel test`

1

This test was never directly runnable under `bazel test`, and instead required using `bazel run` with the wrapper target `//build/bazelutil:lint` that set up some dependencies and some environment variables appropriately....

### What version of rules_go are you using? 0.42 ### What version of gazelle are you using? 0.33 ### What version of Bazel are you using? 6.2.1 ### Does this...

*: evaluate uses of `skip.Stress()`

1

With the RBE-based CI and nightlies, this variable will not be set in CI, which means that `skip.Stress()` will always return `false`. The following uses of it should be evaluated...

To ignore certain files in `nogo`, we need to pass in a regex to match the filename as described [here](https://github.com/bazelbuild/rules_go/blob/master/go/nogo.rst#configuring-analyzers). However, there is no one regex that allows us to...

dev: move local loopback cache line from `~/.bazelrc` to `.bazelrc.user`

2

There hasn't been any particular reason we put it in `~/.bazelrc` before, except that it "doesn't hurt" to have it in the home directory, where you get to take advantage...

dev: also build `ts_project` for `eslint-plugin-crdb` in `gen js`

3

Without this, generation can fail with an error like: ``` Rel: can't make relative to /private/var/tmp/_bazel_davidh/d61affdd093572eb0c41c4220c20a747/execroot/com_github_cockroachdb_cockroach/bazel-out/darwin_arm64-fastbuild/bin/pkg/ui/workspaces/eslint-plugin-crdb/dist ``` It's unclear right now why this is happening. My suspicion is there is...