HTHou opened a new pull request, #18011:
URL: https://github.com/apache/iotdb/pull/18011

   ## Summary
   
   - Exclude `jakarta.annotation:jakarta.annotation-api` from the managed 
`org.apache.thrift:libthrift` dependency.
   - Keep the project-level version management intact because 
REST/OpenAPI/Jersey still use Jakarta annotations through their own dependency 
path.
   
   ## Motivation
   
   After upgrading Thrift, `libthrift` started exposing 
`jakarta.annotation-api` as a runtime transitive dependency. IoTDB's Thrift 
Java generator suppresses `@Generated` annotations, and current 
Thrift-generated sources use `org.apache.thrift.annotation.Nullable`, so the 
Thrift path does not need Jakarta annotation APIs.
   
   ## Validation
   
   - `mvn -pl iotdb-protocol/thrift-commons dependency:tree 
-Dincludes=jakarta.annotation:jakarta.annotation-api,org.apache.thrift:libthrift
 -Dverbose`
   - `mvn -pl external-service-impl/rest-openapi dependency:tree 
-Dincludes=jakarta.annotation:jakarta.annotation-api -Dverbose`


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