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

Sylvain Lebresne commented on CASSANDRA-5371:
---------------------------------------------

bq. The stress tool doesn't have a wide row scenario so it's hard to simulate 
out of the box

Agreed, and that's definitively lacking. I believe there is a few knobs that 
allow to do wideish rows, but that's probably not very realistic.

{quote}
I think LCS only ever makes sense on SSD
since LCS is for wide row or heavy updates
{quote}

I'm not sure I agree. Maybe there is some truth to it in our current 
implementation, but that would then be more of a quirk of the implementation 
that the goal. Typically, I'm not really sure why only wide rows would benefit 
it. There is certainly nothing in theory that makes it so. As for "it's for 
heavy updates only", I think that LCS has a number of nice properties (like 
avoiding huge files that require half of you disk in free space) that are nice 
even if you have a moderate to low update rate (and in that case you can 
definitively afford LCS on HDD). More concretely, I'm pretty sure we have tons 
on users on LCS on HDD.

Anyway, all this to say that I don't necessary agree on optimizing LCS for 
heavy writes + wide rows + SSD *if* that's done at the expense of all other 
type of workload (and I'm not saying that's what this patch is doing, just that 
discarding other type of workload as unimportant is not ok imo).

bq. LCS has a 5MB limit you still end up with a 10MB sstable with a single row

If having 10MB sstables being split due to row too wide is a problem, then you 
should either not use LCS or pick a 10MB limit for LCS, not 5MB.

Anyway, I'm not vetoing this or anything like this. Just trying to get a better 
understanding of why this is a good thing to do in general.
                
> Perform size-tiered compactions in L0
> -------------------------------------
>
>                 Key: CASSANDRA-5371
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-5371
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>            Reporter: Jonathan Ellis
>            Assignee: Jonathan Ellis
>             Fix For: 2.0
>
>         Attachments: HybridCompactionStrategy.java
>
>
> If LCS gets behind, read performance deteriorates as we have to check bloom 
> filters on man sstables in L0.  For wide rows, this can mean having to seek 
> for each one since the BF doesn't help us reject much.
> Performing size-tiered compaction in L0 will mitigate this until we can catch 
> up on merging it into higher levels.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to