[GitHub] surekhasaharan edited a comment on issue #6901: Introduce published segment cache in broker

[GitBox]

surekhasaharan edited a comment on issue #6901: Introduce published segment 
cache in broker
URL: https://github.com/apache/incubator-druid/pull/6901#issuecomment-459146133
 
 
   > Can you also please record the impact on heap size in the broker? We 
already have pretty massive broker heaps and putting more pressure on them 
would be problematic.
   
   This is used memory comparison on our test broker over a day with about 67K 
segments, grey line is without this patch and blue line is with this patch. 
There is slightly higher memory usage with current published segments cache on 
broker. 
   <img width="1512" alt="broker-heap-compare" 
src="https://user-images.githubusercontent.com/13008746/52018434-3e971700-249f-11e9-9924-6da8200d0982.png";>
   

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


With regards,
Apache Git Services

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