[ZOOKEEPER-4747] Java api lacks synchronous version of sync() call

• Type: New Feature
• Status: Closed
• Priority: Major
• Resolution: Implemented
• Fix Version/s: 3.10.0, 3.9.3
• Component/s: java client
• Labels: pull-request-available
• Assignee: Kezhu Wang
• Reporter: Kezhu Wang
• Created: 25/Sep/23 03:56
• Updated: 24/Oct/24 20:09
• Resolved: 10/Sep/24 06:32

Description

Ideally, it should be redundant just as what breed says in ZOOKEEPER-1167.

it wasn't an oversight. there is no reason for a synchronous version. because of the ordering guarantees, if you issue an asynchronous sync, the next call, whether synchronous or asynchronous will see the updated state.

But in case of connection loss and absent of ZOOKEEPER-22, client has to check result of asynchronous sync before next call. So, currently, we can't simply issue an fire-and-forget asynchronous sync and an read to gain strong consistent. Then in a synchronous call chain, client has to convert asynchronous sync to synchronous to gain strong consistent. This is what I do in EagerACLFilterTest::syncClient, it is apparently unfriendly to end users.

Issue Links

• relates to: ZOOKEEPER-1167 C api lacks synchronous version of sync() call.
• relates to: ZOOKEEPER-22 Automatic request retries on connect failover
• links to: GitHub Pull Request #2068