bzp2010 commented on code in PR #12216:
URL: https://github.com/apache/apisix/pull/12216#discussion_r2101507333


##########
apisix/cli/file.lua:
##########
@@ -293,7 +294,9 @@ function _M.read_yaml_conf(apisix_home)
         end
     end
 
-    if default_conf.deployment.config_provider == "yaml" then
+    --- using `not ngx` to check whether the current execution environment is 
apisix cli module,
+    --- because it is only necessary to parse and validate `apisix.yaml` in 
apisix cli.
+    if default_conf.deployment.config_provider == "yaml" and not ngx then
         local apisix_conf_path = profile:yaml_path("apisix")

Review Comment:
   Actually I don't see what the following code does, it just reads the yaml 
and parses the variables whose values it contains. Other than that, there is no 
utilization of the values it reads.
   So whether we add this extra judgment or not doesn't affect the 
functionality of using it. I read your PR description and this is just an 
efficiency difference? I didn't get the need for it.



-- 
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.

To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to