[ 
https://issues.apache.org/jira/browse/RATIS-288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16570512#comment-16570512
 ] 

Tsz Wo Nicholas Sze commented on RATIS-288:
-------------------------------------------

After removed ratis-proto-shaded/src/main/java/ and 
ratis-hadoop-shaded/src/main/java/, {{mvn clean package -DskipTests}} works.

Now, {{mvn -DskipShade test}} failed with
{code}
[ERROR] Failed to execute goal on project ratis-common: Could not resolve 
dependencies for project org.apache.ratis:ratis-common:jar:0.3.0-SNAPSHOT: 
Failure to find org.apache.ratis:ratis-proto-shaded:jar:0.3.0-SNAPSHOT in 
https://repository.apache.org/snapshots was cached in the local repository, 
resolution will not be reattempted until the update interval of 
apache.snapshots has elapsed or updates are forced -> [Help 1]
{code}
Then, I have tried {{mvn install -DskipTests}} and then {{mvn -DskipShade 
test}} worked fine.

> Pom cleanup/simplification
> --------------------------
>
>                 Key: RATIS-288
>                 URL: https://issues.apache.org/jira/browse/RATIS-288
>             Project: Ratis
>          Issue Type: Improvement
>          Components: build
>            Reporter: Josh Elser
>            Assignee: Josh Elser
>            Priority: Minor
>         Attachments: RATIS-288.005.patch, RATIS-288.005_rat.txt
>
>
> I'm noticing quite a bit of over-complication in the build, mostly around 
> ratis-proto-shaded. From what I can tell in the git history, this is holdover 
> from quite some time ago (when the module itself was introduced).
> Some weird things I see:
>  * Everything being marked as optional
>  * Explicit scope=compile being listed (this is the default)
>  * Inheriting all configuration from the netty-all pom (not sure why we'd 
> want this)
>  * Recompilation of source files included in ratis-proto-shaded (shade-plugin 
> can do this already)
> My only guess is that some of this was to support the {{skipShade}} option. I 
> think I can halve the amount of time for the ratis-proto-shaded model, and 
> still support a workflow that will let folks skip re-compilation if they 
> haven't changed the protobufs



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to