slfan1989 commented on code in PR #15897: URL: https://github.com/apache/iceberg/pull/15897#discussion_r3292201938
########## kafka-connect/kafka-connect-runtime/runtime-deps.txt: ########## @@ -141,12 +140,11 @@ io.perfmark:perfmark-api:0.27 io.projectreactor.netty:reactor-netty-core:1.2 io.projectreactor.netty:reactor-netty-http:1.2 io.projectreactor:reactor-core:3.7 -jakarta.activation:jakarta.activation-api:1.2 -javax.annotation:javax.annotation-api:1.3 -javax.servlet.jsp:jsp-api:2.1 -javax.servlet:javax.servlet-api:3.1 -javax.xml.bind:jaxb-api:2.2 -javax.xml.stream:stax-api:1.0-2 +jakarta.annotation:jakarta.annotation-api:1.3 Review Comment: `runtime-deps.txt` no longer contains `javax.servlet-api`, `jaxb-api`, `stax-api`, or `jakarta.annotation-api`, so I did not add those to the main runtime LICENSE. Instead, I removed the stale entries from `main/LICENSE`. For the hive distribution, the current classpath still includes `jakarta.annotation-api`, `jakarta.servlet-api`, `jakarta.servlet.jsp-api`, and `javax.xml.bind:jaxb-api`, so I updated `hive/LICENSE` to reflect those actual dependencies and removed the old/duplicate entries. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
