Botond Dénes

Also seen in [test_tablet_drain_failure_during_decommission](https://jenkins.scylladb.com//job/scylla-6.0/job/scylla-ci/182/testReport/junit/%28root%29/test_topology_failure_recovery/Tests___Unit_Tests___test_tablet_drain_failure_during_decommission) [🔍](https://github.com/scylladb/scylladb/issues?q=is:issue+is:open+test_tablet_drain_failure_during_decommission). [test_tablet_drain_failure_during_decommission.dev.4.zip](https://github.com/user-attachments/files/16687405/test_tablet_drain_failure_during_decommission.dev.4.zip)

I'm investigating this based on https://github.com/scylladb/scylladb/issues/20060#issuecomment-2301282313. Just as I suspected, we apparently attempt to drop the keyspace twice. We can see the successful (first) attemp here (node 2714): ``` 937...

Looking at where this error is coming from (`replica/table.cc`): ``` 2759 future table::discard_sstables(db_clock::time_point truncated_at) { 1 // truncate_table_on_all_shards() disables compaction for the truncated 2 // tables and views, so we...

https://github.com/scylladb/scylladb/pull/19789 should fix this race, so I'm assigning this issue to @nodep.

Seen again on https://jenkins.scylladb.com/job/scylla-master/job/scylla-ci/11694/.

Seen again in https://jenkins.scylladb.com//job/scylla-master/job/scylla-ci/11956/testReport/junit/%28root%29/test_topology_failure_recovery/Tests___Unit_Tests___test_tablet_drain_failure_during_decommission_dev_5.

We need review from @nyh.

@tchaikov I think this change will break all dtests which work with java tools. How are dtests passing? Note that we are in the process of removing java tools from...

> We keep compatibility for longer than two releases, for the benefit of our sister project Scylla Enterprise How long is this longer right now? Can this PR be merged...

> > > We keep compatibility for longer than two releases, for the benefit of our sister project Scylla Enterprise > > > > > > How long is this...