irfan sharif

Results 26 comments of irfan sharif

I get that whitelisting this .ccache path works, but this is still pretty surprising behaviour to have by default. We're trying to roll-out bazel usage for engineers at [cockroachdb/cockroach](https://github.com/cockroachdb/cockroach), and...

The workarounds I've seen elsewhere (https://github.com/RobotLocomotion/drake/issues/4464, https://github.com/RobotLocomotion/drake/pull/5130) feel a tad-bit unfortunate. It's similar to what we're thinking of in https://github.com/cockroachdb/dev/issues/11, and feels more appropriate to fix up-stream. Curious to hear...

I'm told the path issue is an underlying problem in https://github.com/cockroachdb/cockroach-go/blob/master/testserver/testserver.go, but this is where I ran into it, so 🏃

CRDB does not maintain n^2 gossip connections, it should be logarithmic. The gossip information disseminated cluster-wide should be linear to the number of nodes in the system, so it's a...

@dankinder, do you mind grabbing a profile for this running cluster, if still running? You could do it using the following (using the real address of course): ``` go tool...

yo @a-robinson, https://github.com/a-robinson/cockroach/commit/952c5011bb81e6315d677e1a160f44c90a5ae4e2 LGTM. These internal vtables are not that useful, definitely not worth the cost you're seeing. Some of them are being used in roachtests that have either already...

These aged roachtests could also just explicitly enable this cluster setting if we want to keep them (== roachtests) around, but that's all just unnecessary hedginess -- this feels fine...

> Perhaps exporting Go's `/sched/latencies:seconds` to have visibility in Go scheduler latencies. This has proven extremely valuable to do in internal AC-related experiments (re: #75066). https://github.com/irfansharif/cockroach/tree/220614.export-tracing is a prototype that...

From an [internal doc](https://docs.google.com/document/d/18S4uE8O1nRxULhSg9Z1Zt4jUPBiLJgMh7X1I6shsbug/edit#), re: "Information needed from Go runtime": ``` Runnable info: Minimally, we need the number of runnable goroutines, sampled at some reasonably high rate (100hz?). It would...

Adding Andrew here too to pick through the list within the next two weeks, it'll be a good way to get our feet wet.