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

ASF GitHub Bot commented on METRON-610:
---------------------------------------

Github user cestella commented on a diff in the pull request:

    https://github.com/apache/incubator-metron/pull/388#discussion_r91103873
  
    --- Diff: metron-analytics/metron-statistics/pom.xml ---
    @@ -52,4 +52,62 @@
                 <version>${global_hbase_guava_version}</version>
             </dependency>
         </dependencies>
    +    <build>
    +        <plugins>
    +            <plugin>
    +                <groupId>org.apache.maven.plugins</groupId>
    +                <artifactId>maven-shade-plugin</artifactId>
    +                <version>${global_shade_version}</version>
    +                <configuration>
    +                    
<createDependencyReducedPom>true</createDependencyReducedPom>
    +                </configuration>
    +                <executions>
    +                    <execution>
    +                        <phase>package</phase>
    +                        <goals>
    +                            <goal>shade</goal>
    +                        </goals>
    +                        <configuration>
    +                            <relocations>
    +                                <relocation>
    +                                    <pattern>com.tdunning</pattern>
    +                                    
<shadedPattern>org.apache.metron.tdunning</shadedPattern>
    +                                </relocation>
    +                            </relocations>
    +                            <artifactSet>
    +                                <excludes>
    +                                    <exclude>storm:storm-core:*</exclude>
    +                                    <exclude>storm:storm-lib:*</exclude>
    +                                    <exclude>org.slf4j.impl*</exclude>
    +                                    
<exclude>org.slf4j:slf4j-log4j*</exclude>
    +                                </excludes>
    +                            </artifactSet>
    +                            <transformers>
    +                                <transformer
    +                                  
implementation="org.apache.maven.plugins.shade.resource.DontIncludeResourceTransformer">
    +                                     <resources>
    +                                        <resource>.yaml</resource>
    +                                        <resource>LICENSE.txt</resource>
    +                                        <resource>ASL2.0</resource>
    +                                        <resource>NOTICE.txt</resource>
    +                                      </resources>
    +                                </transformer>
    +                                <!-- UNCOMMENT THIS IF YOU NEED TO 
REGENERATE THE BEST GUESS NOTICES FILE WHICH REQUIRES PRUNING EVERY RELEASE -->
    +                                <!--transformer 
implementation="org.apache.maven.plugins.shade.resource.ApacheNoticeResourceTransformer">
    +                                    <addHeader>false</addHeader>
    +                                    
<projectName>${project.name}</projectName>
    +                                </transformer-->
    --- End diff --
    
    Probably.  It's not needed here in any case since this isn't a jar that 
gets distributed.


> OnlineStatisticsProvider serialization is broken at random in the REPL
> ----------------------------------------------------------------------
>
>                 Key: METRON-610
>                 URL: https://issues.apache.org/jira/browse/METRON-610
>             Project: Metron
>          Issue Type: Bug
>            Reporter: Casey Stella
>            Assignee: Casey Stella
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> We rely on the t-digest library version 3.1 and elasticsearch brings along 
> 3.0.  There is a small API incompatibility between the two versions (namely  
> the static method TDigest.createAvlTreeDigest() is available in 3.1 but not 
> 3.0).
> If the classpath for the Stellar REPL chooses the 3.0 version of the library, 
> then deserialization is broken.
> Strictly speaking this is not a problem of the serialized form being 
> incorrect, but a problem in the custom kryo serialization code in the class.  
> It relies on the default constructor being called and then the digest being 
> deserialized using the code within the t-digest library.  Because the default 
> constructor initializes the digest via a call that does not exist in 3.0, it 
> breaks.  The serialization logic is safe to use in both versions, but the 
> object can't be constructed in 3.0.
> This fix directly instantiates the AvlTreeDigest, which exists in both 
> versions.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to