[GitHub] b-slim edited a comment on issue #6719: use SnapTreeMap instead of ConcurrentSkipListMap

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

b-slim edited a comment on issue #6719: use SnapTreeMap instead of ConcurrentSkipListMap
URL: https://github.com/apache/incubator-druid/pull/6719#issuecomment-445844711
 
 
   While 35% is spent in the `.get` call is true but most of that time 34% is consumed by the Druid comparator code, so not sure how the  snapTreeMap is going to solve this? can you present some results or benchmarks ?
   Plus the last commit in this project is 6 years old and seems that some issues are not fixed https://github.com/nbronson/snaptree/issues @dyf6372 are you aware of other projects using this ?

----------------------------------------------------------------
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