Yahor Yuzefovich

colmem: improve memory-limiting behavior of the accounting helpers

13

**colmem: introduce a helper method when no memory limit should be applied** This commit is a pure mechanical change. Release note: None **colmem: move some logic of capacity-limiting into the...

lint: add a linter to prohibit map[...]bool in favor of map[...]struct{}

3

The linter is currently limited to `sql/opt/norm` package. This commit was prompted by an article mentioned in the Golang Weekly newsletter. The rationale is that `map[...]struct{}` is more efficient, but...

sql: race in TestCancelMultipleQueued under stress

3

I hit a [race](https://teamcity.cockroachdb.com/viewLog.html?buildId=6045909&buildTypeId=Cockroach_UnitTests_Testrace&tab=artifacts&branch_Cockroach_UnitTests=%3Cdefault%3E#%2FlogTestCancelMultipleQueued3416152674;%2Fraw.0.json.txt.tgz) on one of my branches, but I don't think that my change (#81995) is to blame. This was on c1d005cd7ddf6a52305a211f79a1b80f9317cca8. ``` ================== WARNING: DATA RACE Write...

flowinfra: hydrate types for the inbound streams

1

The inbound stream handler receives the type information as metadata (unlike other components of the row-based flows), and we forgot to do the hydration of those types. The impact is...

It seems to me that lately (like the last week or two) it takes longer to load TeamCity webpages (especially when clicking on some “initial” link and not navigating team...

sql: retry validation of unique constraint

3

Previously, when running the validation of unique constraint we would execute the query only once, and if that fails for any reason, we'd return an error. This was suboptimal because...

persistedsqlstats: speed up a test

2

Previously, a single unit test could take on the order of 4 minutes (or even exceed 5 minute timeout, rarely) because the job monitor checks whether a cluster setting has...

#86829 added the regions information into the telemetry. In order to minimize the changes and to make the commit backportable it relied on the existing infrastructure used for the `EXPLAIN`...

sql: internal error: lookup join with no lookup columns

5

When running SQLite logic tests with `--crdb_test` flag (which is currently not added in CI due to #58089) with ``` ./dev testlogic sqlite --ignore-cache --config=local --files=slt_good_22 -- --test_env=COCKROACH_RANDOM_SEED='-6676986672122001987' ``` ```...

rowcontainer: switch to using Fingerprint for in-memory hashing

1

This commit makes it so that we use value encoding for TSQuery and TSVector types in the in-memory hash row container. Previously, we would unconditionally use key encoding and would...