nevzheng commented on PR #12171: URL: https://github.com/apache/gravitino/pull/12171#issuecomment-5075157020
@bharos Thanks for the PR. supportive of the direction. Could you update the PR description (or add a short comment) with a cache safety argument covering: 1. Current state. how batchListEntitiesByRelation uses the cache/locks today 2. Problem — why that blocks writers under mixed list+create 3. Proposed solution — cache-aside, no lock across the backend call 4. Safety — why this is safe given our consistency model (what we keep / what we relax; any residual races) 5. Efficacy — why the cache remains useful / evidence from the regression test + load results That’ll help reviewers reason about the architecture change, not just the lock removal. Thanks -- 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]
