Skip to content

Persist: plumb incremental apply APIs through #33045

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
Open
wants to merge 1 commit into
base: dov/run-based-spine-replacement
Choose a base branch
from
Open

Persist: plumb incremental apply APIs through #33045

wants to merge 1 commit into from
+468 −163

Conversation

DAlperin
Copy link
Member

Depends on #33044

Motivation

Tips for reviewer

Checklist

  • This PR has adequate test coverage / QA involvement has been duly considered. (trigger-ci for additional test/nightly runs)
  • This PR has an associated up-to-date design doc, is a design doc (template), or is sufficiently small to not require a design.
  • If this PR evolves an existing $T ⇔ Proto$T mapping (possibly in a backwards-incompatible way), then it is tagged with a T-proto label.
  • If this PR will require changes to cloud orchestration or tests, there is a companion cloud PR to account for those changes that is tagged with the release-blocker label (example).
  • If this PR includes major user-facing behavior changes, I have pinged the relevant PM to schedule a changelog post.

@DAlperin DAlperin requested review from ptravers, SangJunBak and a team as code owners July 15, 2025 21:09
@DAlperin DAlperin force-pushed the dov/enable-incremental-apis branch from 6f74d0b to c60199e Compare July 17, 2025 02:51
@DAlperin DAlperin force-pushed the dov/run-based-spine-replacement branch from f035a90 to 23e21c8 Compare July 17, 2025 02:51
@DAlperin DAlperin force-pushed the dov/enable-incremental-apis branch from c60199e to 5779b4d Compare July 17, 2025 14:26
@DAlperin DAlperin force-pushed the dov/run-based-spine-replacement branch from 23e21c8 to f497c4a Compare July 17, 2025 14:26
@DAlperin DAlperin force-pushed the dov/enable-incremental-apis branch from 5779b4d to 9f945f4 Compare July 17, 2025 14:34
@DAlperin DAlperin force-pushed the dov/run-based-spine-replacement branch from f497c4a to ae7312a Compare July 17, 2025 14:39
@DAlperin DAlperin force-pushed the dov/enable-incremental-apis branch 2 times, most recently from b75952b to 0d9a612 Compare July 17, 2025 15:08
@DAlperin DAlperin force-pushed the dov/run-based-spine-replacement branch 2 times, most recently from 2bb9775 to a15c651 Compare July 17, 2025 15:11
@DAlperin DAlperin force-pushed the dov/enable-incremental-apis branch from 0d9a612 to 43e3595 Compare July 17, 2025 15:11
@DAlperin DAlperin force-pushed the dov/run-based-spine-replacement branch from a15c651 to 85910d2 Compare July 17, 2025 15:21
@DAlperin DAlperin force-pushed the dov/enable-incremental-apis branch 2 times, most recently from ac52cb3 to 1ddef6e Compare July 21, 2025 15:17
@DAlperin DAlperin force-pushed the dov/run-based-spine-replacement branch from 85910d2 to 0274b1f Compare July 21, 2025 15:17
@DAlperin DAlperin force-pushed the dov/enable-incremental-apis branch from 1ddef6e to b4e20a7 Compare July 21, 2025 20:09
@DAlperin DAlperin force-pushed the dov/run-based-spine-replacement branch 2 times, most recently from ee85994 to 3c38325 Compare July 21, 2025 20:53
@DAlperin DAlperin force-pushed the dov/enable-incremental-apis branch 2 times, most recently from 2ac14ef to 0188a39 Compare July 21, 2025 20:55
@DAlperin DAlperin force-pushed the dov/run-based-spine-replacement branch from 3c38325 to a97ff26 Compare July 21, 2025 20:55
@DAlperin DAlperin force-pushed the dov/run-based-spine-replacement branch from a97ff26 to a946d3c Compare July 23, 2025 14:12
@DAlperin DAlperin force-pushed the dov/enable-incremental-apis branch from 0188a39 to a9fad35 Compare July 23, 2025 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ptravers ptravers Awaiting requested review from ptravers

@SangJunBak SangJunBak Awaiting requested review from SangJunBak

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@DAlperin