jnturton opened a new pull request, #2730:
URL: https://github.com/apache/drill/pull/2730

   # [DRILL-8378](https://issues.apache.org/jira/browse/DRILL-8378): Support 
doing Maven releases using modern JDKs
   
   ## Description
   
   While [DRILL-8113](https://issues.apache.org/jira/browse/DRILL-8113) enabled 
the building of Drill using a modern JDK, more work is required to enable a 
Maven release of Drill using a modern JDK. Presently, the Maven Release Plugin 
will fail on Javadoc generation when run with a newer JDK while it succeeds 
with JDK 8. The failures are due to dependencies missing from the Maven Javadoc 
Plugin's config which I assume get treated with a more lenient "warn and skip" 
policy in the javadoc tool shipped with JDK 8 but cause errors in newer JDKs 
(in my case OpenJDK 17).
   
   In particular, the presence of the `sourcepath` property in the javadoc 
plugin's config in the root pom causes the default javadoc:javadoc goal to try 
to generate docs for our src/test packages. Unlike the javadoc:test-javadoc, 
the javadoc:javadoc goal does not inherit dependencies declared with `test` 
scope so it fails to resolve those.
   
   ## Documentation
   N/A
   
   ## Testing
   Successfully run maven release:prepare using OpenJDK 17.
   Successfully run mvn javadoc:javadoc in the Drill root module using OpenJDK 
17 with HTML output generated under each module's target/site/apidocs directory.
   


-- 
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: dev-unsubscr...@drill.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to