cockroachdb / cockroach

CockroachDB - the open source, cloud-native distributed SQL database.
https://www.cockroachlabs.com
Other
29.32k stars 3.67k forks source link

rttanalysis: extend framework to track number of descriptors fetched #90960

Open ajwerner opened 1 year ago

ajwerner commented 1 year ago

Is your feature request related to a problem? Please describe.

The rttanalysis framework is helpful because it lets us track round-trips used to run certain queries. Round-trips are not everything. While in multi-region settings, round-trips often dominate the cost of execution, deserializing and validating many descriptors is more expensive. In some cases we can do an index-join to look up a small number of descriptors and that's preferable to scanning all descriptors if the number of descriptors is large (#90116).

Describe the solution you'd like We should also track the number of descriptors which need to be decoded and other metrics related to the cost of an operation.

Jira issue: CRDB-21043

github-actions[bot] commented 1 month ago

We have marked this issue as stale because it has been inactive for 18 months. If this issue is still relevant, removing the stale label or adding a comment will keep it active. Otherwise, we'll close it in 10 days to keep the issue queue tidy. Thank you for your contribution to CockroachDB!