Timothy J. Raymond

Results 44 comments of Timothy J. Raymond

mmm... fair point. We'd likely only run it every six months, where we find that other things have encoded knowledge about the Go version that weren't also updated in the...

@rjdenney Please revise the title and add a description to this PR. It's not clear to reviewers what the problem is and why your proposed changes fixes that problem.

@rbtr well, the patch is _correct_ as it stands in isolation, because bd236e72f6 introduced a defect to `zapai.(*Core).clone()` where the mutex is copied. I guess what I don't understand is...

@rbtr btw, I approved this because the code in its current state is provably broken, and this patch does provide a minimal fix. I agree that we should investigate using...

@camrynl Given @tamilmani1989 's comment above, please mark this as a draft until it's ready to merge.

I'll give this a shot. As discussed in Slack on 9/21/18 (discussion began roughly [here](https://gophers.slack.com/archives/C9LRAQN8N/p1537535672000200)), we're going to try to do this with [opencensus.io](https://opencensus.io/), and more specifically, the [ochttp](https://godoc.org/go.opencensus.io/plugin/ochttp) plugin....

@arschles Apologies, I had to take a step back due to some family issues that came up back in November. If someone else wants to pick this up, please feel...

@Polar-Beard @zachgersh Still interested in taking this on?

@Daavidaviid This seems similar to some other issues addressed by #201 . Would you mind pulling down that branch and seeing if it fixes your issue? Alternatively, could you post...

Hi @jtackaberry , I attempted to reproduce this issue in Docker since I'm on a Mac. Interestingly, I don't get the same output (using the ubuntu 18.04 image): ``` $...