Skip to content

Commit a0b8e47

Browse files
Merge pull request #151044 from cockroachdb/blathers/backport-release-25.3-150933
release-25.3: server: change sort order of local hot ranges to descending
2 parents e29b20f + c292491 commit a0b8e47

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

pkg/server/application_api/storage_inspection_test.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -340,8 +340,6 @@ func TestHotRanges2Response(t *testing.T) {
340340
defer leaktest.AfterTest(t)()
341341
defer log.Scope(t).Close(t)
342342

343-
skip.WithIssue(t, 146917)
344-
345343
srv := rangetestutils.StartServer(t)
346344
defer srv.Stopper().Stop(context.Background())
347345
ts := srv.ApplicationLayer()

pkg/server/status.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3069,7 +3069,7 @@ func (s *systemStatusServer) localHotRanges(
30693069

30703070
// sort the slices by cpu
30713071
slices.SortFunc(resp.Ranges, func(a, b *serverpb.HotRangesResponseV2_HotRange) int {
3072-
return cmp.Compare(a.CPUTimePerSecond, b.CPUTimePerSecond)
3072+
return cmp.Compare(b.CPUTimePerSecond, a.CPUTimePerSecond)
30733073
})
30743074

30753075
// truncate the response if localLimit is set

0 commit comments

Comments
 (0)