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

Table snapshot: use C*2.1 pattern #111

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rhardouin
Copy link
Contributor

@rhardouin rhardouin commented Aug 21, 2017

Not compatible with C*2.0

Not compatible with C*2.0
@rhardouin rhardouin force-pushed the table_snap_with_UUID branch from 182151e to bb93b86 Compare August 21, 2017 15:12
@rhardouin
Copy link
Contributor Author

Snapshotting a table doesn't work with C*2.1 because of UUID (it works for whole keyspace).
If we want to handle both 2.0 & 2.1 naming we can drop the -:

table_glob = '%s*' % snapshot_table

But then we can have troubles if a table is a substring of another one e.g.:

users
users_by_email

Let's say we want to backup users only, then we would end up with both tables because of users*.
So we have to make a choice, keep using glob and drop C*2.0 support or scan dirs with a regex.

@tbarbugli
Copy link
Owner

one option would be to make this configurable (eg. --cassandra-2.0)

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.

2 participants