Skip to content

Commit

Permalink
[#378][FOLLOWUP] fix(server): Fix huge_partition_num metric (#1669)
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

Fix the metric huge_partition_num.

### Why are the changes needed?

A follow-up PR for: #494.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Existing UTs.
  • Loading branch information
rickyma authored and Heng Qin committed Apr 30, 2024
1 parent 51a2639 commit f4a4c8f
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -774,7 +774,7 @@ public void removeResources(String appId, boolean checkAppExpired) {
storageManager.removeResources(new AppPurgeEvent(appId, shuffleTaskInfo.getUser()));
if (shuffleTaskInfo.hasHugePartition()) {
ShuffleServerMetrics.gaugeAppWithHugePartitionNum.dec();
ShuffleServerMetrics.gaugeHugePartitionNum.dec(shuffleTaskInfo.getHugePartitionSize());
ShuffleServerMetrics.gaugeHugePartitionNum.dec();
}
LOG.info(
"Finish remove resource for appId["
Expand Down

0 comments on commit f4a4c8f

Please sign in to comment.