[ 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)