Reviewed 3 of 3 files at r1.
Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @jbowens, @petermattis, and @sumeerbhola)

iterator.go, line 132 at r1 (raw file):

	prefixOrFullSeekKey []byte
	readSampling        readSampling
	stats IteratorStats

Nit: align types

iterator.go, line 1174 at r1 (raw file):

// ResetStats resets the stats to 0.
func (i *Iterator) ResetStats() {

Is this called anywhere?

iterator.go, line 1237 at r1 (raw file):

func (stats *IteratorStats) SafeFormat(s redact.SafePrinter, verb rune) {
	for i := range stats.ForwardStepCount {
		if i == 0 {

Nit: i == InterfaceCall ?

iterator.go, line 1240 at r1 (raw file):

			s.SafeString("interface: ")
		} else {
			s.SafeString(" internal: ")

Wonder if we could do better spacing / separation before internal, right now it seems to be a continuation of the interface stats.

TFTR!

Reviewable status: 1 of 3 files reviewed, 2 unresolved discussions (waiting on @itsbilal, @jbowens, and @petermattis)

iterator.go, line 132 at r1 (raw file):

Done. Odd that goland did not do it automatically.

iterator.go, line 1174 at r1 (raw file):

Not yet. It will be called by pebbleIterator.Close when reusing the underlying iterator.

iterator.go, line 1237 at r1 (raw file):

Done

iterator.go, line 1240 at r1 (raw file):

I've adjusted this to add another pair of parentheses and cut out some repetition of literals (I was not happy with the lengthiness).

Reviewed 2 of 2 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @jbowens and @petermattis)