-
Type:
Improvement
-
Resolution: Unresolved
-
Priority:
Major - P3
-
None
-
Affects Version/s: None
-
Component/s: None
-
Service Arch
The ClusterRole is class filling a pretty Service Arch niche. In working with it, I noticed that it could use a refresh.
This thing is basically an unsigned integer with some accessors, and isn't IMO served well by the amount of C++ overloading and out-of-line function calls, nested constants container of a different type, etc.
Particularly the nested Value enum, the implicit constructors, and the capitalization of constants like ClusterRole::ShardServer. These are going to be used a lot so I think it's important to bring them inline with our naming convention.
I think ServiceArch could take a crack at improving it in these early days.
- is depended on by
-
SERVER-87656 Complete TODO listed in SERVER-78810
-
- Blocked
-
- is duplicated by
-
SERVER-78810 Improve Cluster Role API usability
-
- Closed
-
- is related to
-
SERVER-78810 Improve Cluster Role API usability
-
- Closed
-