Skip to content

[backport 3.3] box: fix box.commit({wait = 'none'}) could yield#11363

Merged
sergepetrenko merged 1 commit intorelease/3.3from
backport/release/3.3/11314
Apr 10, 2025
Merged

[backport 3.3] box: fix box.commit({wait = 'none'}) could yield#11363
sergepetrenko merged 1 commit intorelease/3.3from
backport/release/3.3/11314

Conversation

@TarantoolBot
Copy link
Collaborator

@TarantoolBot TarantoolBot commented Apr 10, 2025

(This PR is a backport of #11314 to release/3.3 to a future 3.3.2 release.)


The issue is in order to decide whether submitting request to journal would yield or not we check the journal queue size. But the size can be less then limit while queue is not empty. At this moment submitting new request will yield.

Closes #11224

The issue is in order to decide whether submitting request to journal
would yield or not we check the journal queue size. But the size can be
less then limit while queue is not empty. At this moment submitting
new request will yield.

Closes #11224

NO_DOC=bugfix

(cherry picked from commit c580f3a)
@TarantoolBot TarantoolBot requested a review from a team as a code owner April 10, 2025 07:50
@TarantoolBot TarantoolBot added the full-ci Enables all tests for a pull request label Apr 10, 2025
@TarantoolBot TarantoolBot changed the title [Backport release/3.3] box: fix box.commit({wait = 'none'}) could yield [backport 3.3] box: fix box.commit({wait = 'none'}) could yield Apr 10, 2025
@sergepetrenko sergepetrenko removed the full-ci Enables all tests for a pull request label Apr 10, 2025
@coveralls
Copy link

Coverage Status

coverage: 87.462% (-0.03%) from 87.494%
when pulling 973008d on backport/release/3.3/11314
into 79deee0
on release/3.3
.

@sergepetrenko sergepetrenko merged commit 7cf3c88 into release/3.3 Apr 10, 2025
37 of 41 checks passed
@sergepetrenko sergepetrenko deleted the backport/release/3.3/11314 branch April 10, 2025 09:25
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.

5 participants