This change is 

did you confirm that this fixes the issue?

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks pending.

pkg/ccl/importccl/csv.go, line 711 at r1 (raw file):

		count += int64(len(kvBatch))
		for _, kv := range kvBatch {
			if err := writer.Put(kv.Key, kv.Value.RawBytes); err != nil {

Adding the call to InitChecksum would be a bit more natural here, but it's up to you.

Comments from Reviewable

Yes, the test I added produces the error in the bug.

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks pending.

pkg/ccl/importccl/csv.go, line 711 at r1 (raw file):

Sadly this function (writeRocksDB) is only called in the local version of import, so adding it here would also mean adding it in a second place for the distributed version. The place where it is now is best because that's the latest point before the implementations diverge.

Comments from Reviewable

How does this interact with #24126? Do we still need this for mixed-version clusters or is it obsolete?

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

#24126 will fix #23984 for any 2.0 clusters, but the change here is still a good idea for mixed-version clusters where some of the nodes may require checksums.

I'm not opposed to closing this PR if #24126 is a sufficient fix. Will that make it into 2.0 or 2.0.1? We could choose to not merge this PR since it'd just be a performance decrease for no value.

We should merge and cherry-pick this (for 2.0.1, I think) since it will help for mixed-version clusters. Then we can remove the checksum computation at the same time we remove all the others.