Skip to content

distsql: add memory accounting to mergeJoiner #26896

@asubiotto

Description

@asubiotto

The mergeJoiner accumulates rows with the help of two streamGroupAccumulator. These streamGroupAccumulators copy rows and use a slice that refers to these EncDatumRows. @jordanlewis suggested adding some memory accounting to EncDatumRowAlloc which would be useful in a lot of other places that this is used.

Bonus: add the max allocated memory to the visual query plan stats. Refer to a40b96b for an example

cc @yuzefovich

Metadata

Metadata

Assignees

Labels

A-sql-executionRelating to SQL execution.E-easyEasy issue to tackle, requires little or no CockroachDB experiencegood first issue

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions