yuqi1129 opened a new pull request, #11041: URL: https://github.com/apache/gravitino/pull/11041
### What changes were proposed in this pull request? Make Iceberg schema and table import idempotent when multiple Gravitino nodes try to import the same object concurrently. If `EntityStore.put(..., true)` reports that the entity already exists, the dispatcher now reuses the existing Gravitino entity instead of failing the request. ### Why are the changes needed? In HA deployments, another node can finish the same import first. The current code treats that race as an error, even though the entity is already present in Gravitino. That makes already-existing schema/table imports fail spuriously. Fix: #11040 ### Does this PR introduce _any_ user-facing change? Yes. Concurrent import/load requests become idempotent instead of failing with `EntityAlreadyExistsException` in the import path. ### How was this patch tested? `./gradlew :core:test --tests org.apache.gravitino.catalog.TestTableOperationDispatcher --tests org.apache.gravitino.catalog.TestSchemaOperationDispatcher -PskipITs` -- 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]
