nevzheng commented on code in PR #12132:
URL: https://github.com/apache/gravitino/pull/12132#discussion_r3646956856


##########
common/build.gradle.kts:
##########


Review Comment:
   Addressed in 99b5defeb. The selected Jackson declarations predate this PR 
and remain required by the existing DTO/JSON code in `common`; this change does 
not add Jackson to `api`, and `api/build.gradle.kts` is unchanged. I removed 
the fixture-only Commons Lang declaration introduced by this PR and replaced 
its single use with JDK validation. The remaining fixture dependencies expose 
`KmsReference` and JUnit contract types to fixture consumers. Common fixture 
compilation, fixture JAR packaging, tests, and Spotless pass.



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