Skip to content

Commit

Permalink
Switch to estimateCompactSize when reporting outputBytes statistics
Browse files Browse the repository at this point in the history
  • Loading branch information
kewang1024 committed Apr 9, 2024
1 parent 8db0b71 commit cfee0c8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions velox/exec/Driver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -590,7 +590,7 @@ StopReason Driver::runInternal(
if (ctx_->queryConfig().validateOutputFromOperators()) {
validateOperatorResult(intermediateResult, *op);
}
resultBytes = intermediateResult->estimateFlatSize();
resultBytes = intermediateResult->estimateCompactSize();
{
auto lockedStats = op->stats().wlock();
lockedStats->addOutputVector(
Expand Down Expand Up @@ -699,7 +699,7 @@ StopReason Driver::runInternal(
{
auto lockedStats = op->stats().wlock();
lockedStats->addOutputVector(
result->estimateFlatSize(), result->size());
result->estimateCompactSize(), result->size());
}

// This code path is used only in single-threaded execution.
Expand Down

0 comments on commit cfee0c8

Please sign in to comment.