Jon Marius Venstad

Results 16 comments of Jon Marius Venstad

Hi, this is great! As luck would have it, I had to rebase my fork onto 3.9.2 a few weeks ago, and due to conflicts with #2111 I already reviewed...

I think the changes in this PR look good 👍 From what I can tell, the union of #2111, this, and #2154 (which I just opened) should together cover #1925,...

I reworked the Learner-socket-close code to be a bit less prone to misuse, while handling a conflict with the first commit, just now.

https://github.com/apache/zookeeper/commit/d2ee4dd050174c24c9f645102072ceaccd25f0fd fixes the last comment by @changruill.

A seemingly unrelated unit test failed in the PR jenkins job: it sleeps for a while, while waiting for some state that wasn't reached within timeout; it doesn't depend on...

> @jonmv What is your Jira id? It's also "jonmv".