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

[YSQL] New test suite for testing local limit #22819

Closed
5 of 9 tasks
pao214 opened this issue Jun 11, 2024 · 1 comment
Closed
5 of 9 tasks

[YSQL] New test suite for testing local limit #22819

pao214 opened this issue Jun 11, 2024 · 1 comment
Assignees
Labels
area/ysql Yugabyte SQL (YSQL) kind/bug This issue is a bug priority/medium Medium priority issue

Comments

@pao214
Copy link
Contributor

pao214 commented Jun 11, 2024

Jira Link: DB-11716

Description

Motivation

We do not have a lot of tests for local limit and that makes changes to this feature challenging.

Issues Found

  1. [YSQL] Investigate why the first request picks a local limit only when the read time is not set. #22158
  2. [YSQL] Set local limit in a multi page read #22821

Plan

TODO:

  • check local limit with one tablet
  • single tablet multi-page scan
  • check local limit with multi-tablet but read time picked on proxy
  • check local limit with read time picked on another docdb node
  • check local limit after insert
  • local limit must work properly with all kinds of restarts
  • local limit must work properly with RC
  • explicit locking cases

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.
@pao214 pao214 added area/ysql Yugabyte SQL (YSQL) status/awaiting-triage Issue awaiting triage labels Jun 11, 2024
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels Jun 11, 2024
@pao214 pao214 changed the title [YSQL] New test framework for testing local limit [YSQL] New test suite for testing local limit Jun 11, 2024
@pao214 pao214 self-assigned this Jun 11, 2024
@pao214
Copy link
Contributor Author

pao214 commented Jul 20, 2024

Resolved by #23076

@pao214 pao214 closed this as completed Jul 20, 2024
@yugabyte-ci yugabyte-ci removed the status/awaiting-triage Issue awaiting triage label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ysql Yugabyte SQL (YSQL) kind/bug This issue is a bug priority/medium Medium priority issue
Projects
Status: Done
Development

No branches or pull requests

2 participants