Skip to content

[REOPEN]: Wait for database update to become durable#4889

Open
kim wants to merge 5 commits intomasterfrom
kim/confirmed-database-updates
Open

[REOPEN]: Wait for database update to become durable#4889
kim wants to merge 5 commits intomasterfrom
kim/confirmed-database-updates

Conversation

@kim
Copy link
Copy Markdown
Contributor

@kim kim commented Apr 24, 2026

Re-submit of #4846, which got reverted due to a flying spaghetti monster attack.

kim added 5 commits April 20, 2026 15:21
Confirmed reads applies only to subscription clients, calls to the the
HTTP API publish endpoint return a success response before the operation
is confirmed.

While we await scheduling of a new database, updates require to wait for
the update transaction to be confirmed. To allow this, the
`TransactionOffset` channel and the database's `DurableOffset` need to
be returned all the way up to the request handler.

Note that waiting for confirmation is almost always the right choice, so
can't be opted out of at the time of submission of this patch. Callers
may, however, extend the timeout after which waiting for confirmation is
cancelled.
@kim kim enabled auto-merge April 24, 2026 13:10
@kim kim requested review from Shubham8287 and gefjon April 24, 2026 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants