Skip to content

[3.4] Backport clientv3: remove v3.WithFirstKey() in Barrier.Wait()#16843

Merged
ahrtr merged 1 commit intoetcd-io:release-3.4from
jmhbnz:backport-barrier-wait-fix
Oct 28, 2023
Merged

[3.4] Backport clientv3: remove v3.WithFirstKey() in Barrier.Wait()#16843
ahrtr merged 1 commit intoetcd-io:release-3.4from
jmhbnz:backport-barrier-wait-fix

Conversation

@jmhbnz
Copy link
Copy Markdown
Member

@jmhbnz jmhbnz commented Oct 27, 2023

Backports #16156 to fix the unexpected blocking when using Barrier.Wait(), e.g. NewBarrier(client, a).Wait() will block if key a is not existed but a0 is existed, but it should return immediately.

fix the unexpected blocking when using Barrier.Wait(), e.g.
NewBarrier(client, a).Wait() will block if key a is not existed but a0 is existed, but it should return immediately.

Signed-off-by: James Blair <mail@jamesblair.net>
@jmhbnz jmhbnz mentioned this pull request Oct 27, 2023
14 tasks
Copy link
Copy Markdown
Member

@chaochn47 chaochn47 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @jmhbnz

Copy link
Copy Markdown
Member

@ahrtr ahrtr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks @jmhbnz

@ahrtr ahrtr merged commit 888625c into etcd-io:release-3.4 Oct 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants