spacewander commented on a change in pull request #3848:
URL: https://github.com/apache/apisix/pull/3848#discussion_r597598980



##########
File path: apisix/discovery/consul_kv.lua
##########
@@ -244,7 +244,7 @@ end
 local function read_dump_srvs()
     local data, err = util.read_file(dump_params.path)
     if not data then
-        log.warn("read dump file get error: ", err)
+        log.notice("read dump file get error: ", err)

Review comment:
       @yongboy 
   We just add the heavy library in the master branch: 
https://github.com/apache/apisix/blob/1064e2d4b7edcc6f590ca69dc34a100ae6bbc12c/rockspec/apisix-master-0.rockspec#L68.
   Would be fine if we can find another usage of this library.




-- 
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:
us...@infra.apache.org


Reply via email to