dennishuo commented on issue #1123: URL: https://github.com/apache/polaris/issues/1123#issuecomment-2745050027
Playing around with it more, unfortunately it does seem like there are still transaction serialization conflicts during the inserts even after fixing the `entities_active` index. We should probably still fix the index if possible since otherwise `lookupByName` right now is horribly inefficient, but ultimately for the legacy three-entity-table EclipseLink implementation we probably do need to add low-level retries at least, and the new AtomicOperation postgres impl should fare much better. We'll still want to add low-level retries where appropriate in AtomicOperationMetaStoreManager though. -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
