Skip to content

Conversation

@dt
Copy link
Contributor

@dt dt commented Jan 30, 2026

This implements online restore using the normal restore distsql flow, rather than the bespoke online restore paths used in its initial prototype. This is behind a setting for now, but eventually could be the basis for extending online restore -- now unified with regular restore -- to support ingesting rather than linking some data (such as revision-history layers).

Release note: none.
Epic: none.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@dt dt force-pushed the restore-unification branch from de5c6c1 to 997baba Compare January 30, 2026 22:06
@dt dt marked this pull request as ready for review January 30, 2026 22:11
@dt dt requested a review from a team as a code owner January 30, 2026 22:11
@dt dt requested review from andrew-r-thomas and msbutler and removed request for a team January 30, 2026 22:11
@github-actions
Copy link

Potential Bug(s) Detected

The three-stage Claude Code analysis has identified potential bug(s) in this PR that may warrant investigation.

Next Steps:
Please review the detailed findings in the workflow run.

Note: When viewing the workflow output, scroll to the bottom to find the Final Analysis Summary.

After you review the findings, please tag the issue as follows:

  • If the detected issue is real or was helpful in any way, please tag the issue with O-AI-Review-Real-Issue-Found
  • If the detected issue was not helpful in any way, please tag the issue with O-AI-Review-Not-Helpful
@github-actions github-actions bot added the o-AI-Review-Potential-Issue-Detected AI reviewer found potential issue. Never assign manually—auto-applied by GH action only. label Jan 30, 2026
@dt dt force-pushed the restore-unification branch from 997baba to 00b97e2 Compare January 30, 2026 23:14
@dt dt added the O-AI-Review-Real-Issue-Found AI reviewer found real issue label Jan 30, 2026
This implements online restore using the normal restore distsql flow,
rather than the bespoke online restore paths used in its initial prototype.
This is behind a setting for now, but eventually could be the basis for
extending online restore -- now unified with regular restore -- to support
ingesting rather than linking some data (such as revision-history layers).

Release note: none.
Epic: none.
@dt dt force-pushed the restore-unification branch from 00b97e2 to 56dbb7a Compare January 31, 2026 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

o-AI-Review-Potential-Issue-Detected AI reviewer found potential issue. Never assign manually—auto-applied by GH action only. O-AI-Review-Real-Issue-Found AI reviewer found real issue

2 participants