Aggarwal-Raghav commented on PR #6210: URL: https://github.com/apache/hive/pull/6210#issuecomment-3568228875
@deniskuzZ Based on the changes done in this PR, what I understand is that the goal is to sync the maven compiler and javadoc plugin in standalone-metastore and storage-api module. Which makes sense but why hardcoded version is used? IMO, it would be better to introduce variables in `standalone-metastore/pom.xml` and `storage-api/pom.xml`. In order to verify if variables are getting resolved correctly `mvn help:effective-pom` can be ran in respective module. -- 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]
