search

hapijs / catbox-memory

Memory adapter for catbox
Other
33 stars 30 forks source link

Memory limit does not restrict data size properly #7

Closed kpdecker closed 10 years ago

kpdecker commented 10 years ago

Currently running a catbox cache using config of

        "pack": {
                "cache": [
                        {
                                "engine": "catbox-memory",
                                "shared": true
                        }
                ]
        },

Which should default to a 100MB cache limit. Currently seeing instances of this cache in the 389MB range as reported by heapdump while under load. Trying to get more information about the state, but mdb is not cooperating right now.

kpdecker commented 10 years ago

This is fixed by https://github.com/kpdecker/catbox-memory/commit/f048264a476c7e38f0eebf634d40767268bb9973 but that code is dependent on #9.

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.