[YARN-3005] [JDK7] Use switch statement for String instead of if-else statement in RegistrySecurity.java

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.7.0
• Fix Version/s: 2.7.0
• Labels: newbie
• Target Version/s: 2.7.0
• Hadoop Flags: Reviewed
• Assignee: Kengo Seki
• Reporter: Akira Ajisaka
• Created: 05/Jan/15 17:07
• Updated: 15/Apr/15 19:03
• Resolved: 15/Jan/15 12:32

Description

Since we have moved to JDK7, we can refactor the below if-else statement for String.

    // TODO JDK7 SWITCH
    if (REGISTRY_CLIENT_AUTH_KERBEROS.equals(auth)) {
      access = AccessPolicy.sasl;
    } else if (REGISTRY_CLIENT_AUTH_DIGEST.equals(auth)) {
      access = AccessPolicy.digest;
    } else if (REGISTRY_CLIENT_AUTH_ANONYMOUS.equals(auth)) {
      access = AccessPolicy.anon;
    } else {
      throw new ServiceStateException(E_UNKNOWN_AUTHENTICATION_MECHANISM
                                      + "\"" + auth + "\"");
    }