Skip to content

Cassandra 2.1#1

Open
jivemonkey wants to merge 5 commits into
jeffjirsa:cassandra-2.1from
newrelic-forks:cassandra-2.1
Open

Cassandra 2.1#1
jivemonkey wants to merge 5 commits into
jeffjirsa:cassandra-2.1from
newrelic-forks:cassandra-2.1

Conversation

@jivemonkey

Copy link
Copy Markdown

@jeffjirsa how does this look?

@jeffjirsa

Copy link
Copy Markdown
Owner

As discussed, I think it's a reasonable change. It's unusual to write data to a table with varying TTLs, but it seems relatively low risk.

My primary concern is that many people are going to expect the behavior to match other compaction strategies - for that reason, I'm interested in perhaps creating a toggle to allow either this behavior OR the old/established behavior.

Do you have a strong opinion on that?

@jivemonkey

Copy link
Copy Markdown
Author

Sounds completely reasonable. I'll make the change.

@jippi

jippi commented May 2, 2016

Copy link
Copy Markdown

is this irrelevant now?

@jeffjirsa

Copy link
Copy Markdown
Owner

@jippi - This is a patch that optimizes TWCS for using varied TTLs in the same Table, and specifically is useful only if you're pushing C*/TWCS so hard that you need to aggressively reclaim disk space as you cross various TTL boundaries. It's not that it's irrelevant, but it's an edge case that doesn't generally apply to most people. I'll merge it when it's cleaned up a bit (or when I have time to clean it up a bit).

@jippi

jippi commented Feb 17, 2017

Copy link
Copy Markdown

@jeffjirsa i guess this can be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants