From ce8d4dcd357b727bba6f77953613cb361b0b71c8 Mon Sep 17 00:00:00 2001 From: Andrew Sisley Date: Tue, 12 Jul 2022 10:37:11 -0400 Subject: [PATCH] PR Suggestion - remove length variable --- query/graphql/planner/count.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/query/graphql/planner/count.go b/query/graphql/planner/count.go index 77a1b5a3bf..ee5898a641 100644 --- a/query/graphql/planner/count.go +++ b/query/graphql/planner/count.go @@ -98,8 +98,6 @@ func (n *countNode) Next() (bool, error) { switch v.Kind() { // v.Len will panic if v is not one of these types, we don't want it to panic case reflect.Array, reflect.Chan, reflect.Map, reflect.Slice, reflect.String: - length := v.Len() - if source.Filter != nil { switch array := property.(type) { case []core.Doc: @@ -158,7 +156,7 @@ func (n *countNode) Next() (bool, error) { } } } else { - count = count + length + count = count + v.Len() } } }