[CASSANDRASC-129] Remove tableId from list snapshot response

• Type: Bug
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 1.0
• Component/s: Rest API
• Labels: pull-request-available
• Bug Category: Correctness - Unrecoverable Corruption / Loss
• Severity: Normal
• Complexity: Low Hanging Fruit
• Discovered By: User Report

Source Control Link

https://github.com/apache/cassandra-sidecar/commit/e95786a077e1137dcaae206854986987edc6a71e

Test and Documentation Plan

Added tests for client compatibility. Updated existing tests, and tested new utils methods

• Assignee: Francisco Guerrero
• Reporter: Francisco Guerrero

Authors

Francisco Guerrero

Reviewers

Francisco Guerrero, Yifan Cai

• Created: 08/May/24 23:26
• Updated: 09/May/24 01:04
• Resolved: 09/May/24 01:04

Description

In CASSANDRASC-94, a new field tableId was added to the payload of the list snapshots endpoint. Old Sidecar clients are compatible with new fields added from the server side. However, other clients can potentially break and for that reason we should remove the tableId field from the response and make any potential breaking changes with an API version change instead. As an alternative, we can add the tableId as part of the tableName field instead.

Issue Links

• links to: GitHub Pull Request #120