moscajs / aedes-persistence-redis

Aedes persistence, backed by redis
MIT License
21 stars 23 forks source link

[feat] Cache size option #63

Open robertsLando opened 4 years ago

robertsLando commented 4 years ago

Should this be an option?

https://github.com/moscajs/aedes-persistence-redis/blob/bd60484535a72bf3138c9659fa6fc0c61f591631/persistence.js#L37

gnought commented 4 years ago

What’s the impact if we set to HLRU to 1?

robertsLando commented 4 years ago

It will allow to have just 1 inflight message (Qos1/2), outgoingUpdate/outgoingClearMessageId will throw error when doesn’t find the key on the cache