vigneshio commented on PR #4646: URL: https://github.com/apache/polaris/pull/4646#issuecomment-4692148364
Thanks @dimas-b @nandorKollar @adutra . Based on the discussion on the dev list, I've updated the PR: * `TARGET_ENTITY_CONCURRENTLY_MODIFIED` → **503** (temporary error, can be retried) * `ENTITY_CANNOT_BE_RESOLVED` and `CATALOG_PATH_CANNOT_BE_RESOLVED` → **404 NoSuchNamespaceException**, grouped together as suggested by @adutra. I have also opened follow-up issue #4729 for server-side retries on rename conflicts. I agree with @rmannibucau that server-side retry is the better long-term solution, and that work can be handled separately. One note: the only failing check is `IcebergCatalogRelationalTest > testEventsAreEmitted()`, which appears to be an unrelated async event test. This change does not touch that area, and the rename test passed successfully. -- 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]
