I agree that there are simpler ways of doing this. However, @imotov and I were discussing the recovery API several weeks ago, he mentioned that we have a standard way of defining enums and urged that I follow the convention. See for example IndexShardState.java. Is it okay to just keep this enum as-is for the sake of consistency?