Skip to content

discussion: should thread_pool be renamed? #412

Description

@sgerbino

From the review result:

What's wrong with boost.thread thread_pool was asked somewhere? Well really this type isn't so much a thread pool as an execution context (the base class). I'd suggest renaming to thread_pool_context or threaded_context clears up the confusion here.

Source: Boost review result (conditional acceptance), Boost mailing list, September 2026.

The name is load-bearing: get_system_context() returns thread_pool&, and the type appears throughout the examples and docs. Renaming is a breaking change; options include:

  • Rename to thread_pool_context or threaded_context
  • Keep thread_pool and add an alias
  • Keep the name and document the "execution context that owns worker threads" framing where the confusion arises

Related: #395 (type renaming umbrella). If the naming pass there lands, this decision should ride along with it.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    • Status
      Ready

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions