ivandika3 commented on PR #4922: URL: https://github.com/apache/ozone/pull/4922#issuecomment-1598151292
@sodonnel Thank you for the info regarding EC containers and pipelines. Yes, I don't think moving the owner to pipeline level should be done for now. > I think the omID needs to be the same for all om instances in a HA group. There should not be a need to close all containers on SCM just because there is an OM leader switch. Coming back to this, currently omID (UUID) is used to uniquely differentiate between the OMs as certificate clients. Therefore, I don't think this should be set to the same value across all the OMs. This might require us to have a new UUID representing the OM HA Service. The way I see it, there are two possible issues with regards SCM container owners: 1. Prevent sharing containers between OM HA services 2. Reduce the number of containers I believe that by changing the owner to omID (UUID), it will resolve the first issue. If we only want to resolve the second issue, the easiest way is to use the OM Service ID for the container owner. If we want to resolve both, I think it requires a new unique UUID to represent the OM HA Service, which requires to come up with a more involved solution (e.g. how to synchronize the UUID between OMs, etc). We can limit the scope for this patch to achieve the first goal. @sodonnel @guohao-rosicky Please let me know what you think. -- 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]
