Skip to content
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.

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

Do not send auto savepoint in simpleQuery mode fixes Issue 3107 #3112

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

davecramer
Copy link
Member

@davecramer davecramer commented Feb 9, 2024

Change name of AutoRollBackTestSuite to AutoRollBackTest so that it would run. There are exclude rules to stop Suite tests from running

…ould run. There are exclude rules to stop *Suite* tests from running

Do not send savepoint in simpleQuery mode
@vlsi
Copy link
Member

vlsi commented Feb 22, 2024

Previously the query was sent in simple mode due to a test failure: #955
We need to ensure we do not re-introduce the bug.

At the same time, if we go with "extended", then we create much more messages (parse+bind+exec) which might significantly affect the performance.

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.

None yet

2 participants