Skip to content

Commit

Permalink
HPCC-31062 Update comment
Browse files Browse the repository at this point in the history
Signed-off-by: Shamser Ahmed <[email protected]>
  • Loading branch information
shamser committed Apr 29, 2024
1 parent ffd6ee1 commit f3c0d77
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion common/workunit/workunit.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -9023,7 +9023,13 @@ void CLocalWorkUnit::setStatistic(StatisticCreatorType creatorType, const char *

// Special case - update aggregates for dfu FileAccessCost. This is needed because although
// fileservices can update dfu cost in the workunit, it does not have a mechanism to update
// the aggregates for this stat.
// the aggregates for this stat (i.e. fileservices doesn't have access to a StatsAggregator.
// And say the StatsAggregator were created here, it would need to load all the global stats
// from dali, mark the stat as 'dirty', and then have the aggregator refresh the aggregates.
// This would be slow and unnessarily cumbersome.) Note also that if the stat is written
// with the expectation that the hThor/Thor stats aggregator would generate the aggregate for
// this stat, it would not be the case as the aggregates are only created for stats marked
// 'dirty'.
if (scopeType == SSTdfuworkunit && kind == StCostFileAccess && deltaValue)
{
StringBuffer currentScope(scope), parent;
Expand Down

0 comments on commit f3c0d77

Please sign in to comment.