Reviewed 7 of 7 files at r1, 4 of 4 files at r2, 2 of 2 files at r3.
Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks broke.

pkg/kv/batch_test.go, line 102 at r3 (raw file):

			t.Errorf("got %s, expected %s", rk, leftKey)
		}
		rk, err = prev(ba, leftKey)

looks like something got truncated here, and there's a spurious empty line below

pkg/roachpb/gen_batch.go, line 54 at r1 (raw file):

		field := t.Field(i).Name
		info.name = field
		// The ResponseUnion fields match those in RequestUnion, with one exception.

"the following exceptions:" so the next person doesn't have to edit this again

Comments from Reviewable

Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks broke.

Comments from Reviewable

Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks broke.

pkg/kv/batch_test.go, line 102 at r3 (raw file):

Removed this line and the blank one

pkg/roachpb/gen_batch.go, line 54 at r1 (raw file):

Done.

Comments from Reviewable

Reviewed 1 of 7 files at r4, 4 of 4 files at r5, 2 of 2 files at r6.
Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable