Skip to content

sql/stats: don't use a nil eval context when merging statistics #126856

Closed
#127521
Closed
sql/stats: don't use a nil eval context when merging statistics#126856
#127521
Assignees
Uzair5162
Labels
A-sql-table-statsTable statistics (and their automatic refresh).C-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.T-sql-queriesSQL Queries Teambranch-masterFailures and bugs on the master branch.
@Uzair5162

Description

@Uzair5162
Uzair5162
opened on Jul 8, 2024

We shouldn't use a nil eval.Context here:

cockroach/pkg/sql/stats/merge.go

Line 166 in cc1f931

var cmpCtx *eval.Context

since some column types (ex. DTimestampTZ) depend on this for functions like compare().

Jira issue: CRDB-40161

Metadata

Metadata

Assignees

Labels

A-sql-table-statsTable statistics (and their automatic refresh).C-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.T-sql-queriesSQL Queries Teambranch-masterFailures and bugs on the master branch.

Type

No type

Projects

SQL Queries

Status

Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions