Merge cockroachdb#25307

25307: release-2.0: importccl: check node health and compatibility during IMPORT planning r=mjibson a=mjibson

Backport 1/1 commits from cockroachdb#25162.

/cc @cockroachdb/release

---

Simplify the LoadCSV signature by taking just a PlanHookState for any
argument that can be fetched from it. Determine the node list using
this new health check function. We can remove the rand.Shuffle call
because the map iteration should produce some level of randomness.

Fixes cockroachdb#12876

Release note (bug fix): Fix problems with imports sometimes failing
after node decommissioning.


Co-authored-by: Matt Jibson <[email protected]>

Merge cockroachdb#25126

25126: opt: support for EXPLAIN r=RaduBerinde a=RaduBerinde

#### opt: introduce and build ExplainOp

The ExplainOp must remember the physical props required by the
underlying query.

For now we only support `EXPLAIN (VERBOSE)`.

Release note: None

#### opt: execbuild support for EXPLAIN

Release note: None

#### opt: remove exec-explain

Remove the `exec-explain` directive and convert all the tests to exec
an EXPLAIN statement. Add `hide-columns` option to suppress the header
with the column names and types, for cleaner output (and to avoid a
ton of diffs in this change).

Release note: None


Co-authored-by: Radu Berinde <[email protected]>

Merge cockroachdb#25126

25126: opt: support for EXPLAIN r=RaduBerinde a=RaduBerinde

#### opt: introduce and build ExplainOp

The ExplainOp must remember the physical props required by the
underlying query.

For now we only support `EXPLAIN (VERBOSE)`.

Release note: None

#### opt: execbuild support for EXPLAIN

Release note: None

#### opt: remove exec-explain

Remove the `exec-explain` directive and convert all the tests to exec
an EXPLAIN statement. Add `hide-columns` option to suppress the header
with the column names and types, for cleaner output (and to avoid a
ton of diffs in this change).

Release note: None


Co-authored-by: Radu Berinde <[email protected]>

Merge cockroachdb#24889

24889: cherrypick-1.1: build: Update etcd r=bdarnell a=bdarnell

Picks up a cherry-picked version of etcd-io/etcd#9073, to fix cockroachdb#18601

Release note (bug fix): Fixes potential cluster unavailability after
raft logs grow too large.

Co-authored-by: Ben Darnell <[email protected]>

Merge cockroachdb#24825

24825: cherrypick-2.0: distsql: clean up lookup join internal columns r=pbardea a=pbardea

Cherry-picks cockroachdb#24697 and makes a few changes from the original PR:
- passing `nil` as the flowCtx in`processorBase.init` call in joinreader.go
- the output of the `EXPLAIN (VERBOSE)` test in the `lookup_join` logic test

Fixes cockroachdb#24401.

The distsql physical planner was not properly forming the post processor
spec according to the internal columns of lookup joins.

For the hash and merge joiners, the internal columns depend on the
streamToColMap of its upstream nodes. However since the lookup join's
internal columns consist of the entire right side of table columns, the
planner must modify the post processor spec and various maps.

This simplifies code inside the lookup joiner.


Release note (bug fix): Fix a bug causing some lookup join queries to
report incorrect type errors.

Co-authored-by: Paul Bardea <[email protected]>

Merge cockroachdb#24594

24594: ts: Initial TimeSeries TestModel. r=mrtracy a=mrtracy

Creates a new "testmodel" package under "ts" which contains an in-memory
test model designed to be much more conceptually simple than the
optimized query engine actually used by time series.

The new test model stores time series in sorted arrays as full-fidelity
nanosecond data points. It then uses a highly decomposed set of
functions to process that data; these functions have been designed to be
individually tested.

The "Query" method shows an example of how simple this system is when
compared to the actual time series system, which uses iterators and
breaks up queries into multiple chunks.

This model has not yet been hooked up to time series tests; it will soon
be used to replace the existing test model, which is *not* easier to
understand than the actual time series database.

Co-authored-by: Matt Tracy <[email protected]>

Merge pull request cockroachdb#24458 from petermattis/pmattis/2.0-dum…

…my-commit

cli: remove a superfluous TODO

Merge pull request cockroachdb#24393 from mberhault/backport2.0-24390

backport-2.0: sql: fill in required privileges at Validate time.

makefile: add a missing variable

... causing the "S3 build binaries" build to fail.

Release note: None

Merge pull request cockroachdb#24073 from a-robinson/backport1.1-23851

cherrypick-1.1: server: Use remote_debugging setting to restrict all access to keys