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