yuqi1129 opened a new pull request, #11392: URL: https://github.com/apache/gravitino/pull/11392
### What changes were proposed in this pull request? - Add a new `iceberg-listener` module for Iceberg listener event APIs. - Move Iceberg listener events out of `iceberg-rest-server`. - Replace Iceberg REST request/response objects in listener events with lightweight JSON payloads. - Keep Iceberg request/response serialization inside `iceberg-rest-server`. - Add tests to guard the listener module from Iceberg/Servlet dependencies. ### Why are the changes needed? Iceberg listener events are expected to be consumed by modules like search, but the current event classes live in `iceberg-rest-server` and expose Iceberg REST API classes. This introduces unnecessary dependencies and can cause class conflicts. Fix: #10464 ### Does this PR introduce _any_ user-facing change? No end-user behavior change. This changes the developer-facing Iceberg listener event payload API by replacing Iceberg REST objects with lightweight payload wrappers. ### How was this patch tested? - `./gradlew :iceberg:iceberg-listener:test :iceberg:iceberg-rest-server:test -PskipITs` - `git diff --check` - Verified `iceberg-listener/src/main/java` has no `org.apache.iceberg`, `org.apache.gravitino.iceberg`, or `javax.servlet` imports. -- 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]
