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

Sylvain Lebresne commented on CASSANDRA-3871:
---------------------------------------------

It would be cleaner to centralize the handling of null in 
CompressionParameters.parseCompressorClass(). I'd also prefer using the empty 
string to set no compression (partly because I think it's more natural, but 
also partly because I don't want to have to care about some special casing of 
the grammar for CASSANDRA-3783).

bq. IMO a 2ary index CF should "inherit" the compression setting for its parent.

Agreed. But as this is not fully related to that issue, I've created 
CASSANDRA-3877 for that.
                
> Turn compression on by default
> ------------------------------
>
>                 Key: CASSANDRA-3871
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3871
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jonathan Ellis
>            Assignee: Pavel Yaskevich
>             Fix For: 1.1
>
>         Attachments: CASSANDRA-3871-v1.patch, CASSANDRA-3871.patch
>
>
> Compression has been available but off by default since 1.0.0.  It's enabled 
> in a lot of production environments now, with no major problems found.  (Some 
> problems with customizing the block size were found and fixed in the 1.0 
> releases.)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to