[GitHub] storm pull request: STORM-761: An option for new/updated Redis key...

2015-06-03 Thread emulvaney
Github user emulvaney commented on the pull request: https://github.com/apache/storm/pull/514#issuecomment-108367739 Thanks, everyone. :-) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] storm pull request: STORM-761: An option for new/updated Redis key...

2015-04-08 Thread emulvaney
Github user emulvaney commented on the pull request: https://github.com/apache/storm/pull/514#issuecomment-91002146 Okay, cluster stuff in place. I don't mind waiting for #504. Just good to get this in at some point, ideally on the 0.10.x-branch (where storm-redis first app

[GitHub] storm pull request: STORM-761: An option for new/updated Redis key...

2015-04-07 Thread emulvaney
GitHub user emulvaney opened a pull request: https://github.com/apache/storm/pull/514 STORM-761: An option for new/updated Redis keys to expire in RedisMapState This is much the same as the mechanism already available via `RedisStateUpdater` for `RedisState` instances. Users can

[GitHub] storm pull request: An option for new/updated Redis keys to expire...

2015-04-07 Thread emulvaney
Github user emulvaney closed the pull request at: https://github.com/apache/storm/pull/501 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] storm pull request: An option for new/updated Redis keys to expire...

2015-04-04 Thread emulvaney
Github user emulvaney commented on the pull request: https://github.com/apache/storm/pull/501#issuecomment-89707197 Thanks for the feedback. `RedisClusterMapState`, yes that makes sense. I haven't set up a Redis cluster yet, but I can see that being useful in the future.

[GitHub] storm pull request: An option for new/updated Redis keys to expire...

2015-04-02 Thread emulvaney
GitHub user emulvaney opened a pull request: https://github.com/apache/storm/pull/501 An option for new/updated Redis keys to expire in RedisMapState This is much the same as the mechanism already available via `RedisStateUpdater` for `RedisState` instances. Users can now set