add review comments

Author: zhangchiqing

storage/operation/badgerimpl/reader.go

@@ -53,6 +53,8 @@ func (b dbReader) Get(key []byte) ([]byte, io.Closer, error) {
 //   - have a prefix equal to startPrefix OR
 //   - have a prefix equal to the endPrefix OR
 //   - have a prefix that is lexicographically between startPrefix and endPrefix
+//
+// no errors are expected during normal operation
 func (b dbReader) NewIter(startPrefix, endPrefix []byte, ops storage.IteratorOption) (storage.Iterator, error) {
 	return newBadgerIterator(b.db, startPrefix, endPrefix, ops), nil
 }

storage/operation/badgerimpl/writer.go

@@ -20,7 +20,7 @@ type ReaderBatchWriter struct {
 var _ storage.ReaderBatchWriter = (*ReaderBatchWriter)(nil)
 
 // GlobalReader returns a database-backed reader which reads the latest committed global database state ("read-committed isolation").
-// This reader will not read writes written to ReaderBatchWriter.Writer until the write batch is committed.
+// This reader will not read un-committed writes written to ReaderBatchWriter.Writer until the write batch is committed.
 // This reader may observe different values for the same key on subsequent reads.
 func (b *ReaderBatchWriter) GlobalReader() storage.Reader {
 	return b.globalReader