k8ssandra / management-api-for-apache-cassandra

RESTful / Secure Management Sidecar for Apache Cassandra
Apache License 2.0
70 stars 51 forks source link

TableOpsResources should not use compactRequest.tables #450

Closed burmanm closed 5 months ago

burmanm commented 5 months ago

We already had a "tables" variable, which was empty ArrayList in case the compactRequest.tables was null. This just wasn't used if tokenRange wasn't set.

Also, add logging in case we get Throwable so it makes debugging easier.

Fixes #437

Tested manually:

➜  cass-operator git:(master) ✗ kubectl get cassandratask                                     
NAME              DATACENTER   JOB       SCHEDULED   STARTED   COMPLETED
example-compact   dc2          compact               41s       31s
➜  cass-operator git:(master) ✗