Skip to content

Add txn_isolation reference #2991

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

Merged
merged 7 commits into from
Jul 5, 2022
Merged

Add txn_isolation reference #2991

merged 7 commits into from
Jul 5, 2022

Conversation

p7nov
Copy link
Contributor

@p7nov p7nov commented Jun 30, 2022

Resolves #2793

@github-actions github-actions bot temporarily deployed to branch-gh-2793-isolation June 30, 2022 09:43 Inactive
@p7nov p7nov requested a review from rindblack15 June 30, 2022 09:57
@github-actions github-actions bot temporarily deployed to branch-gh-2793-isolation July 1, 2022 06:22 Inactive
@github-actions github-actions bot temporarily deployed to branch-gh-2793-isolation July 1, 2022 08:54 Inactive
@p7nov p7nov requested a review from Mons July 1, 2022 11:09
@github-actions github-actions bot temporarily deployed to branch-gh-2793-isolation July 1, 2022 11:30 Inactive
@github-actions github-actions bot temporarily deployed to branch-gh-2793-isolation July 4, 2022 03:31 Inactive
@p7nov p7nov requested a review from patiencedaur July 5, 2022 03:47
Transaction manager options
~~~~~~~~~~~~~~~~~~~~~~~~~~~
Setting the transaction isolation level
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

.. note::
Copy link
Contributor

Choose a reason for hiding this comment

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

The note mentions terms that haven't yet been introduced. Consider moving the note lower.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, moved the note.

.. note::

You can also do this in the net.box :ref:`stream:begin() <net_box-stream_begin>` method.
You can set the isolation level in the net.box :ref:`stream:begin() <net_box-stream_begin>` method
and :ref:`IPROTO_BEGIN <box_protocol-begin>` binary protocol request.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure that you can interfere with the IPROTO_BEGIN request per se. It is under the hood of stream:begin().

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If I read the binary protocol intro right, then yes, it's possible.

Co-authored-by: Dia Patience Daur <[email protected]>
@github-actions github-actions bot temporarily deployed to branch-gh-2793-isolation July 5, 2022 05:16 Inactive
@github-actions github-actions bot temporarily deployed to branch-gh-2793-isolation July 5, 2022 05:26 Inactive
@p7nov p7nov requested a review from patiencedaur July 5, 2022 05:34
@p7nov p7nov merged commit c79bc9a into latest Jul 5, 2022
@p7nov p7nov deleted the gh-2793-isolation branch July 5, 2022 05:53
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.

Document transaction isolation settings
3 participants