sadanand48 commented on PR #10838: URL: https://github.com/apache/ozone/pull/10838#issuecomment-5338631964
About why the ledger/cache for the accounted keys is needed: Table presence only tells us that a deleted row exists; it does not tell us whether trapped accounting has already been applied for that row. We need exactly-once semantics across retries/replays and across multiple paths (snapshot create, DDS expand/promote, purge, snapshot-delete move). This is especially important for FSO where only dir roots are accounted at create, while subtree files can be accounted later during DDS traversal and may be revisited multiple times. A persisted ledger provides durable “already-accounted / purged” state and preserves ownership when rows move between snapshots, which table membership alone cannot represent. -- 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]
