[GitHub] [incubator-doris] wangbo commented on pull request #3320: (#3319) support java version hyperloglog

[GitBox <gi...@apache.org>]

wangbo commented on pull request #3320:
URL: https://github.com/apache/incubator-doris/pull/3320#issuecomment-619336137


   >Is it possible to consider using some test data, comparing the hll data obtained from the C++ version with the java version to verify whether the two versions of the algorithm are consistent? And this verification can be put into unit tests.
   @morningman Current UT has contains the case that the approximation from``` 0 to Short.MAX ```  is consistent with be. I'll try to find more special case


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org