cassandra-medusa
cassandra-medusa copied to clipboard
Make node_backup and cluster_backup classes pojo-like
There's a rather ugly way we treat attributes of those classes. For example
@manifest.setter
def manifest(self, manifest):
self.cached_manifest = None
self._storage.storage_driver.upload_blob_from_string(self.manifest_path, manifest)
Elsewhere in the code, we then just do manifest = something
, which is nice, but it's also not obvious this is a call to the storage backend. Now that we are more aware of the backend connection and try to manage it nicely, it becomes obvjous that these hidden interactions are not nice. Mostly, because they prevent us from closing the connections cleanly.
For the refactoring week, I'd like to go see if we can actually move away form this entirely. It'll give us a cleaner and more maintainable code.
┆Issue is synchronized with this Jira Story by Unito ┆Issue Number: MED-24