cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sam Tunnicliffe (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (CASSANDRA-9839) Move crc_check_chance out of compressions options
Date Sat, 19 Sep 2015 10:56:05 GMT

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

Sam Tunnicliffe edited comment on CASSANDRA-9839 at 9/19/15 10:55 AM:
----------------------------------------------------------------------

Thanks, updated bundled python driver and committed in {{e25453ba55d35cac802719c6fb906460f13a9c36}}.

There was another small dtest change required (to {{test_describe}} so I've opened a new pull
request for that ([link|https://github.com/riptano/cassandra-dtest/pull/567]). 

edit: plus follow up {{41731b84fd51bcd6bf1cb17ea22eaa6877df23a0}} because I didn't see [~carlyeks]'s
comment before committing.


was (Author: beobal):
Thanks, updated bundled python driver and committed in {{e25453ba55d35cac802719c6fb906460f13a9c36}}.

There was another small dtest change required (to {{test_describe}} so I've opened a new pull
request for that ([link|https://github.com/riptano/cassandra-dtest/pull/567]). 

> Move crc_check_chance out of compressions options
> -------------------------------------------------
>
>                 Key: CASSANDRA-9839
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9839
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Aleksey Yeschenko
>            Assignee: Paulo Motta
>            Priority: Minor
>              Labels: client-impacting, docs-impacting
>             Fix For: 3.0.0 rc1
>
>
> Follow up to CASSANDRA-8384. The option doesn't belong to compression params - it doesn't
affect compression, itself, and isn't passed to compressors upon initialization.
> While it's true that it is (currently) only being honored when reading compressed sstables,
it still doesn't belong to compression params (and is causing CASSANDRA-7978 -like issues).
> [~tjake] suggested we should make it an option of its own, and I think we should.



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

Mime
View raw message