Spencer Judge

Results 93 comments of Spencer Judge

Superseded by https://github.com/temporalio/api/pull/553

@sainad2222 Thanks for your interest. The issue mentions you can search for this number, that leads you to two places the client needs to be changed and that's mostly it....

That's a bit tough for me given that I'm authoring a library, so my users find it very convenient to consume Prom metrics directly. I can make a PR to...

The original intended behavior is that the tags would apply to all emitted metrics. I agree with you that the two separate options for OTel makes good sense, we can...

Not a priority for us at the moment, but @deepika-awasthi if you'd like to make a PR yourself we'd welcome it. Thanks!

> Maybe we can make name optional? > I would avoid having conditional mutexes, we can always achieve that by using await + mutex +1 to both of these. I...

This approach is looking good to me :+1:

Since this is a large request that would require substantial server work, transferring to server for triage

This is the sort of small change we'd be more than happy to accept a PR for if you're interested!