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

Comment-driven query timeouts do not work when shard-targeting #7031

Closed
aquarapid opened this issue Nov 12, 2020 · 0 comments · Fixed by #15898
Closed

Comment-driven query timeouts do not work when shard-targeting #7031

aquarapid opened this issue Nov 12, 2020 · 0 comments · Fixed by #15898
Assignees
Labels
Component: Query Serving Type: Enhancement Logical improvement (somewhere between a bug and feature)

Comments

@aquarapid
Copy link
Contributor

Tested on a current master release (c134e38).

  • Connect to a Vitess instance. Using the 101_initial_cluster.sh local examples in my case.
  • Run a select sleep(1) query with a small timeout (1 ms). See that it gets timed out.
  • Target a specific shard, and run the same query again. Observe that it does not get timed out.
$ mysql -c -u root -P 15306 -A
Welcome to the MySQL monitor.  Commands end with ; or \g.
Your MySQL connection id is 7
Server version: 5.7.9-Vitess Percona Server (GPL), Release 34, Revision 2e68637
.
.
.
mysql> use commerce;
Database changed
mysql> select /*vt+ QUERY_TIMEOUT_MS=1 */ sleep(1);
ERROR 1317 (HY000): vtgate: http://localhost:15001/: target: commerce.0.master, used tablet: zone1-100 (localhost): vttablet: rpc error: code = DeadlineExceeded desc = context deadline exceeded
mysql> use commerce/0;
Database changed
mysql> select /*vt+ QUERY_TIMEOUT_MS=1 */ sleep(1);
+----------+
| sleep(1) |
+----------+
|        0 |
+----------+
1 row in set (1.00 sec)
@aquarapid aquarapid added the Type: Enhancement Logical improvement (somewhere between a bug and feature) label Nov 12, 2020
@ajm188 ajm188 removed the P2 label Mar 9, 2023
@harshit-gangal harshit-gangal self-assigned this May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Query Serving Type: Enhancement Logical improvement (somewhere between a bug and feature)
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants