Review status: 0 of 3 files reviewed at latest revision, 2 unresolved discussions, all commit checks successful.

sql/session.go, line 100 [r1] (raw file):

  // Set up an EventLog for session events.
  ctx = log.WithEventLog(ctx, "sql."+args.User, remoteStr)

nit: "sql.root" didn't look obvious to me that it contained a user name. How about "sql [user]"

sql/session.go, line 181 [r1] (raw file):

  // TODO(andrei): this is the same as Session.Trace. Consider removing this and
  // passing the Session along everywhere the trace is needed.
  tr trace.Trace

good riddance

Comments from Reviewable

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

sql/session.go, line 100 [r1] (raw file):

Comments from Reviewable