search

aserafin / grape_logging

Request logging for Grape!
MIT License
147 stars 76 forks source link

Under the concurrent consumption of memory #3

Closed tian-xiaobo closed 9 years ago

tian-xiaobo commented 9 years ago

Under the concurrent consumption of memory!!!!!

aserafin commented 9 years ago

Could you explain little more?

ckorhonen commented 9 years ago

If I had to guess I'd imagine @tian-xiaobo may be referring to memory usage when using the gem - we've been experimenting with it recently and saw our unicorn's average memory usage jump by around 100mb with logging enabled.

Nothing fancy, just using the basic configuration as described in the readme.

aserafin commented 9 years ago

Even with the latest version? 4 cze 2015 8:59 PM "Chris Korhonen" notifications@github.com napisał(a):

If I had to guess I'd imagine @tian-xiaobo https://github.com/tian-xiaobo may be referring to memory usage when using the gem - we've been experimenting with it recently and saw our unicorn's average memory usage jump by around 100mb with logging enabled.

Nothing fancy, just using the basic configuration as described in the readme.

— Reply to this email directly or view it on GitHub https://github.com/aserafin/grape_logging/issues/3#issuecomment-109010180 .

marshall-lee commented 9 years ago

@ckorhonen @aserafin i think it's #7 related!

aserafin commented 9 years ago

@ckorhonen @tian-xiaobo @marshall-lee fixed thanks to @marshall-lee PR and released to rubygems https://rubygems.org/gems/grape_logging/versions/1.1.1