Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cassandra 2.1 #1

Open
wants to merge 5 commits into
base: cassandra-2.1
Choose a base branch
from

Conversation

jivemonkey
Copy link

@jeffjirsa how does this look?

@jeffjirsa
Copy link
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
Author

Sounds completely reasonable. I'll make the change.

@jippi
Copy link

jippi commented May 2, 2016

is this irrelevant now?

@jeffjirsa
Copy link
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
Copy link

jippi commented Feb 17, 2017

@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