Skip to content

Fix: establish transaction and session before snapshot promotion/demotion#4899

Merged
treysp merged 7 commits intomainfrom
trey/promote-demote-session
Jul 7, 2025
Merged

Fix: establish transaction and session before snapshot promotion/demotion#4899
treysp merged 7 commits intomainfrom
trey/promote-demote-session

Conversation

@treysp
Copy link
Contributor

@treysp treysp commented Jul 3, 2025

No description provided.

Copy link
Collaborator

@erindru erindru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was going to comment saying that this needed a test, but I think that updating the failing tests (which are failing because the new calls are unexpected) to now expect the calls should cover this

@treysp treysp force-pushed the trey/promote-demote-session branch 2 times, most recently from baeb229 to 3e7facd Compare July 3, 2025 23:43
@treysp treysp force-pushed the trey/promote-demote-session branch 5 times, most recently from 9dc0521 to 94ce708 Compare July 7, 2025 17:08
@treysp treysp force-pushed the trey/promote-demote-session branch 3 times, most recently from 266779f to 5f51295 Compare July 7, 2025 20:24
@treysp treysp force-pushed the trey/promote-demote-session branch from 5f51295 to 447394d Compare July 7, 2025 21:35
@treysp treysp force-pushed the trey/promote-demote-session branch from 447394d to db950e5 Compare July 7, 2025 21:58
@treysp treysp merged commit 12bd71e into main Jul 7, 2025
27 checks passed
@treysp treysp deleted the trey/promote-demote-session branch July 7, 2025 22:18
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.

4 participants