Skip to content

Commit

Permalink
executor: fix cte mem tracker failpoint condition again (#54523) (#54732
Browse files Browse the repository at this point in the history
)

close #54379
  • Loading branch information
ti-chi-bot authored Jul 21, 2024
1 parent 3369560 commit ac79ace
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions executor/cte.go
Original file line number Diff line number Diff line change
Expand Up @@ -436,9 +436,7 @@ func (p *cteProducer) computeRecursivePart(ctx context.Context) (err error) {
iterNum++
failpoint.Inject("assertIterTableSpillToDisk", func(maxIter failpoint.Value) {
if iterNum > 0 && iterNum < uint64(maxIter.(int)) && err == nil {
if p.iterInTbl.GetMemBytes() != 0 || p.iterInTbl.GetDiskBytes() == 0 ||
p.iterOutTbl.GetMemBytes() != 0 || p.iterOutTbl.GetDiskBytes() == 0 ||
p.resTbl.GetMemBytes() != 0 || p.resTbl.GetDiskBytes() == 0 {
if p.iterInTbl.GetDiskBytes() == 0 && p.iterOutTbl.GetDiskBytes() == 0 && p.resTbl.GetDiskBytes() == 0 {
p.logTbls(ctx, err, iterNum)
panic("assert row container spill disk failed")
}
Expand Down

0 comments on commit ac79ace

Please sign in to comment.