Revert "perf(router): iterate routes with for loop (#11111)"

This reverts commit e8173ca.
Kong · Aug 10, 2023 · 66363e4 · 66363e4 · khcp-gha-bot · Aug 10, 2023
1 parent f55614c
commit 66363e4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/kong/router/atc.lua b/kong/router/atc.lua
@@ -188,9 +188,7 @@ local function new_from_scratch(routes, get_exp_and_priority)
 
   local new_updated_at = 0
 
-  for i = 1, routes_n do
-    local r = routes[i]
-
+  for _, r in ipairs(routes) do
     local route = r.route
     local route_id = route.id
 
@@ -249,9 +247,7 @@ local function new_from_previous(routes, get_exp_and_priority, old_router)
   local new_updated_at = 0
 
   -- create or update routes
-  for i = 1, #routes do
-    local r = routes[i]
-
+  for _, r in ipairs(routes) do
     local route = r.route
     local route_id = route.id
 

diff --git a/kong/router/utils.lua b/kong/router/utils.lua
@@ -288,8 +288,8 @@ do
     local v0              = 0
     local v1              = 0
 
-    for i = 1, #routes do
-      local r = routes[i].route
+    for _, route in ipairs(routes) do
+      local r = route.route
 
       local paths_t     = r.paths or empty_table
       local headers_t   = r.headers or empty_table