Pre-allocate memory for slices to prevent repeated reallocation and copy during capacity expansion

[dream-kzx]

opengemini-client-go/opengemini/retention_policy.go

Lines 81 to 129 in 5898fcc

	func convertRetentionPolicy(queryResult *QueryResult) []RetentionPolicy {
	var (
	retentionPolicy = make([]RetentionPolicy, 0)
	rpColumnLen = 8
	)
	if len(queryResult.Results) == 0 || len(queryResult.Results[0].Series) == 0 {
	return retentionPolicy
	}
	for _, v := range queryResult.Results[0].Series[0].Values {
	if len(v) < rpColumnLen {
	break
	}
	var (
	ok bool
	replicaNum float64
	)
	rp := new(RetentionPolicy)
	if rp.Name, ok = v[0].(string); !ok {
	break
	}
	if rp.Duration, ok = v[1].(string); !ok {
	break
	}
	if rp.ShardGroupDuration, ok = v[2].(string); !ok {
	break
	}
	if rp.HotDuration, ok = v[3].(string); !ok {
	break
	}
	if rp.WarmDuration, ok = v[4].(string); !ok {
	break
	}
	if rp.IndexDuration, ok = v[5].(string); !ok {
	break
	}
	if replicaNum, ok = v[6].(float64); !ok {
	break
	}
	rp.ReplicaNum = int64(replicaNum)
	if rp.IsDefault, ok = v[7].(bool); !ok {
	break
	}
	retentionPolicy = append(retentionPolicy, *rp)
	}
	return retentionPolicy
	}

variable: retentionPolicy

[hezhangjian]

Although query retention policy isn't performance sensitive, but I think it can be optimized. #102 also reveals another problems of this class method. cc @cyruslo @Chenxulin97

[dream-kzx]

There are many other places with the same problem. I've only listed this one.

[Chenxulin97]

There are many other places with the same problem. I've only listed this one.

It's not a problem , we will find the whole places and fix them . thanx for ur contribution.

[dream-kzx]

ok

[hezhangjian]

@dream-kzx Let's keep this issue open util it's resolved :)

[hezhangjian]

closed by #117