roryqi opened a new pull request, #14540:
URL: https://github.com/apache/gravitino/pull/14540

   ### What changes were proposed in this pull request?
   
   Record IRC creators during the existing schema, table and view import write, 
and remove the separate creator-only database updates. New ancestors of nested 
namespaces inherit the leaf audit during insertion.
   
   Preserve catalog-reported creators and existing entity audits. Limit the 
fallback creator to the object being created or registered, and clear it after 
the import, including failures.
   
   ### Why are the changes needed?
   
   Follow-up to #14469 addressing [Jerry's review 
comment](https://github.com/apache/gravitino/pull/14469#discussion_r4440726927):
 recording a creator should not require another database operation after 
importing the entity.
   
   Depends on #14469. This branch currently includes its commits and should be 
rebased after it merges.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No REST API or configuration changes. IRC creator audits are persisted with 
fewer database writes.
   
   ### How was this patch tested?
   
   - 129 core tests passed: entity import context, schema/table/view 
dispatchers, and schema persistence on H2. Tests verify one initial entity 
write, no audit-only update, audit preservation, scope cleanup, thread 
isolation, and ancestor audits.
   - 114 Iceberg REST dispatcher tests passed, including create/register and 
import failure paths.
   - Spotless checks and `git diff --check` passed.
   
   The Iceberg test run reused the previously built and verified common JAR 
(`-x :common:jar`).
   


-- 
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]

Reply via email to