[GitHub] clintropolis commented on issue #6750: update roaring from 0.5.18 to 0.7.30

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

clintropolis commented on issue #6750: update roaring from 0.5.18 to 0.7.30
URL: https://github.com/apache/incubator-druid/pull/6750#issuecomment-448372032
 
 
   >I saw you had an issue filed upstream for Roaring, did that get resolved?
   
   The issue is resolved :+1: 
   
   However, it might be worth stalling this PR to wait for another new release because newer patches have made roaring a multi-release jar to allow it to take advantage of some jdk 11 stuff when possible. This is cool, but requires bumping the version of `animal-sniffer-maven-plugin` to 1.16 on our end, because current version we are using breaks with the jdk 11 stuff.
   
   > Also, what about the other way, can bitmaps produced by 0.7.30 be read by 0.5.18?
   
   Heh, good point will test. 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

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