cockroachdb / cockroachdb/cockroach

c2c: consider adding a retry policy around pts update

Open
#103,918 1 comment 0 reactions 0 assignees View on GitHub
A-disaster-recovery C-wishlist T-disaster-recovery
Dominant language
Go
Stars
32.5k
Forks
4.1k
PR merge metrics
PR metrics pending

Description

In https://github.com/cockroachdb/cockroach/pull/103834, we decided to surface pts update errors to the job level, which will then get retried. We should consider retrying the PTS update in the frontier processor, though its unclear what benefit this would have.

With https://github.com/cockroachdb/cockroach/pull/103834, we'll have the same behavior as CDC, though they are considering getting a smarter around retrying pts updates in https://github.com/cockroachdb/cockroach/issues/98585. For c2c, it seems prudent to see what CDC ends up doing and if our first c2c customers hit issues related to pts updates.

Jira issue: CRDB-28259

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.