Skip to content

colexec: audit users of colconv package to make sure they perform memory accounting for datums #67889

@yuzefovich

Description

@yuzefovich

Miscellaneous methods from colconv package perform the conversion from a natively supported value to its tree.Datum counterpart. The methods themselves don't account for the created datums and push the responsibility onto the callers. However, I believe no caller currently is actually performing the memory accounting (unless the datums are put into the datum vector which is then properly accounted for). We should audit all call sites.

Jira issue: CRDB-8764

Metadata

Metadata

Assignees

No one assigned

    Labels

    C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.E-quick-winLikely to be a quick win for someone experienced.T-sql-queriesSQL Queries Team

    Type

    No type

    Projects

    Status

    Backlog

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions