Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-67927

change ServiceExecutor::ThreadingModel definition

    XMLWordPrintableJSON

Details

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major - P3 Major - P3
    • 6.1.0-rc0
    • None
    • None
    • None
    • Fully Compatible
    • Service Arch 2022-07-25

    Description

      The transport::ServiceExecutor defines the enum class ThreadingModel.
      It has only the kBorrowed and kDedicated states. By its nature, this can really only have two states and an enum class is needlessly opaque. But type safety is desired so we don't want to just replace it with bool.

      Change the definition from enum class ThreadingModel to enum HasDedicatedThread : bool to get best of both worlds.

      Remove the redundant typedef in ServiceExecutorContext. No need for that.

      Attachments

        Activity

          People

            billy.donahue@mongodb.com Billy Donahue
            billy.donahue@mongodb.com Billy Donahue
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: