[
https://issues.apache.org/jira/browse/CASSANDRA-8384?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14602544#comment-14602544
]
Benjamin Lerer commented on CASSANDRA-8384:
-------------------------------------------
the branch containing the patch is
[here|https://github.com/blerer/cassandra/tree/8384-8099].
The patch has been made on top of {{8099}} as discussed off line with
[~iamaleksey].
The patch still accept the old option names but will log a warning the first
time that they are used.
The unit tests are
[here|http://cassci.datastax.com/view/Dev/view/blerer/job/blerer-8384-8099-testall/lastCompletedBuild/testReport/]
and the Dtests are
[here|http://cassci.datastax.com/view/Dev/view/blerer/job/blerer-8384-8099-dtest/lastCompletedBuild/testReport/].
I do not think that my patch introduced any failure but it is hard to say as
a lot of tests are failing with {{8099}}.
> 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)
