Currently contained monitor works only on Linux.

Looking at code
org.apache.hadoop.yarn.server.nodemanager.containermanager.monitor.ContainerMonitorImpl
it seems best
to have pluggable processinfotree object instead of ContainerMonitor.

ProcfsBasedProcessTree will be pluggable implementation something like
resourcecalculatorplugin.

I like the idea, and more portable code is good (although we should
also allow per-platform overrides in case that can provide better
results). Can you file a JIRA for this with your ideas/implementation
and provide back a link on this conversation?

On Thu, Apr 26, 2012 at 3:19 AM, Radim Kolar <[EMAIL PROTECTED]> wrote:
> Currently contained monitor works only on Linux.
>
> Looking at code
> org.apache.hadoop.yarn.server.nodemanager.containermanager.monitor.ContainerMonitorImpl
> it seems best
> to have pluggable processinfotree object instead of ContainerMonitor.
>
> ProcfsBasedProcessTree will be pluggable implementation something like
> resourcecalculatorplugin.

--
Harsh J

+1 for making ProcfsBasedProcessTree an implementation of a pluggable
interface. Will help projects like the port to Windows.

On 4/25/12 11:14 PM, "Harsh J" <[EMAIL PROTECTED]> wrote:

>I like the idea, and more portable code is good (although we should
>also allow per-platform overrides in case that can provide better
>results). Can you file a JIRA for this with your ideas/implementation
>and provide back a link on this conversation?
>
>On Thu, Apr 26, 2012 at 3:19 AM, Radim Kolar <[EMAIL PROTECTED]> wrote:
>> Currently contained monitor works only on Linux.
>>
>> Looking at code
>>
>>org.apache.hadoop.yarn.server.nodemanager.containermanager.monitor.Contai
>>nerMonitorImpl
>> it seems best
>> to have pluggable processinfotree object instead of ContainerMonitor.
>>
>> ProcfsBasedProcessTree will be pluggable implementation something like
>> resourcecalculatorplugin.
>
>
>
>--
>Harsh J
>        String[] wargs = { "taskkill", "F", "/PID ", pid };
>        args = wargs;
>

https://issues.apache.org/jira/browse/MAPREDUCE-4204 tracks something on
these lines for the port to Windows.
On 4/25/12 2:49 PM, "Radim Kolar" <[EMAIL PROTECTED]> wrote:

>Currently contained monitor works only on Linux.
>
>Looking at code
>org.apache.hadoop.yarn.server.nodemanager.containermanager.monitor.Contain
>erMonitorImpl
>it seems best
>to have pluggable processinfotree object instead of ContainerMonitor.
>
>ProcfsBasedProcessTree will be pluggable implementation something like
>resourcecalculatorplugin.