Skip to content

core: do not rename user threads#12177

Merged
sergepetrenko merged 1 commit intotarantool:masterfrom
drewdzzz:user_thread_rename
Dec 25, 2025
Merged

core: do not rename user threads#12177
sergepetrenko merged 1 commit intotarantool:masterfrom
drewdzzz:user_thread_rename

Conversation

@drewdzzz
Copy link
Contributor

When a user thread uses Tarantool C API functions, user thread can be renamed because of creation of cord on demand. The commit removes thread renaming for these cords.

There is no portable way to set/get thread name, so let's simply run the test only on Linux.

Closes #12175

@drewdzzz drewdzzz requested a review from a team as a code owner December 24, 2025 17:30
@drewdzzz drewdzzz added the full-ci Enables all tests for a pull request label Dec 24, 2025
@drewdzzz drewdzzz force-pushed the user_thread_rename branch 2 times, most recently from a3afdfe to 6800478 Compare December 24, 2025 17:52
@drewdzzz drewdzzz self-assigned this Dec 24, 2025
@coveralls
Copy link

coveralls commented Dec 24, 2025

Coverage Status

coverage: 87.649% (+0.006%) from 87.643%
when pulling 221575a on drewdzzz:user_thread_rename
into dcdb3ee
on tarantool:master
.

@drewdzzz drewdzzz assigned nshy and unassigned drewdzzz Dec 25, 2025
@nshy nshy assigned drewdzzz and unassigned nshy Dec 25, 2025
When a user thread uses Tarantool C API functions, user thread can be
renamed because of creation of cord on demand. The commit removes thread
renaming for these cords.

There is no portable way to set/get thread name, so let's simply run the
test only on Linux.

Closes tarantool#12175

NO_DOC=bugfix
@drewdzzz drewdzzz assigned sergepetrenko and unassigned drewdzzz Dec 25, 2025
@sergepetrenko sergepetrenko added backport/3.2 Automatically create a 3.2 backport PR backport/3.3 Automatically create a 3.3 backport PR backport/3.5 Automatically create a 3.5 backport PR backport/3.6 Automatically create a 3.6 backport PR and removed full-ci Enables all tests for a pull request labels Dec 25, 2025
Copy link
Collaborator

@sergepetrenko sergepetrenko left a comment

Choose a reason for hiding this comment

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

Thanks!

@sergepetrenko sergepetrenko merged commit 495740f into tarantool:master Dec 25, 2025
92 of 96 checks passed
@TarantoolBot
Copy link
Collaborator

Successfully created backport PR for release/3.2:

@TarantoolBot
Copy link
Collaborator

Successfully created backport PR for release/3.3:

@TarantoolBot
Copy link
Collaborator

Successfully created backport PR for release/3.5:

@TarantoolBot
Copy link
Collaborator

Successfully created backport PR for release/3.6:

@TarantoolBot
Copy link
Collaborator

Backport summary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/3.2 Automatically create a 3.2 backport PR backport/3.3 Automatically create a 3.3 backport PR backport/3.5 Automatically create a 3.5 backport PR backport/3.6 Automatically create a 3.6 backport PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tarantool C API functions rename user threads to "unknown"

6 participants