ben-manes commented on PR #1339: URL: https://github.com/apache/polaris/pull/1339#issuecomment-2810172689
> where the changes made to the id-to-name index are made visible before cache operations complete. I think you could mask that by using retry loops and versioning so that its treated like an optimistic read with a recheck condition. Then while you can make it internally visible prematurely but externally not. You could wrap the `Set<K>` keys by an `Index` type that also includes extra metadata, e.g. the value to perform an identity check and re-read if a stale traversal. If you want to avoid races causing redundant loads then there's no way except through an exclusive lock (or hiding that latency by returning a future if the caller can do other work in the meantime). -- 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]
