You are viewing a plain text version of this content. The canonical link for it is here.
Posted to yarn-issues@hadoop.apache.org by "Vinod Kumar Vavilapalli (JIRA)" <ji...@apache.org> on 2014/04/17 00:34:19 UTC

[jira] [Updated] (YARN-1942) Many of ConverterUtils methods need to have public interfaces

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

Vinod Kumar Vavilapalli updated YARN-1942:
------------------------------------------

    Summary: Many of ConverterUtils methods need to have public interfaces  (was: ConverterUtils should not be Private)

I haven't looked at the entire class if it can be made public. Will do so, but for now, changing title to say that some utils there need to be public. It'll be great if we can come up with a list if the entire thing cannot be made public

> Many of ConverterUtils methods need to have public interfaces
> -------------------------------------------------------------
>
>                 Key: YARN-1942
>                 URL: https://issues.apache.org/jira/browse/YARN-1942
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: api
>    Affects Versions: 2.4.0
>            Reporter: Thomas Graves
>            Priority: Critical
>
> ConverterUtils has a bunch of functions that are useful to application masters.   It should either be made public or we make some of the utilities in it public or we provide other external apis for application masters to use.  Note that distributedshell and MR are both using these interfaces. 
> For instance the main use case I see right now is for getting the application attempt id within the appmaster:
> String containerIdStr =
>           System.getenv(Environment.CONTAINER_ID.name());
> ConverterUtils.toContainerId
> ContainerId containerId = ConverterUtils.toContainerId(containerIdStr);
>       ApplicationAttemptId applicationAttemptId =
>           containerId.getApplicationAttemptId();
> I don't see any other way for the application master to get this information.  If there is please let me know.



--
This message was sent by Atlassian JIRA
(v6.2#6252)