[
https://issues.apache.org/jira/browse/CASSANDRA-7978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14144324#comment-14144324
]
Jason Brown edited comment on CASSANDRA-7978 at 9/23/14 4:44 AM:
-
I was slightly concerned about the unexpected appearance of the CFMD that's
only used in one place, and hoped we could maybe get around that (as well as
reducing the scope of CP's dependencies). You are correct that we need some
wrapper to get those updates, and I just now spent time trying to see if it's
just easier to pass the CFMD instance into the CP constructor (and always have
a reference to that CFMD, rather than waiting for SSTR.getCompressionMetadata()
to pass it in). Unfortunately, it's not simple as CP has 4 constructors, called
from different paths, and being able to pass the correct CFMD instance to those
is not overly friendly. So, I'll stop making noise now and let you commit the
patch so we can move on :).
was (Author: jasobrown):
I was slightly concerned about the unexpected appearance of the CFMD that's
only used in one place, and hoped we could maybe get around that (as well as
reducing the scope of CP's dependencies). You are correct that we need some
wrapper to get those updates, and I just now spent time trying to see if it's
just easier to pass the CFMD instance into the CP constructor (and always have
a reference to that CFMD, rather than waiting for
SSTR.getCompressionMetadata()). Unfortunately, it's not simple as CP has 4
constructors, called from different paths, and being able to pass the correct
CFMD instance to those is not overly friendly. So, I'll stop making noise now
and let you commit the patch so we can move on :).
CrcCheckChance should not be stored as part of an sstable
-
Key: CASSANDRA-7978
URL: https://issues.apache.org/jira/browse/CASSANDRA-7978
Project: Cassandra
Issue Type: Improvement
Reporter: sankalp kohli
Assignee: T Jake Luciani
Priority: Minor
Fix For: 2.0.11
Attachments: 7978.txt, 7978v2.txt
CrcCheckChance is stored with compression parameters in the sstable. The only
way to change it is to do upgrade sstable. I don't see why it should not be a
hot property.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)