Skip to content

direct: separate state flushing from closing - #6764

Draft
denik wants to merge 1 commit into
mainfrom
denik/dstate-flush-close
Draft

denik wants to merge 1 commit into
mainfrom
denik/dstate-flush-close

Conversation

@denik

@denik denik commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Changes

  • Separate direct state flushing from closing so DMS version completion can read state features directly.
  • remove redundant recordsHistory field (features map is the source of truth).

This PR was written with Codex.

@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: d2ec9ae

Run: 36692660844

Env ✅​pass 🙈​skip Time
✅​ aws linux 276 15 5:35
✅​ aws windows 278 13 4:58
✅​ azure linux 275 15 5:38
✅​ azure windows 277 13 3:57
✅​ gcp linux 276 15 5:31
✅​ gcp windows 278 13 4:55
Top 6 slowest tests (at least 2 minutes):
duration env testname
4:55 aws windows TestAccept
4:52 gcp windows TestAccept
4:10 aws linux TestAccept
4:06 gcp linux TestAccept
3:55 azure linux TestAccept
3:39 azure windows TestAccept

@denik
denik force-pushed the denik/dstate-flush-close branch from 909c9f9 to ccac919 Compare September 21, 2026 12:32
@denik
denik force-pushed the denik/dstate-flush-close branch from ccac919 to 9076159 Compare September 30, 2026 08:17
@github-actions github-actions Bot added the DABs DABs related issues label Sep 30, 2026
@denik
denik force-pushed the denik/dstate-flush-close branch from 9076159 to d827ba6 Compare September 30, 2026 08:29
@denik
denik force-pushed the denik/dstate-flush-close branch from d827ba6 to d2ec9ae Compare September 30, 2026 08:53

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DABs DABs related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants