[jira] [Commented] (CASSANDRA-3537) ExpiringMap timer is not exception-proof

2012-02-24 Thread Sylvain Lebresne (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-3537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13215537#comment-13215537 ] Sylvain Lebresne commented on CASSANDRA-3537: - +1 (nit: the overriden

[jira] [Commented] (CASSANDRA-3537) ExpiringMap timer is not exception-proof

2012-02-23 Thread Sylvain Lebresne (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-3537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13214515#comment-13214515 ] Sylvain Lebresne commented on CASSANDRA-3537: - bq. In other words, If any

[jira] [Commented] (CASSANDRA-3537) ExpiringMap timer is not exception-proof

2012-02-23 Thread Jonathan Ellis (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-3537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13214667#comment-13214667 ] Jonathan Ellis commented on CASSANDRA-3537: --- bq. Another would be to

[jira] [Commented] (CASSANDRA-3537) ExpiringMap timer is not exception-proof

2012-02-23 Thread Sylvain Lebresne (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-3537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13214675#comment-13214675 ] Sylvain Lebresne commented on CASSANDRA-3537: - No I meant that we would

[jira] [Commented] (CASSANDRA-3537) ExpiringMap timer is not exception-proof

2012-02-22 Thread MaHaiyang (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-3537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13213530#comment-13213530 ] MaHaiyang commented on CASSANDRA-3537: -- {quote} Simple patch attached to make

[jira] [Commented] (CASSANDRA-3537) ExpiringMap timer is not exception-proof

2012-02-22 Thread Jonathan Ellis (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-3537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13213670#comment-13213670 ] Jonathan Ellis commented on CASSANDRA-3537: --- catching Throwable or Error is

[jira] [Commented] (CASSANDRA-3537) ExpiringMap timer is not exception-proof

2012-02-22 Thread Sylvain Lebresne (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-3537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13213798#comment-13213798 ] Sylvain Lebresne commented on CASSANDRA-3537: - I like the idea of using

[jira] [Commented] (CASSANDRA-3537) ExpiringMap timer is not exception-proof

2012-02-22 Thread Jonathan Ellis (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-3537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13213893#comment-13213893 ] Jonathan Ellis commented on CASSANDRA-3537: --- bq. I'm not sure I understand