-
Notifications
You must be signed in to change notification settings - Fork 4k
Closed
Labels
A-sql-executionRelating to SQL execution.Relating to SQL execution.E-easyEasy issue to tackle, requires little or no CockroachDB experienceEasy issue to tackle, requires little or no CockroachDB experiencegood first issue
Description
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.Relating to SQL execution.E-easyEasy issue to tackle, requires little or no CockroachDB experienceEasy issue to tackle, requires little or no CockroachDB experiencegood first issue