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

Benjamin Lerer commented on CASSANDRA-8384:
-------------------------------------------

I had a look, but moving {{crc_check_chance}} out of the compression options is 
not straight forward. So, I prefer if we address it in another ticket.

I fixed the review comments and implemented some of the refactoring discussed 
off line with [~iamaleksey]. 
The branch is 
[here|https://github.com/apache/cassandra/compare/trunk...blerer:8384]

> Change CREATE TABLE syntax for compression options
> --------------------------------------------------
>
>                 Key: CASSANDRA-8384
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-8384
>             Project: Cassandra
>          Issue Type: Sub-task
>            Reporter: Aleksey Yeschenko
>            Assignee: Benjamin Lerer
>              Labels: doc-impacting, docs
>             Fix For: 3.x
>
>
> Currently, `compression` table options are inconsistent with the likes of it 
> (table `compaction`, keyspace `replication`).
> I suggest we change it for 3.0, like we did change `caching` syntax for 2.1 
> (while continuing to accept the old syntax for a release).
> I recommend the following changes:
> 1. rename `sstable_compression` to `class`, to make it consistent 
> `compression` and `replication`
> 2. rename `chunk_length_kb` to `chunk_length_in_kb`, to match 
> `memtable_flush_period_in_ms`, or, alternatively, to just `chunk_length`, 
> with `memtable_flush_period_in_ms` renamed to `memtable_flush_period` - 
> consistent with every other CQL option everywhere else
> 3. add a boolean `enabled` option, to match `compaction`. Currently, the 
> official way to disable comression is an ugly, ugly hack (see CASSANDRA-8288)



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

Reply via email to