Skip to content

box: fix error propagation in box.atomic#11826

Merged
locker merged 1 commit intotarantool:masterfrom
nshy:gh-11823-box-atomic-fix-error-propagation
Sep 9, 2025
Merged

box: fix error propagation in box.atomic#11826
locker merged 1 commit intotarantool:masterfrom
nshy:gh-11823-box-atomic-fix-error-propagation

Conversation

@nshy
Copy link
Contributor

@nshy nshy commented Sep 8, 2025

Currently there is mistaken conversion to string. We don't need it at all.

Closes #11823

@nshy nshy requested a review from a team as a code owner September 8, 2025 13:07
@coveralls
Copy link

coveralls commented Sep 8, 2025

Coverage Status

coverage: 87.623% (+0.03%) from 87.597%
when pulling 2d7e197 on nshy:gh-11823-box-atomic-fix-error-propagation
into 61b744c
on tarantool:master
.

@nshy nshy requested a review from locker September 8, 2025 13:36
@locker locker assigned nshy and unassigned locker Sep 8, 2025
Currently there is mistaken conversion to string. We don't need it at
all.

Closes tarantool#11823

NO_DOC=bugfix
@nshy nshy force-pushed the gh-11823-box-atomic-fix-error-propagation branch from 5dd636b to 2d7e197 Compare September 9, 2025 09:31
@nshy nshy added the full-ci Enables all tests for a pull request label Sep 9, 2025
@nshy nshy assigned locker and unassigned nshy Sep 9, 2025
@locker locker merged commit 4fe0e61 into tarantool:master Sep 9, 2025
76 of 79 checks passed
@locker
Copy link
Member

locker commented Sep 9, 2025

Cherry-picked to 3.2, 3.3, 3.4, 3.5.

@nshy nshy deleted the gh-11823-box-atomic-fix-error-propagation branch September 15, 2025 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full-ci Enables all tests for a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

box.atomic mangles plain errors

4 participants