ramitg254 commented on code in PR #5720:
URL: https://github.com/apache/hive/pull/5720#discussion_r2029719385
##########
pom.xml:
##########
@@ -783,9 +784,9 @@
<version>${jetty.version}</version>
</dependency>
<dependency>
- <groupId>org.eclipse.jetty</groupId>
- <artifactId>jetty-runner</artifactId>
- <version>${jetty.version}</version>
+ <groupId>org.apache.tomcat</groupId>
Review Comment:
Actually, I thought of it but as per apache-jsp from org.eclipse.jetty has
test dependencies with 3 cve for the current jetty version so i thought of not
taking risk of adding it although it is under test scope and also as apache-jsp
from org.mortbay.jasper was not alone enough to prevent compilation failures so
i also dropped for apache-jsp from org.eclipse.jetty
--
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: gitbox-unsubscr...@hive.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
