Skip to content

client: support streams for interactive transactions#155

Open
drewdzzz wants to merge 4 commits intotarantool:masterfrom
drewdzzz:txn_streams
Open

client: support streams for interactive transactions#155
drewdzzz wants to merge 4 commits intotarantool:masterfrom
drewdzzz:txn_streams

Conversation

@drewdzzz
Copy link
Copy Markdown
Collaborator

@drewdzzz drewdzzz commented Mar 3, 2026

Description TBD.

@drewdzzz drewdzzz force-pushed the txn_streams branch 3 times, most recently from d8c1cda to 6081c21 Compare March 3, 2026 18:32
@drewdzzz drewdzzz self-assigned this Mar 13, 2026
@drewdzzz drewdzzz force-pushed the txn_streams branch 7 times, most recently from f64fbe7 to aa284c5 Compare March 27, 2026 12:47
MacOS 13 github runner is not supported anymore, so let's replace it
with the newest macos-26 one. Also, two MacOS platforms are enough
actually, so let's remove MacOS 14 runner as well.

Also, let's use the latest MacOS for sanitizing workflows.
Let's set `BreakConstructorInitializers` option to `AfterColon`,
the inherited value was `BeforeComma` which is VERY ugly (each string
started with a comma like `, field(value)`).
All these methods just call several methods of `ConnectionImpl` so,
logically, the belong there.

Also, we are going to add `stream_id` argument for almost each such
private method, so we will add them right to the moved methods in
future.

Along the way, get rid of unused `readyToSend(Connection)` methods.

Part of tarantool#154
@drewdzzz drewdzzz force-pushed the txn_streams branch 4 times, most recently from 5195648 to 5e5a064 Compare March 30, 2026 12:53
The commit adds support of Tarantool streams, allows to set txn
isolation level and timeout for them. See doc changes for details.

Closes tarantool#154
@drewdzzz drewdzzz assigned mkostoevr and CuriousGeorgiy and unassigned drewdzzz Mar 30, 2026
@drewdzzz drewdzzz marked this pull request as ready for review March 30, 2026 12:59
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.

3 participants