cockroachdb / cockroachdb/cockroach

lint,build: break dependency on `pkg/sql/sem/builtins` from `pkg/testutils/lint`

Open
#123,255 0 comments 0 reactions 0 assignees View on GitHub
A-build-system C-enhancement T-dev-inf
Dominant language
Go
Stars
32.5k
Forks
4.1k
PR merge metrics
PR metrics pending

Description

`pkg/sql/sem/builtins` is a heavy package with a lot of dependencies and `lint_test.go` only needs it for `builtins.AllBuiltinNames()` and `builtins.CastBuiltinName`. It would be a big improvement to the build time for this test if we didn't need this dependency, though I'm not sure right now how to do this.

Epic: CRDB-17171

Jira issue: CRDB-38260

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.