Skip to content

Commit 1b43146

Browse files
doc: add merge note for printStatistics + GetStat integration
After merging print-statistics, the printStatistics call site should pass s.GetStat() instead of s.GetOutput() so :perftrace redirection applies to performance statistics output.
1 parent ccd2a31 commit 1b43146

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

pkg/sqlcmd/sqlcmd.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -237,7 +237,10 @@ func (s *Sqlcmd) SetError(e io.WriteCloser) {
237237
s.err = e
238238
}
239239

240-
// GetStat returns the io.Writer to use for performance statistics
240+
// GetStat returns the io.Writer to use for performance statistics.
241+
// Falls back to GetOutput() when no :perftrace redirection is active.
242+
// After merging PR #631 (print-statistics), update the printStatistics
243+
// call site from s.GetOutput() to s.GetStat().
241244
func (s *Sqlcmd) GetStat() io.Writer {
242245
if s.stat == nil {
243246
return s.GetOutput()

0 commit comments

Comments
 (0)