[ 
https://issues.apache.org/jira/browse/YARN-11334?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Shilun Fan updated YARN-11334:
------------------------------
        Parent: YARN-5597
    Issue Type: Sub-task  (was: Improvement)

> [Federation] Improve SubClusterState#fromString parameter and LogMessage
> ------------------------------------------------------------------------
>
>                 Key: YARN-11334
>                 URL: https://issues.apache.org/jira/browse/YARN-11334
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: federation
>    Affects Versions: 3.4.0
>            Reporter: Shilun Fan
>            Assignee: Shilun Fan
>            Priority: Trivial
>              Labels: pull-request-available
>             Fix For: 3.4.0
>
>
> The SubClusterState object has a fromString method, which can be improved in 
> the following 2 places
> {code:java}
> /**
>  * Convert a string into {@code SubClusterState}.
>  *
>  * @param x the string to convert in SubClusterState
>  * @return the respective {@code SubClusterState}
>  */
> public static SubClusterState fromString(String x) {
>   try {
>     return SubClusterState.valueOf(x);
>   } catch (Exception e) {
>     LOG.error("Invalid SubCluster State value in the StateStore does not"
>         + " match with the YARN Federation standard.");
>     return null;
>   }
> } {code}
>  * The parameter is named x, which cannot well express the meaning of the 
> input parameter.
>  * The error log does not print the error value.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to