membphis commented on a change in pull request #2306:
URL: https://github.com/apache/apisix/pull/2306#discussion_r494299962



##########
File path: apisix/init.lua
##########
@@ -107,6 +107,10 @@ function _M.http_init_worker()
     require("apisix.debug").init_worker()
     require("apisix.upstream").init_worker()
 
+    -- need call seed
+    local uuid = require("resty.jit-uuid")
+    uuid.seed()

Review comment:
       https://github.com/apache/apisix/blob/master/apisix/init.lua#L72
   
   We have set the seed with a better way. So we do not need to set it agian.

##########
File path: apisix/init.lua
##########
@@ -107,6 +107,10 @@ function _M.http_init_worker()
     require("apisix.debug").init_worker()
     require("apisix.upstream").init_worker()
 
+    -- need call seed
+    local uuid = require("resty.jit-uuid")
+    uuid.seed()

Review comment:
       you can print more log for confirming why it works badly. we need to 
know the reason.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to