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

Hoss Man commented on SOLR-7467:
--------------------------------


Beyond some basic bug fixes, 3.1 includes 2 notable changes...

* new static {{TDigest.createDigest()}} factory method to create "a TreeDigest 
of whichever type is the currently recommended type"
** looked into switching to this to minimize code changes needed in Solr as the 
library itself improves in future versions, but this only helps for creating 
empty instances -- when attempting to merge the byte[] data from multiple 
shards, you still have to know the concrete TDigest implementation used to know 
which "fromBytes" method to call -- so it's not really useul to us yet (i filed 
https://github.com/tdunning/t-digest/issues/52)

* new MergingDigest implementation
** this looks interesting and might be worth switching to in the future, but 
based on the comments in the class level javadocs about more testing needed, 
and since the {{TDigest.createDigest()}} method mentioned above still uses 
{{AVLTreeDigest}} we should probably to leave to just leave our code alone and 
keep using {{AVLTreeDigest}} in Solr for now.



> upgrade tdigest (3.1)
> ---------------------
>
>                 Key: SOLR-7467
>                 URL: https://issues.apache.org/jira/browse/SOLR-7467
>             Project: Solr
>          Issue Type: Task
>            Reporter: Hoss Man
>            Assignee: Hoss Man
>
> The jar is a drop in replacement, just need a trivial change to 
> ivy-versions.properties and replacing the jar's sha1 file



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to