[
https://issues.apache.org/jira/browse/YARN-9562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16889228#comment-16889228
]
Eric Badger commented on YARN-9562:
-----------------------------------
Uploading patch 002 with unit tests added.
bq. When it is passing object, then developer may expect any type of object to
work, which is not the case. Many of the good answers argues against passing
Object for strong type language can be found here. Can we change to a subclass
please?
I'm open to changing this. What do you have in mind for subclassing instead?
bq. I think I understand the reason to make container image management as
plugins better base on your feedback. Thank you for that. Do we want to define
device plugin interface in here? CSI, CNI, GPU are examples that may need some
way to define availability of those resource, and claims of using them.
I think devices plugins are a little bit different. The plugins in this patch
are only for how to get the resources necessary to create a rootfs for the
container to run in (basically the docker image files). Device plugins would be
what we need to add on to the container to allow it to work with bind-mounts,
special runc runtimes. Those are things that we will probably want to add on in
the future (as we have with Docker), but I think they are beyond the scope of
this JIRA. Let me know if I'm misunderstanding what you're saying here.
> Add Java changes for the new RuncContainerRuntime
> -------------------------------------------------
>
> Key: YARN-9562
> URL: https://issues.apache.org/jira/browse/YARN-9562
> Project: Hadoop YARN
> Issue Type: Sub-task
> Reporter: Eric Badger
> Assignee: Eric Badger
> Priority: Major
> Attachments: YARN-9562.001.patch, YARN-9562.002.patch
>
>
> This JIRA will be used to add the Java changes for the new
> RuncContainerRuntime. This will work off of YARN-9560 to use much of the
> existing DockerLinuxContainerRuntime code once it is moved up into an
> abstract class that can be extended.
--
This message was sent by Atlassian JIRA
(v7.6.14#76016)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
