[CASSANDRA-15570] Change name of table metrics virtual table percentile columns

• Type: Improvement
• Status: Resolved
• Priority: Low
• Resolution: Fixed
• Fix Version/s: 4.0-alpha4, 4.0
• Component/s: Feature/Virtual Tables
• Labels: pull-request-available
• Change Category: Operability
• Complexity: Low Hanging Fruit
• Platform: All

Source Control Link

https://github.com/apache/cassandra/commit/47a315eaaa2288c78c508682a8961dac65122607

Test and Documentation Plan

na

• Assignee: Chris Lohfink
• Reporter: Chris Lohfink

Authors

Chris Lohfink

Reviewers

David Capwell

• Created: 14/Feb/20 22:32
• Updated: 16/Mar/22 13:05
• Resolved: 26/Feb/20 03:33

Description

The column names 50th and 90th are difficult to query as a selector or in aggregation functions since our parser expects a letter for initial char. It currently requires quoting like:

SELECT table_name, "50th_ms", "99th_ms" FROM system_views.local_read_latency

Which is kinda difficult and non obvious, the error we give when not quoted also isnt that helpful. Since it is percentile a simple thing is to just ad a p infront:

SELECT table_name, p50th_ms, p99th_ms FROM system_views.local_read_latency

Issue Links

• links to: GitHub Pull Request #447