symious commented on PR #10822: URL: https://github.com/apache/ozone/pull/10822#issuecomment-5112401872
@aswinshakil Thanks for the questions. > why not `KeyInfo.keyLocationList` We evaluated it and rejected it early on (it's under Alternatives in the OEP): - All versions in one RocksDB value means every PUT/DELETE is a full read-modify-write of a record that grows with the version count, instead of one small KV write per version operation. - A location group carries no per-version dataSize / mtime / ETag, and cannot express a delete marker or the null version. > Snapshot I've added an "Interaction with Ozone snapshots" section to the OEP. PTAL. Given how many edge cases sit in this intersection, we're inclined to gate it in phase 1: reject `CreateSnapshot` on buckets with versioning Enabled/Suspended. -- 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]
