[ https://issues.apache.org/jira/browse/YARN-3005?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kengo Seki updated YARN-3005: ----------------------------- Attachment: YARN-3005.002.patch [~ajisakaa], thank you for your comment. Revised patch attached. > [JDK7] Use switch statement for String instead of if-else statement in > RegistrySecurity.java > -------------------------------------------------------------------------------------------- > > Key: YARN-3005 > URL: https://issues.apache.org/jira/browse/YARN-3005 > Project: Hadoop YARN > Issue Type: Improvement > Affects Versions: 2.7.0 > Reporter: Akira AJISAKA > Priority: Trivial > Labels: newbie > Attachments: YARN-3005.001.patch, YARN-3005.002.patch > > > Since we have moved to JDK7, we can refactor the below if-else statement for > String. > {code} > // 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 + "\""); > } > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332)