Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[YCQL] Update metrics for timedout CQL calls #20960

Open
1 task done
amitanandaiyer opened this issue Feb 7, 2024 · 0 comments
Open
1 task done

[YCQL] Update metrics for timedout CQL calls #20960

amitanandaiyer opened this issue Feb 7, 2024 · 0 comments
Assignees
Labels
area/ycql Yugabyte CQL (YCQL) kind/bug This issue is a bug priority/medium Medium priority issue

Comments

@amitanandaiyer
Copy link
Contributor

amitanandaiyer commented Feb 7, 2024

Jira Link: DB-9935

Description

causing timeouts from CQL -> docdb by

dev-server-amitanand2:~/code/yugabyte-db/build [2.21.1.0-b63|✚ 3⚑ 1]
04:25 $ latest/bin/yb-ts-cli -server_address 127.0.0.2:9100 set_flag -force client_read_write_timeout_ms 1
dev-server-amitanand2:~/code/yugabyte-db/build [2.21.1.0-b63|✚ 3⚑ 1]
04:29 $ latest/bin/yb-ts-cli -server_address 127.0.0.1:9100 set_flag -force client_read_write_timeout_ms 1
dev-server-amitanand2:~/code/yugabyte-db/build [2.21.1.0-b63|✚ 3⚑ 1]
04:29 $

does not Update the metrics.

This is likely because we exit early at https://phorge.dev.yugabyte.com/source/yugabyte-db/browse/master/src/yb/yql/cql/ql/exec/executor.cc$1919

We should update the metrics even if the call fails/timesout

Issue Type

kind/bug

Warning: Please confirm that this issue does not contain any sensitive information

  • I confirm this issue does not contain any sensitive information.
@amitanandaiyer amitanandaiyer added area/ycql Yugabyte CQL (YCQL) status/awaiting-triage Issue awaiting triage labels Feb 7, 2024
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels Feb 7, 2024
@amitanandaiyer amitanandaiyer changed the title [YCQL] Update metrics for timeout calls [YCQL] Update metrics for timedout CQL calls Feb 7, 2024
@rthallamko3 rthallamko3 removed the status/awaiting-triage Issue awaiting triage label Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ycql Yugabyte CQL (YCQL) kind/bug This issue is a bug priority/medium Medium priority issue
Projects
None yet
Development

No branches or pull requests

4 participants