search

twitter-archive / kestrel

simple, distributed message queue system (inactive)
http://twitter.github.io/kestrel
Other
2.77k stars 312 forks source link

Memory leak in TimeSeriesCollectorConfig #95

Closed vogelito closed 12 years ago

vogelito commented 12 years ago

Removing the following from our config got rid of a significant memory leak from our kestrel servers:

  admin.statsNodes = new StatsConfig {
    reporters = new TimeSeriesCollectorConfig
  }

The leak was identified by using yourkit to capture memory snapshots of a live kestrel server at different points in time and noticing that the com.twitter.ostrich.admin.TimeSeriesCollector object was growing significantly in size.

robey commented 12 years ago

closing as duplicate of issue 96.

robey commented 12 years ago

oops, forgot the magic for github autoformat:

issue #96

vogelito commented 12 years ago

Per my comment on #96, I think these are separate issues.

Let me know if I can provide any more information that would be useful for you. Please notice that I was looking at the retained size (not the shallow size) of these objects when doing my debugging.