Skip to content

Commit 62486a2

Browse files
committed
Pre-alloate slices for better performance
Signed-off-by: Marco Franssen <marco.franssen@gmail.com>
1 parent 40a3935 commit 62486a2

File tree

2 files changed

+11
-11
lines changed

2 files changed

+11
-11
lines changed

cmd/doc-gen/main.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -229,8 +229,6 @@ type Pair struct {
229229
type KubeTypes []Pair
230230

231231
func ParseDocumentationFrom(src string, dl_name string, shouldSort bool) []KubeTypes {
232-
var docForTypes []KubeTypes
233-
234232
fset := token.NewFileSet()
235233
f, err := parser.ParseFile(fset, src, nil, parser.ParseComments)
236234
if err != nil {
@@ -258,6 +256,7 @@ func ParseDocumentationFrom(src string, dl_name string, shouldSort bool) []KubeT
258256
})
259257
}
260258

259+
docForTypes := make([]KubeTypes, 0, len(types))
261260
for _, kubType := range types {
262261
structType, _ := kubType.Decl.Specs[0].(*ast.TypeSpec).Type.(*ast.StructType)
263262

controllers/fluentbitconfig_controller.go

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -247,23 +247,24 @@ func (r *FluentBitConfigReconciler) processNamespacedFluentBitCfgs(
247247
[]fluentbitv1alpha2.MultilineParserList, []fluentbitv1alpha2.ClusterMultilineParserList, []string, error,
248248
) {
249249
var nsCfgs fluentbitv1alpha2.FluentBitConfigList
250-
var filters []fluentbitv1alpha2.FilterList
251-
var outputs []fluentbitv1alpha2.OutputList
252-
var parsers []fluentbitv1alpha2.ParserList
253-
var clusterParsers []fluentbitv1alpha2.ClusterParserList
254-
var multilineParsers []fluentbitv1alpha2.MultilineParserList
255-
var clusterMultilineParsers []fluentbitv1alpha2.ClusterMultilineParserList
256-
var rewriteTagConfigs []string
257250
// set of rewrite_tag plugin configs to mutate tags for log records coming out of a namespace
258251
selector, err := metav1.LabelSelectorAsSelector(&fb.Spec.NamespacedFluentBitCfgSelector)
259252
if err != nil {
260-
return filters, outputs, parsers, clusterParsers, multilineParsers, clusterMultilineParsers, nil, err
253+
return nil, nil, nil, nil, nil, nil, nil, err
261254
}
262255

263256
if err := r.List(ctx, &nsCfgs, client.MatchingLabelsSelector{Selector: selector}); err != nil {
264-
return filters, outputs, parsers, clusterParsers, multilineParsers, clusterMultilineParsers, nil, err
257+
return nil, nil, nil, nil, nil, nil, nil, err
265258
}
266259

260+
filters := make([]fluentbitv1alpha2.FilterList, 0, len(nsCfgs.Items))
261+
outputs := make([]fluentbitv1alpha2.OutputList, 0, len(nsCfgs.Items))
262+
parsers := make([]fluentbitv1alpha2.ParserList, 0, len(nsCfgs.Items))
263+
clusterParsers := make([]fluentbitv1alpha2.ClusterParserList, 0, len(nsCfgs.Items))
264+
multilineParsers := make([]fluentbitv1alpha2.MultilineParserList, 0, len(nsCfgs.Items))
265+
clusterMultilineParsers := make([]fluentbitv1alpha2.ClusterMultilineParserList, 0, len(nsCfgs.Items))
266+
rewriteTagConfigs := make([]string, 0, len(nsCfgs.Items))
267+
267268
// Form a slice of list of resources per namespace
268269
for _, cfg := range nsCfgs.Items {
269270
filterList, outputList, parserList, clusterParserList, multilineParsersList, clusterMultilineParsersList, err := r.ListNamespacedResources(

0 commit comments

Comments
 (0)