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

   ### What changes were proposed in this pull request?
   - Refactor Iceberg listener post/failure event payloads to avoid direct 
dependencies on Iceberg REST request/response classes.
   - Replace event fields/getters from Iceberg types (for example, 
`CreateTableRequest`, `LoadTableResponse`) to `Map<String, Object>` payloads.
   - Add `IcebergRESTUtils.toSerializableMap(Object)` to deep-clone event 
payloads into serializable map structures.
   - Apply this change consistently across table/namespace/view post and 
failure events.
   
   ### Why are the changes needed?
   - Issue #10464 asks to make Iceberg listener events independent from Iceberg 
API classes, enabling cleaner module boundaries and easier extraction.
   
   Fix: #10464
   
   ### Does this PR introduce _any_ user-facing change?
   - Listener event payload accessors now return `Map<String, Object>` instead 
of Iceberg-specific classes for affected post/failure events.
   
   ### How was this patch tested?
   - `./gradlew :iceberg:iceberg-rest-server:compileJava`
   - `./gradlew :iceberg:iceberg-rest-server:test --tests 
org.apache.gravitino.iceberg.service.rest.TestIcebergTableOperations --tests 
org.apache.gravitino.iceberg.service.rest.TestIcebergNamespaceOperations 
--tests org.apache.gravitino.iceberg.service.rest.TestIcebergViewOperations`


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