cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sylvain Lebresne (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-7392) Abort in-progress queries that time out
Date Tue, 14 Jul 2015 16:34:07 GMT


Sylvain Lebresne commented on CASSANDRA-7392:

General approach lgtm from a quick glance but I'm gonna hand off review to [~blerer] since
my plate is a bit full and I don't want to delay review too long.

Small aside: I'm really not a huge fan of having paths specific to tests in the middle of
actually used code (as in {{ReadCommand}}). I understand there isn't a much easier way to
test this currently and I don't oppose this in the short term, but it would make me very happy
if we could find a cleaner way to deal with this in the longer run. Maybe we could have a
way to mock up a SSTableReader (and to inject it so it's used by queries), which would allow
to make it run very slowly. Or something like that.

> Abort in-progress queries that time out
> ---------------------------------------
>                 Key: CASSANDRA-7392
>                 URL:
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Core
>            Reporter: Jonathan Ellis
>            Assignee: Stefania
>             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

View raw message