Skip to content

kv: finish up impl of TargetBytes scan limit#45319

Closed
tbg wants to merge 1 commit intocockroachdb:masterfrom
tbg:kvfetcher-limit
Closed

kv: finish up impl of TargetBytes scan limit#45319
tbg wants to merge 1 commit intocockroachdb:masterfrom
tbg:kvfetcher-limit

Conversation

@tbg
Copy link
Copy Markdown
Member

@tbg tbg commented Feb 24, 2020

This PR finishes up the work initiated in #44925 to allow (forward and
reverse) scans to specify a TargetBytes hint which (mod overshooting by
one row) restricts the size of the responses.

The plan is to use it in kvfetcher, however this is left as a separate
commit - here we focus on testing the functionality only.

Release note: None

This PR finishes up the work initiated in cockroachdb#44925 to allow (forward and
reverse) scans to specify a TargetBytes hint which (mod overshooting by
one row) restricts the size of the responses.

The plan is to use it in kvfetcher, however this is left as a separate
commit - here we focus on testing the functionality only.

Release note: None
@tbg tbg requested a review from nvb February 24, 2020 14:09
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@tbg
Copy link
Copy Markdown
Member Author

tbg commented Feb 24, 2020

Nevermind, might as well flip the bit: #45323

@tbg tbg closed this Feb 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants