tanishq-chugh commented on code in PR #5404:
URL: https://github.com/apache/hive/pull/5404#discussion_r2062921264
##########
pom.xml:
##########
@@ -1894,6 +1910,29 @@
<artifactId>jamon-maven-plugin</artifactId>
<version>${jamon.plugin.version}</version>
</plugin>
+ <plugin>
+ <groupId>org.apache.maven.plugins</groupId>
+ <artifactId>maven-javadoc-plugin</artifactId>
Review Comment:
Hi @ayushtkn
The javadoc plugin implementation has been more stricter in jdk-17 as
compared to jdk-8. With jdk-17, All the deps being used should be available in
classpath when javadoc is being run, this check was not so strict with jdk-8.
This plugin is needed now specifically to add the
`io.dropwizard.metrics:metrics-core` dependency on the classpath at aggregation
time as it being referenced directly.
--
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]
