try showing the first 100 volume ids and an extra ...

This commit is contained in:
bingoohuang 2020-05-29 16:15:33 +08:00
parent 1a642b9876
commit 1f8782a1ed
3 changed files with 16 additions and 7 deletions

View file

@ -195,16 +195,11 @@ func (dn *DataNode) ToDataNodeInfo() *master_pb.DataNodeInfo {
// GetVolumeIds returns the human readable volume ids limited to count of max 100.
func (dn *DataNode) GetVolumeIds() string {
volumesLen := len(dn.volumes)
if volumesLen > 100 {
return "..."
}
ids := make([]int, 0, volumesLen)
ids := make([]int, 0, len(dn.volumes))
for k := range dn.volumes {
ids = append(ids, int(k))
}
return util.HumanReadableInts(ids...)
return util.HumanReadableIntsMax(100, ids...)
}

View file

@ -5,6 +5,15 @@ import (
"sort"
)
// HumanReadableIntsMax joins a serials of inits into a smart one like 1-3 5 ... for human readable.
func HumanReadableIntsMax(max int, ids ...int) string {
if len(ids) <= max {
return HumanReadableInts(ids...)
}
return HumanReadableInts(ids[:max]...) + " ..."
}
// HumanReadableInts joins a serials of inits into a smart one like 1-3 5 7-10 for human readable.
func HumanReadableInts(ids ...int) string {
sort.Ints(ids)

View file

@ -5,6 +5,11 @@ import (
"testing"
)
func TestHumanReadableIntsMax(t *testing.T) {
assert.Equal(t, "1-2 ...", HumanReadableIntsMax(2, 1, 2, 3))
assert.Equal(t, "1 3 ...", HumanReadableIntsMax(2, 1, 3, 5))
}
func TestHumanReadableInts(t *testing.T) {
assert.Equal(t, "1-3", HumanReadableInts(1, 2, 3))
assert.Equal(t, "1 3", HumanReadableInts(1, 3))