[ 
https://issues.apache.org/jira/browse/CASSANDRA-7392?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14736665#comment-14736665
 ] 

Stefania commented on CASSANDRA-7392:
-------------------------------------

Thanks for looking into this so quickly. I agree on the first 3 bullet points 
and I will address them tomorrow. 

As for extending AtomicReference from MonitorableImpl, I think it would look 
really odd and not very clean to have ReadCommand be an AtomicReference. I 
think we would be abusing the "is a" principle a bit too much.

Do let me know if you have more comments and I will try to address them all 
together.

> Abort in-progress queries that time out
> ---------------------------------------
>
>                 Key: CASSANDRA-7392
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-7392
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Core
>            Reporter: Jonathan Ellis
>            Assignee: Stefania
>            Priority: Critical
>             Fix For: 3.x
>
>
> Currently we drop queries that time out before we get to them (because node 
> is overloaded) but not queries that time out while being processed.  
> (Particularly common for index queries on data that shouldn't be indexed.)  
> Adding the latter and logging when we have to interrupt one gets us a poor 
> man's "slow query log" for free.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to