JacksonYao287 commented on PR #3360: URL: https://github.com/apache/ozone/pull/3360#issuecomment-1117104859
thanks @errose28 for the review! the main motivation behind this change is the SCM memory usage. i originally thought it will take too much RAM if there are two many deleted containers. here, seems i have overthought it. will close this PR shortly. >a better approach might be to track deleted containers in RocksDB only. can you please explain this in more detail? do you mean if we have to track a massive amount of containers, no matter put or get the state of container , we just hit rocksdb directly, but not maintain a state map in RAM? -- 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]
