nevzheng commented on code in PR #12132: URL: https://github.com/apache/gravitino/pull/12132#discussion_r3646956856
########## common/build.gradle.kts: ########## Review Comment: @jerryshao, this cleanup request is reasonable. I removed the unnecessary test-fixture dependency introduced by this PR and replaced its only use with JDK code. The remaining selected Jackson dependencies predate this change and are used broadly throughout `common`. Removing them would require auditing and refactoring existing consumers, reconsidering the shared JSON/DTO module boundary, and validating downstream dependency metadata and builds. That is worthwhile, but outside the scope of this KMS API PR. Tracked separately in [#12193](https://github.com/apache/gravitino/issues/12193). -- 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]
