Don't count only requests in statistics

Author: Totktonada

graphql/bfs_executor.lua

@@ -117,8 +117,22 @@
 ---                                 iterator_opts = <...>,
 ---                                 is_full_scan = <boolean>,
 ---                             }
----                             select_state = <...>,
----                             select_opts = <...>,
+---                             select_state = {
+---                                 count = <number>,
+---                                 objs = <list of objects>,
+---                                 pivot_found = <boolean>,
+---                                 qcontext = <table>,
+---                             },
+---                             select_opts = {
+---                                 model = <compiled avro schema>,
+---                                 limit = <number>,
+---                                 filter = <table>,
+---                                 do_filter = <boolean>,
+---                                 pivot_filter = <table>,
+---                                 resulting_object_cnt_max = <number>,
+---                                 fetched_object_cnt_max = <number>,
+---                                 resolveField = <function>,
+---                             },
 ---                             collection_name = <string>,
 ---                             from = <...>,
 ---                             filter = <...>,
@@ -625,10 +639,23 @@ local function invoke_resolve(prepared_object, context, opts)
         else
             if is_item_cache_only then
                 -- prevent statisting from being affected by cache only requests
-                -- XXX: catch cache misses for such requests
-                -- XXX: avoid deepcopy
-                local extra = field_info.prepared_resolve.prepared_select.extra
-                extra.qcontext = table.deepcopy(extra.qcontext)
+                local prepared_select = field_info.prepared_resolve.prepared_select
+                local extra = prepared_select.extra
+                local select_state = prepared_select.select_state
+                local qcontext = extra.qcontext
+                local new_qcontext = {
+                    initialized = true,
+                    deadline_clock = qcontext.deadline_clock,
+                    statistics = {
+                        resulting_object_cnt = 0,
+                        fetched_object_cnt = 0,
+                        select_requests_cnt = 0,
+                        full_scan_select_requests_cnt = 0,
+                        index_select_requests_cnt = 0,
+                    }
+                }
+                extra.qcontext = new_qcontext
+                select_state.qcontext = new_qcontext
             end
             object_or_list, object_type = field_info.prepared_resolve:invoke()
         end

test/space/nested_args.test.lua

@@ -57,7 +57,6 @@ local common_gql_wrapper = graphql.new({
     accessor = 'space',
     -- gh-137: timeout exceeded
     timeout_ms = 10000, -- 10 seconds
-    fetched_object_cnt_max = 100000, -- XXX: remove it
 })
 
 local emails_gql_wrapper = graphql.new({
@@ -68,7 +67,6 @@ local emails_gql_wrapper = graphql.new({
     accessor = 'space',
     -- gh-137: timeout exceeded
     timeout_ms = 10000, -- 10 seconds
-    fetched_object_cnt_max = 100000, -- XXX: remove it
 })
 
 -- run queries