Skip to content

Commit

Permalink
Merge pull request #19161 from shamser/issue30529
Browse files Browse the repository at this point in the history
HPCC-30529 Make all node number reported in stats one-based

Reviewed-by: Jake Smith <jake.smith@lexisnexisrisk.com>
Merged-by: Gavin Halliday <ghalliday@hpccsystems.com>
  • Loading branch information
ghalliday authored Sep 27, 2024
2 parents 2484ceb + 933d8c8 commit 33b98a7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions system/jlib/jstats.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3186,8 +3186,8 @@ void CRuntimeSummaryStatisticCollection::recordStatistics(IStatisticGatherer & t
//First test is redundant - but protects against minValue != maxValue test above changing.
if ((cur.minNode != cur.maxNode) && isSignificantRange(serialKind, range, mean))
{
target.addStatistic((StatisticKind)(serialKind|StNodeMin), cur.minNode);
target.addStatistic((StatisticKind)(serialKind|StNodeMax), cur.maxNode);
target.addStatistic((StatisticKind)(serialKind|StNodeMin), cur.minNode+1);
target.addStatistic((StatisticKind)(serialKind|StNodeMax), cur.maxNode+1);

if (skewHasMeaning(serialKind))
{
Expand Down

0 comments on commit 33b98a7

Please sign in to comment.