szetszwo commented on PR #1296: URL: https://github.com/apache/ratis/pull/1296#issuecomment-3418602456
> The current approach works functionally, but it might introduce overhead under high QPS scenarios — ... This PR also introduces overhead to appendEntries and stores duplicated information. Also, the new API `okForLocalReadBounded` seems inflexible -- it puts the logic in Ratis instead of Ozone. Ozone cannot change it later on. I agree that the `allowFollowerReadLocalLease(..)` has overhead. How about using it as the first implementation? Then we can measure the performance impact and look for improvement. The overhead may or may not have serious impact. -- 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]
