Skip to content

Commit

Permalink
Merge #44925
Browse files Browse the repository at this point in the history
44925:  batcheval: plumb TargetBytes/NumBytes into {Reverse,}Scan r=nvanbenschoten a=tbg

This hooks these new fields up to their already present counterparts in
the MVCC layer. This means that the KV server will now handle these
correctly.

The missing piece is to teach DistSender to keep track of the total
NumBytes seen across all ranges queried for the current batch, and to
terminate scans that have finished. This (and testing it) is left to a
follow-up PR.

Release note: None

Co-authored-by: Tobias Schottdorf <[email protected]>
  • Loading branch information
craig[bot] and tbg committed Feb 11, 2020
2 parents cdf805f + 8149d12 commit 82b341a
Show file tree
Hide file tree
Showing 7 changed files with 910 additions and 613 deletions.
86 changes: 74 additions & 12 deletions c-deps/libroach/protos/roachpb/api.pb.cc

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

42 changes: 42 additions & 0 deletions c-deps/libroach/protos/roachpb/api.pb.h

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 82b341a

Please sign in to comment.