Jon Chambers
Jon Chambers
`bindByType` does, indeed, resolve the problem. I'll certainly defer to your expertise, but it still seems very strange to me that this works: ```java batch.bind("uuid", (UUID)null); ... batch.bind("uuid", UUID.randomUUID()); ```...
I recognize this is an old issue, but I'd like to add a little more information and respond to a couple points from earlier: > Isn't reject commands on disconnected...
> For the time being, a workaround can be adding a `ChannelHandler` in front of `CommandHandler` via `NettyCustomizer`. Interesting! We'll look into this—thank you for the suggestion!
I'll also emphasize that, as @hdv noted in https://github.com/micrometer-metrics/micrometer/issues/2880#issuecomment-988287759, this also happens if we change this line: https://github.com/micrometer-metrics/micrometer/blob/f766d261076cd1e04f7dd87fdc7192560b3432b1/implementations/micrometer-registry-statsd/src/test/java/io/micrometer/statsd/StatsdMeterRegistryPublishTest.java#L294 …to: ```java IntStream.range(0, n).forEach(ignored -> counter.increment()); ``` …or, to eliminate any possible...
Following up on this: we've found and addressed our problem with under-reporting, and it was not related to this issue. That said, this test certainly fails for any n >...
I've made an attempt to fix this in #4042.
> We would be happy to review such a pull request. @shakuzen I recognize this may not be a top priority, but I'm hoping to take you up on the...
> So I don't think this PR is addressing the issue unless I'm missing something. Did you see different behavior to what I described above? Interesting! I'll have to take...
Well, I do have a live use case right now. I'll try to isolate it in a unit test, but our setup is that we're deploying an application to EKS/Fargate....
Sorry; this fell through the cracks. I'll come back to it as soon as I can, but that might be a little later next week.