Skip to content

Commit

Permalink
fix formatters
Browse files Browse the repository at this point in the history
  • Loading branch information
max-hoffman committed Mar 4, 2025
1 parent 2ed4b57 commit 6a6f861
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions sql/analyzer/costed_index_scan.go
Original file line number Diff line number Diff line change
Expand Up @@ -138,8 +138,8 @@ func costedIndexLookup(ctx *sql.Context, n sql.Node, a *Analyzer, iat sql.IndexA
}

a.Log("new indexed table: %s/%s/%s", ita.Index().Database(), ita.Index().Table(), ita.Index().ID())
a.Log("index stats cnt: ", stats.RowCount())
a.Log("index stats histogram", stats.Histogram().DebugString())
a.Log("index stats cnt: %d", stats.RowCount())
a.Log("index stats histogram: %s", stats.Histogram().DebugString())

// excluded from tree + not included in index scan => filter above scan
if len(filters) > 0 {
Expand Down
6 changes: 3 additions & 3 deletions sql/memo/memo.go
Original file line number Diff line number Diff line change
Expand Up @@ -322,9 +322,9 @@ func (m *Memo) memoizeIndexScan(grp *ExprGroup, ita *plan.IndexedTableAccess, al
// as done early.
func (m *Memo) MemoizeStaticIndexAccess(grp *ExprGroup, aliasName string, idx *Index, ita *plan.IndexedTableAccess, filters []sql.Expression, stat sql.Statistic) {
if m.Debug {
m.Ctx.GetLogger().Debug("new indexed table: %s/%s/%s", ita.Index().Database(), ita.Index().Table(), ita.Index().ID())
m.Ctx.GetLogger().Debug("index stats cnt: ", stat.RowCount())
m.Ctx.GetLogger().Debug("index stats histogram", stat.Histogram().DebugString())
m.Ctx.GetLogger().Debugf("new indexed table: %s/%s/%s", ita.Index().Database(), ita.Index().Table(), ita.Index().ID())
m.Ctx.GetLogger().Debugf("index stats cnt: %d: ", stat.RowCount())
m.Ctx.GetLogger().Debugf("index stats histogram: %s", stat.Histogram().DebugString())
}
if len(filters) > 0 {
// set the indexed path as best. correct for cases where
Expand Down

0 comments on commit 6a6f861

Please sign in to comment.