guohao-rosicky commented on PR #4905: URL: https://github.com/apache/ozone/pull/4905#issuecomment-1594032002
> This change cannot be committed as it is - see my comment above. > > I am interested to know what the use case for "owner" is - I have never seen it used. > > If there is a need for there to be ownership of containers, then for EC, the ownership may have to be at the pipeline level, but that would require managing several pools of pipelines which could be a problem too. > > We would need to understand more about the use case, problem and then think about a solution if one is needed. Currently ozone's container does not use owner, my own version uses owner for some resource isolation. Later I would like to find out by ContainerStateMap#getContainerIDsByOwner, let me modify the code so that it will not close the pipeline. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
