This package was extracted out of pkg/settings/cluster in cockroachdb#45455. Now
that we're coupling it tightly with our migrations infrastructure
(pkg/migration), let's remove some unnecessary stuttering for ease of
use.

```diff
- if <settings handle>.Version.IsActive(ctx, clusterversion.VersionSomething) {
+ if <settings handle>.Version.IsActive(ctx, clusterversion.Something) {
```

All the diffs here are just mechanical goland-renames.

Release note: None

This package was extracted out of pkg/settings/cluster in cockroachdb#45455. Now
that we're coupling it tightly with our migrations infrastructure
(pkg/migration), let's remove some unnecessary stuttering for ease of
use.

```diff
- if <settings handle>.Version.IsActive(ctx, clusterversion.VersionSomething) {
+ if <settings handle>.Version.IsActive(ctx, clusterversion.Something) {
```

All the diffs here are just mechanical goland-renames.

Release note: None

57306: sqlproxyccl: close proxy connections properly r=jbowens a=spaskob

Issue:
The sqlproxy opens a connection to the client and backend and proxies
data between them. The backend connection is never closed. Moreover the
client connection in certain situations may not be closed properly as
well because it may be embedded in a tls connection whose `Close`
method will never be called.

Details:
Proxy connection to the client is closed in `func (s *Server) Serve`
https://github.com/cockroachdb/cockroach/blob/master/pkg/ccl/sqlproxyccl/server.go#L149
However the client connection, if the proxy is using TLS, is further
embedded in `tls.Conn` in func (s *Server) Proxy here
https://github.com/cockroachdb/cockroach/blob/master/pkg/ccl/sqlproxyccl/proxy.go#L117.
`tls.Conn` has a more involved `Close` implementation which will not be
called; see https://golang.org/src/crypto/tls/conn.go. In particular,
readers/writers on this connection may not be notified of the connection
closure potentially leading to never exiting the go routines at
https://github.com/cockroachdb/cockroach/blob/master/pkg/ccl/sqlproxyccl/proxy.go#L204.

Note that this fix does not prevent another party from calling `Close`
on `proxyConn` which may lead again to `tls.Conn Close` not being
called when then proxy is using TLS.

Release note: none.

57318: colexecagg: fix memory accounting for decimals with avg and sum r=yuzefovich a=yuzefovich

Previously, we were not calling `PerformOperation` to account for the
memory used by the decimals (which are of variable size), and this is
now fixed. Additionally, this commit refactors several other templates to
be more similar to each other.

Note that we decided to call `PerformOperation` on all of the aggregate
functions even if we know that all types that the functions operate on
are of fixed size and that call is not strictly necessary. However, this
has negligible impact on performance in most cases and makes the layout
of `Compute` methods of all functions the same ensuring that we don't
forget these calls in the future.

Additionally, this commit adds capturing of the column to force bounds
check to work for min/max and sum aggregates.

Release note: None

57328: ccl, sql: add telemetry, logs for feature denials by feature flags r=otan a=angelapwen

This change adds a telemetry counter, sql.feature_flag.denied, in
the case where a feature is denied by its corresponding feature flag
being set to disabled in cluster settings. It also adds a log
warning for the database administrator to view the denied command
as well as its optional category.

Release note: none

57338: clusterversion: reduce stuttering in package API r=irfansharif a=irfansharif

This package was extracted out of pkg/settings/cluster in #45455. Now
that we're coupling it tightly with our migrations infrastructure
(pkg/migration), let's remove some unnecessary stuttering for ease of
use.

```diff
- if <settings handle>.Version.IsActive(ctx, clusterversion.VersionSomething) {
+ if <settings handle>.Version.IsActive(ctx, clusterversion.Something) {
```

All the diffs here are just mechanical goland-renames.

Release note: None

57349: colexec: fix jsonb - string projection r=yuzefovich a=yuzefovich

Bytes canonical type family represents several types, and whenever we're
performing a binary operation with one datum and one non-datum arguments
we need to convert the latter to the correct datum type. Previously, we
had an incorrect conversion for Minus operation.

Fixes: #57165.

Release note (bug fix): Previously, CockroachDB would encounter an
internal error when performing `JSONB - String` operation via the
vectorized execution engine, and this has been fixed. The bug was
introduced in 20.2.0 version.

Co-authored-by: Spas Bojanov <spas@cockroachlabs.com>
Co-authored-by: Yahor Yuzefovich <yahor@cockroachlabs.com>
Co-authored-by: angelapwen <angelaw@cockroachlabs.com>
Co-authored-by: irfan sharif <irfanmahmoudsharif@gmail.com>