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.