Skip to content

Commit

Permalink
Merge pull request #18366 from shamser/issue31382
Browse files Browse the repository at this point in the history
HPCC-31382 Activity CostFileAccess not always updated to sg scope

Reviewed-by: Jake Smith <[email protected]>
Merged-by: Gavin Halliday <[email protected]>
  • Loading branch information
ghalliday authored Mar 5, 2024
2 parents 77749a8 + ffb592e commit 0b90dc9
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions thorlcr/graph/thgraphmaster.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2925,10 +2925,6 @@ bool CMasterGraph::deserializeStats(unsigned node, MemoryBuffer &mb)
void CMasterGraph::getStats(IStatisticGatherer &stats)
{
stats.addStatistic(StNumSlaves, queryClusterWidth());
cost_type costDiskAccess = getDiskAccessCost();
if (costDiskAccess)
stats.addStatistic(StCostFileAccess, costDiskAccess);

// graph specific stats
graphStats.getStats(stats);

Expand Down Expand Up @@ -2957,6 +2953,9 @@ void CMasterGraph::getStats(IStatisticGatherer &stats)
activity->getActivityStats(stats);
}
}
cost_type costDiskAccess = getDiskAccessCost();
if (costDiskAccess)
stats.addStatistic(StCostFileAccess, costDiskAccess);
}

cost_type CMasterGraph::getDiskAccessCost()
Expand Down

0 comments on commit 0b90dc9

Please sign in to comment.