Reviewed 4 of 4 files at r1, 1 of 1 files at r2, 5 of 5 files at r3.
Review status: all files reviewed at latest revision, 2 unresolved discussions, all commit checks successful.

pkg/sql/session.go, line 72 at r2 (raw file):

	//

	// Database indicates the "current" database for the purpose

Is it worth mentioning how this interacts with the session search path?

pkg/sql/session.go, line 100 at r2 (raw file):

	TxnState txnState
	// PreparedStatement and PreparedPortals store the statements/portals
	// that have been configured via pgwire.

PreparedStatement should be plural, also maybe just say "prepared" via pgwire?

Comments from Reviewable

TFYR

Review status: 1 of 6 files reviewed at latest revision, 2 unresolved discussions.

pkg/sql/session.go, line 72 at r2 (raw file):

Done.

pkg/sql/session.go, line 100 at r2 (raw file):

Done.

Comments from Reviewable