siddhantsangwan commented on PR #8707: URL: https://github.com/apache/ozone/pull/8707#issuecomment-3083768899
> My main concern is that the KeyValueContainer in ContainerSet#containerMap will be inconsistent with the memory and yaml file. When the Datanode restarts, the Container will be reloaded from the yaml. So caching the yaml file can ensure the same logic as before. If you create a cache directly from ContainerData, it may be different from the cause logic. Perhaps we can filter the cached Container, for example, only cache the Container in the CLOSED state @xichen01 In what cases will ContainerData be inconsistent with the yaml file? I was under the assumption that they're kept in sync. -- 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: issues-unsubscr...@ozone.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org For additional commands, e-mail: issues-h...@ozone.apache.org
