Skip to content
This repository was archived by the owner on Feb 2, 2024. It is now read-only.

Configure isolation level as a parameter. #25

Merged
merged 3 commits into from
Mar 27, 2020

Conversation

philippe2803
Copy link
Collaborator

It used to set the isolation level AUTOCOMMIT everywhere. This isolation level unfortunately is irrelevant for most databases.
The isolation level is now optional, which means if not configured, the SQL engine will apply the default isolation of your backend database. You can now set the isolation level of your choice however when running a project. Also added a new parameter to CLI to reflect this new parameter.

It used to set the isolation level AUTOCOMMIT everywhere. This isolation level unfortunately is irrelavant for most databases.
The isolation level is now optional, which means if not configured, the SQL engine will apply the default isolation of your backend database. You can now set the isolation level of your choice however  when running a project. Also added a new parameter to CLI to reflect this new parameter.
@philippe2803
Copy link
Collaborator Author

@sp-enric-fradera @sp-joan-madrid
We have a dev package for you to test it in local.

pip uninstall sqlbucket
pip install sqlbucket==0.3.3.dev3

You can run a small ETL, with a step failing on purpose and check that previous steps were actually committed. I've tested it in local and it worked but I just want to be sure I'm not initiating a breaking change on your system.

@philippe2803 philippe2803 merged commit 273b49b into master Mar 27, 2020
@philippe2803 philippe2803 deleted the feature/isolation-level branch March 27, 2020 16:22
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.

1 participant