cjmctague commented on a change in pull request #995: fixes #890
URL: https://github.com/apache/fluo/pull/995#discussion_r161382736
##########
File path: modules/core/pom.xml
##########
@@ -141,6 +141,33 @@
</execution>
</executions>
</plugin>
+ <plugin>
+ <groupId>org.apache.maven.plugins</groupId>
+ <artifactId>maven-shade-plugin</artifactId>
+ <executions>
+ <execution>
+ <goals>
+ <goal>shade</goal>
+ </goals>
+ <phase>package</phase>
+ <configuration>
+ <minimizeJar>true</minimizeJar>
+ <artifactSet>
+ <includes>
+ <include>org.apache.thrift:libthrift</include>
+ </includes>
+ </artifactSet>
+ <shadedArtifactAttached>true</shadedArtifactAttached>
+ <shadedClassifierName>shaded</shadedClassifierName>
Review comment:
@keith-turner Yes I was still working on this. Currently in Hawaii kinda off
the grid for most of the day for class. Survived the "Incoming ballistic
missile" false alarm though.
I have it shaded and relocated but it is creating an uber-jar in fluo-core
that has all the dependancies from the entire project and not just just that
module so I want to fix that at some point.
I could probably vpn into my workstation later and push what I have changed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
