[CASSANDRA-17023] Eliminate toCQLString() duplication between ReadCommand and AbstractReadQuery

• Type: Task
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 4.1-alpha1, 4.1
• Component/s: CQL/Syntax
• Change Category: Code Clarity
• Complexity: Low Hanging Fruit
• Platform: All

Source Control Link

https://github.com/apache/cassandra/commit/60e0da1bd0f8271f7bbba300695d67b9ad0a497f

Test and Documentation Plan

https://github.com/apache/cassandra/pull/1282/files

• Assignee: Kanthi Subramanian
• Reporter: Caleb Rackliffe

Authors

Kanthi Subramanian

Reviewers

Brandon Williams, Caleb Rackliffe

• Tester: Caleb Rackliffe
• Created: 05/Oct/21 20:56
• Updated: 27/May/22 19:24
• Resolved: 25/Oct/21 14:24

Description

This is certainly low-hanging fruit, but the toCQLString() in ReadCommand now entirely duplicates the one in its superclass AbstractReadQuery. I think we can just remove the former.