yuqi1129 commented on PR #10119:
URL: https://github.com/apache/gravitino/pull/10119#issuecomment-4010436985

   更正并补充 review 意见(关联 #10117):
   
   我复核了本次变更里关于 owner 权限路径的实现,主要风险点在于 checkOwner 从本地 owner cache 改成了每次授权实时查 
relation store,这条路径处于高频授权链路,可能带来明显的性能回退(尤其在 catalog/table 级别高并发鉴权场景)。
   
   建议:如果后续要继续推进去缓存方案,最好同时补充压测/基准数据,并明确依赖的下层缓存命中策略与失效语义,避免在权限热路径出现吞吐抖动。当前回滚 PR 
#10279 的方向是合理的。
   


-- 
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]

Reply via email to