[GitHub] [pulsar] lhotari opened a new pull request #10780: [Build] Fix building the Presto distribution without building everything

[GitBox <gi...@apache.org>]

lhotari opened a new pull request #10780:
URL: https://github.com/apache/pulsar/pull/10780


   ### Motivation
   
   It should be possible to build the full Pulsar distribution in the fastest possible way with this command:
   ```
   mvn clean install -Dmaven.test.skip=true -DskipSourceReleaseAssembly=true -Dspotbugs.skip=true -Dlicense.skip=true \
     -pl distribution/server -am
   ```
   
   This is broken since the Presto project dependencies are missing. The optimization to skip building the distribution when `-DskipBuildDistribution=true` (added as part of #10288) caused the issue. This PR will fix that.
   
   
   ### Modifications
   
   - add dependency from pulsar-presto-distribution to pulsar-presto-connector when skipBuildDistribution != true
   - support building Presto distribution with -Dmaven.test.skip=true


-- 
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.

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

[GitHub] [pulsar] eolivelli merged pull request #10780: [Build] Fix building the Presto distribution without building everything

[GitBox <gi...@apache.org>]

eolivelli merged pull request #10780:
URL: https://github.com/apache/pulsar/pull/10780


   


-- 
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.

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