Skip to content

[9.0] Fix for reserved connection usage with transaction#7666

Merged
harshit-gangal merged 6 commits intovitessio:release-9.0from
planetscale:bp-7646
Mar 16, 2021
Merged

[9.0] Fix for reserved connection usage with transaction#7666
harshit-gangal merged 6 commits intovitessio:release-9.0from
planetscale:bp-7646

Conversation

@harshit-gangal
Copy link
Copy Markdown
Member

Backport of #7646

@harshit-gangal harshit-gangal requested a review from systay March 11, 2021 16:55
@harshit-gangal harshit-gangal changed the base branch from master to release-9.0 March 11, 2021 16:55
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
1. Update the shard sessions only if there is any update i.e. updated transaction id or updated reserved id
2. If there is failure to execute BeginExecute api and the connection is reserved conn then check if shard session can be reset and execute ReserveBeginExecute api

Signed-off-by: Harshit Gangal <harshit@planetscale.com>
…erve connection

Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
@harshit-gangal harshit-gangal marked this pull request as ready for review March 16, 2021 14:09
@harshit-gangal harshit-gangal merged commit 08fd9a1 into vitessio:release-9.0 Mar 16, 2021
@harshit-gangal harshit-gangal deleted the bp-7646 branch March 16, 2021 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants