[jira] [Commented] (YARN-1328) ContainerId should expose a public static string of the ID's prefix

["Zhijie Shen (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/YARN-1328?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13834309#comment-13834309 ] 

Zhijie Shen commented on YARN-1328:
-----------------------------------

bq. We ideally wouldn't change them, but its much better to depend on APIs anyways.

Agreed. Let's keep to APIs to convert the IDs, and determine whether the given strings are IDs. We can find the APIs in ConvertUtils.

Meanwhile, I checked that the prefix constants in ApplicationId and ApplicationAttemptId are already marked \@Private. Let's just leave them there for compatibility.

Close the ticket as Won't Fix

> ContainerId should expose a public static string of the ID's prefix
> -------------------------------------------------------------------
>
>                 Key: YARN-1328
>                 URL: https://issues.apache.org/jira/browse/YARN-1328
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Zhijie Shen
>            Assignee: Zhijie Shen
>            Priority: Minor
>         Attachments: YARN-1328.1.patch, YARN-1328.2.patch
>
>
> Like ApplicationId and ApplicationAttemptId, ContainerId should also have public static string:
> {code}
>   public static final String containerIdStrPrefix = "container_";
> {code}
> Others can refer to this string to verify a containerId string.



--
This message was sent by Atlassian JIRA
(v6.1#6144)