saintstack opened a new pull request, #4761:
URL: https://github.com/apache/hbase/pull/4761

   (Forward port from branch-2; simplified by the fact that there
   is no hadoop-2.0 profile on master branch)
   
       Make it so our published poms carry the minimum needed to run
       an hbase; the published pom has no profiles -- the profiles
       specified at build time are resolved, their dependencies inlined,
       and then they are stripped -- and no build-time, or plugins
       dependencies or properties, etc. Resultant poms have explicit
       hadoop lib versions baked in -- no more being able to choose
       hbase with hadoop2 or haddop3 at downstream build time by setting
       a '-Dhadoop.profile=X.0'.
   
       Pattern is to add profiles when none in sub-modules when
       the flatten plugin complains it can't resolve an hadoop
       dependency's 'version' (e.g. hadoop-common, hadoop-hdfs).
       Adding the profile in the sub-module make it so the flatten
       plugin can figure 'hadoop.version' definitively.
       (In master there is only the hadoop-3.0 profile).
   
       Another spin on the above happens when profiles already exist
       in submodule but the flatten plugin is complaining it can't
       figure figure version on an hadoop dependency NOT under
       profiles. Below, we move the delinquent hadoop dependency under
       existing profiles (minikdc was the usual dependency outside
       profiles in sub-modules that flatten complained about).
   
       Sometimes, moving an hadoop dependency under a profile, there
       would be excludes on the local dependency. If the parent pom
       excludes section was missing the local excludes, we added them
       up to the parent module so all excluding is done up there in
       the parent profile dependencyManagement section.


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

Reply via email to