Details
-
Improvement
-
Resolution: Fixed
-
Major - P3
-
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.