Skip to content
This repository has been archived by the owner on Jan 28, 2021. It is now read-only.

sql: add transaction_isolation default value #589

Merged
merged 1 commit into from
Jan 17, 2019

Conversation

erizocosmico
Copy link
Contributor

Fixes #564

I added READ UNCOMMITED because it's what's most similar to what we do (no transactions at all).

READ UNCOMMITTED

SELECT statements are performed in a nonlocking fashion, but a possible earlier version of a row > might be used. Thus, using this isolation level, such reads are not consistent. This is also called a dirty read. Otherwise, this isolation level works like READ COMMITTED.

@erizocosmico erizocosmico requested a review from a team January 17, 2019 10:43
Copy link
Contributor

@kuba-- kuba-- left a comment

Choose a reason for hiding this comment

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

Fix TestQueries/sequential/SHOW_VARIABLES

Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
@erizocosmico erizocosmico force-pushed the feature/transaction-isolation branch from b66268b to 30c1b05 Compare January 17, 2019 11:40
@erizocosmico
Copy link
Contributor Author

Fixed @kuba--

@ajnavarro ajnavarro merged commit 8e113a9 into src-d:master Jan 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants