Re: Add Snapshot Expiry via TTL to NodeTool

2021-08-17 Thread Stefan Miklosovic
Hi Jack, looking into this more deeply, I think that it should not be its own command. It should be a subfeature of already nodetool clearsnapshot. Something like: nodetool clearsnapshot --older-than 30d it will be much easier with our work since we will have the flag "createdAt" exposed in the

Re: Add Snapshot Expiry via TTL to NodeTool

2021-08-17 Thread Stefan Miklosovic
Hi Jack, we are already working on this under (1), the branch is here (2) This improvement is done under Google Summer of Code programme where Abi Palagashvili is assigned to it already. We are about to finish this feature more or less this week. Would you mind going over this work and raising

Add Snapshot Expiry via TTL to NodeTool

2021-08-17 Thread Jack Casey
Hello! I'm a new contributor to the project, and I have just opened my first issue / PR. I've included the links for reference. More detail on the change itself available in the issue / PR: https://issues.apache.org/jira/browse/CASSANDRA-16860 https://github.com/apache/cassandra/pull/1148 I was