Sean Glover
Sean Glover
I ran the full testsuite locally and successfully (JDK 8, Scala 2.12), but I see it failed in this PR's build results. I observe that other recent PR's have similar...
Highlighting @ijuma since he was the original reporter of 5859. What do you think? May I gain access to see the Confluent System Tests to see how they performed with...
I rebased from trunk last night. I ran the testsuite a few times locally and noticed transient errors with some of the integration tests, just like those reported by the...
Is the current state satisfactory to you @ijuma ? Let me know if there's any additional work you had in mind.
Hey @ijuma , I'm now creating the summary value lazily. When log tracing is not enabled the earliest reference that will cause it to evaluate is when the `requestDequeueTimeNanos` timing...
Hi @ijuma , I choose to always force evaluation of `bodyAndSize` in the `Request.releaseBuffer` method when the buffer is not null. You can see [my comment from earlier this week](https://github.com/apache/kafka/pull/4216#discussion_r153098668)...
@ijuma WDYT? Thanks for your time to date reviewing this PR. It's appreciated.
I've rebased on trunk and squashed this branch.