[ https://issues.apache.org/jira/browse/CASSANDRA-15570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17038579#comment-17038579 ]
David Capwell commented on CASSANDRA-15570: ------------------------------------------- Here is the behavior on trunk; the last query made sure to copy/paste the column name from the output. {code} cqlsh> select * from system_views.local_read_latency; keyspace_name | table_name | 50th_ms | 99th_ms | count | max_ms | per_second --------------------+--------------------------------+---------+---------+-------+---------+------------ schema1 | users | 0 | 0 | 0 | 0 | 0 system | IndexInfo | 0 | 0 | 0 | 0 | 0 system | available_ranges | 0 | 0 | 0 | 0 | 0 system | available_ranges_v2 | 0 | 0 | 0 | 0 | 0 system | batches | 0 | 0 | 0 | 0 | 0 system | built_views | 0 | 0 | 0 | 0 | 0 system | compaction_history | 0 | 0 | 0 | 0 | 0 system | local | 1.63 | 129.558 | 28 | 129.558 | 5.036 system | paxos | 0 | 0 | 0 | 0 | 0 system | peer_events | 0 | 0 | 0 | 0 | 0 system | peer_events_v2 | 0 | 0 | 0 | 0 | 0 system | peers | 0 | 0 | 0 | 0 | 0 system | peers_v2 | 0 | 0 | 0 | 0 | 0 system | prepared_statements | 0 | 0 | 0 | 0 | 0 system | repairs | 0 | 0 | 0 | 0 | 0 system | size_estimates | 0 | 0 | 0 | 0 | 0 system | sstable_activity | 0.219 | 0.316 | 2 | 0.316 | 0.387 system | transferred_ranges | 0 | 0 | 0 | 0 | 0 system | transferred_ranges_v2 | 0 | 0 | 0 | 0 | 0 system | view_builds_in_progress | 0 | 0 | 0 | 0 | 0 system_auth | network_permissions | 0 | 0 | 0 | 0 | 0 system_auth | resource_role_permissons_index | 0 | 0 | 0 | 0 | 0 system_auth | role_members | 0 | 0 | 0 | 0 | 0 system_auth | role_permissions | 0 | 0 | 0 | 0 | 0 system_auth | roles | 1.358 | 1.358 | 1 | 1.358 | 0 system_distributed | parent_repair_history | 0 | 0 | 0 | 0 | 0 system_distributed | repair_history | 0 | 0 | 0 | 0 | 0 system_distributed | view_build_status | 0 | 0 | 0 | 0 | 0 system_schema | aggregates | 0.061 | 0.127 | 4 | 0.127 | 0.774 system_schema | columns | 0.786 | 1.132 | 11 | 1.132 | 2.128 system_schema | dropped_columns | 0.073 | 0.127 | 11 | 0.127 | 2.128 system_schema | functions | 0.073 | 0.152 | 4 | 0.152 | 0.774 system_schema | indexes | 0.061 | 0.127 | 11 | 0.127 | 2.128 system_schema | keyspaces | 0.455 | 1.358 | 4 | 1.358 | 0.774 system_schema | tables | 0.379 | 3.379 | 18 | 3.379 | 3.482 system_schema | triggers | 0.088 | 0.219 | 11 | 0.219 | 2.128 system_schema | types | 0.073 | 0.655 | 4 | 0.655 | 0.774 system_schema | views | 0.088 | 0.127 | 4 | 0.127 | 0.774 system_traces | events | 0 | 0 | 0 | 0 | 0 system_traces | sessions | 0 | 0 | 0 | 0 | 0 (40 rows) cqlsh> cqlsh> select 50th_ms from system_views.local_read_latency; SyntaxException: line 1:9 mismatched input 'th_ms' expecting K_FROM (select 50[th_ms] from...) cqlsh> select "50th_ms" from system_views.local_read_latency; 50th_ms --------- 0 0 0 0 0 0 0 1.63 0 0 0 0 0 0 0 0 0.219 0 0 0 0 0 0 0 1.358 0 0 0 0.061 0.655 0.073 0.073 0.061 0.455 0.455 0.073 0.073 0.088 0 0 (40 rows) {code} With the patch {code} cqlsh> select * from system_views.local_read_latency; keyspace_name | table_name | count | max_ms | p50th_ms | p99th_ms | per_second --------------------+--------------------------------+-------+--------+----------+----------+------------ system | IndexInfo | 0 | 0 | 0 | 0 | 0 system | available_ranges | 0 | 0 | 0 | 0 | 0 system | available_ranges_v2 | 0 | 0 | 0 | 0 | 0 system | batches | 0 | 0 | 0 | 0 | 0 system | built_views | 0 | 0 | 0 | 0 | 0 system | compaction_history | 0 | 0 | 0 | 0 | 0 system | local | 26 | 36.157 | 1.132 | 36.157 | 3.683 system | paxos | 0 | 0 | 0 | 0 | 0 system | peer_events | 0 | 0 | 0 | 0 | 0 system | peer_events_v2 | 0 | 0 | 0 | 0 | 0 system | peers | 0 | 0 | 0 | 0 | 0 system | peers_v2 | 0 | 0 | 0 | 0 | 0 system | prepared_statements | 0 | 0 | 0 | 0 | 0 system | repairs | 0 | 0 | 0 | 0 | 0 system | size_estimates | 0 | 0 | 0 | 0 | 0 system | sstable_activity | 0 | 0 | 0 | 0 | 0 system | transferred_ranges | 0 | 0 | 0 | 0 | 0 system | transferred_ranges_v2 | 0 | 0 | 0 | 0 | 0 system | view_builds_in_progress | 0 | 0 | 0 | 0 | 0 system_auth | network_permissions | 0 | 0 | 0 | 0 | 0 system_auth | resource_role_permissons_index | 0 | 0 | 0 | 0 | 0 system_auth | role_members | 0 | 0 | 0 | 0 | 0 system_auth | role_permissions | 0 | 0 | 0 | 0 | 0 system_auth | roles | 2 | 0 | 0 | 0 | 0.005 system_distributed | parent_repair_history | 0 | 0 | 0 | 0 | 0 system_distributed | repair_history | 0 | 0 | 0 | 0 | 0 system_distributed | view_build_status | 0 | 0 | 0 | 0 | 0 system_schema | aggregates | 3 | 0 | 0 | 0 | 0.46 system_schema | columns | 10 | 0.786 | 0.379 | 0.786 | 1.532 system_schema | dropped_columns | 10 | 0.073 | 0.061 | 0.073 | 1.532 system_schema | functions | 3 | 0 | 0 | 0 | 0.46 system_schema | indexes | 10 | 0.152 | 0.073 | 0.152 | 1.532 system_schema | keyspaces | 6 | 0 | 0 | 0 | 0.919 system_schema | tables | 16 | 0.379 | 0.316 | 0.379 | 2.451 system_schema | triggers | 10 | 0.088 | 0.051 | 0.088 | 1.532 system_schema | types | 3 | 0 | 0 | 0 | 0.46 system_schema | views | 3 | 0.051 | 0.051 | 0.051 | 0.46 system_traces | events | 0 | 0 | 0 | 0 | 0 system_traces | sessions | 0 | 0 | 0 | 0 | 0 (39 rows) cqlsh> select p50th_ms, p99th_ms from system_views.local_read_latency; p50th_ms | p99th_ms ----------+---------- 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 1.132 | 36.157 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0 | 0 0.379 | 0.786 0.061 | 0.073 0 | 0 0.073 | 0.152 0 | 0 0.316 | 0.379 0.051 | 0.088 0 | 0 0.051 | 0.051 0 | 0 0 | 0 (39 rows) {code} > Change name of table metrics virtual table percentile columns > ------------------------------------------------------------- > > Key: CASSANDRA-15570 > URL: https://issues.apache.org/jira/browse/CASSANDRA-15570 > Project: Cassandra > Issue Type: Improvement > Components: Feature/Virtual Tables > Reporter: Chris Lohfink > Assignee: Chris Lohfink > Priority: Low > Labels: pull-request-available > Time Spent: 10m > Remaining Estimate: 0h > > 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: > {code:sql} > SELECT table_name, "50th_ms", "99th_ms" FROM system_views.local_read_latency > {code} > 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: > {code:sql} > SELECT table_name, p50th_ms, p99th_ms FROM system_views.local_read_latency > {code} -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org