search

rabbitmq / rabbitmq-recent-history-exchange

RabbitMQ Recent History Exchange
Other
82 stars 21 forks source link

Store the routing key along with the message #15

Closed carlhoerberg closed 9 years ago

carlhoerberg commented 9 years ago

Fixes #13

videlalvaro commented 9 years ago

Why not cache the whole message as is instead?

carlhoerberg commented 9 years ago

Dunno :P can change that On Apr 13, 2015 6:14 PM, "Alvaro Videla" notifications@github.com wrote:

Why not cache the whole message as is instead?

— Reply to this email directly or view it on GitHub https://github.com/videlalvaro/rabbitmq-recent-history-exchange/pull/15#issuecomment-92300306 .

videlalvaro commented 9 years ago

Keep in mind this exchange, besides it usefulness, started as a teaching example for the book, that's why the functions are so broken up into small pieces and so on. Not storing full messages might be something from those times… or from basing this exchange on the LVC plugin.

So having this feature moved storing messages could actually makes thing simpler in the code and more future proof.