-
Notifications
You must be signed in to change notification settings - Fork 4.1k
colexec: audit users of colconv package to make sure they perform memory accounting for datums #67889
Copy link
Copy link
Open
Labels
C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.E-quick-winLikely to be a quick win for someone experienced.Likely to be a quick win for someone experienced.T-sql-queriesSQL Queries TeamSQL Queries Team
Description
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
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.E-quick-winLikely to be a quick win for someone experienced.Likely to be a quick win for someone experienced.T-sql-queriesSQL Queries TeamSQL Queries Team
Type
Projects
Status
Backlog