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

[DocDB] Infinite loop detected error with system keys in reverse scan #15515

Closed
arpang opened this issue Jan 5, 2023 · 0 comments
Closed

[DocDB] Infinite loop detected error with system keys in reverse scan #15515

arpang opened this issue Jan 5, 2023 · 0 comments
Assignees
Labels
area/docdb YugabyteDB core features kind/bug This issue is a bug priority/medium Medium priority issue

Comments

@arpang
Copy link
Contributor

arpang commented Jan 5, 2023

Jira Link: DB-4907

Description

We hit "infinite loop detected" error when system keys are encountered in a reverse scan. The bug is that in reverse scan, instead of seeking to the previous DocKey, it seeks to the next one causing an infinite loop.

Internal reference: https://yugabyte.slack.com/archives/C03ULJYE0KG/p1672838276448609

@arpang arpang added area/docdb YugabyteDB core features status/awaiting-triage Issue awaiting triage labels Jan 5, 2023
@arpang arpang self-assigned this Jan 5, 2023
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels Jan 5, 2023
@arpang arpang removed the status/awaiting-triage Issue awaiting triage label Jan 5, 2023
arpang added a commit that referenced this issue Jan 5, 2023
Summary: This diff fixes the infinite loop detected error which occurs when system keys are encountered in a reverse scan. The bug was that in reverse scan, instead of seeking to the previous DocKey, it was seeking to the next one causing an infinite loop.

Test Plan: ./yb_build.sh --skip-java --cxx-test docrowwiseiterator-test --gtest-filter=DocRowwiseIteratorTest.ClusteredFilterRangeWithTableTombstoneReverseScan

Reviewers: kpopali, skumar, tnayak

Reviewed By: tnayak

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D22098
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docdb YugabyteDB core features kind/bug This issue is a bug priority/medium Medium priority issue
Projects
None yet
Development

No branches or pull requests

3 participants