New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixes the implementation of Observe() #231

Merged

sighingnow merged 1 commit into etcd-cpp-apiv3:master from sighingnow:ht/fix-election

Jul 1, 2023

Member

sighingnow commented Jul 1, 2023

Fixes #224 by

Fixes the extra Read() as we only support WaitOnce().
Fixes the behavior of CancelObserve().
Add a test case and revisit the example README.


          Fixes the implementation of Observe()

17f12e3

Signed-off-by: Tao He <sighingnow@gmail.com>

sighingnow mentioned this pull request

call API observe more than once will lead to crash #224

Closed

sighingnow merged commit 32fae70 into etcd-cpp-apiv3:master

6 checks passed

sighingnow deleted the ht/fix-election branch

July 1, 2023 05:30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment