From f03544f908fd25c33102d618602c335a9fcf815a Mon Sep 17 00:00:00 2001 From: tuteng Date: Mon, 19 Aug 2019 19:25:19 +0800 Subject: [PATCH] Add default loader for latest pyyaml (#4974) Fixes https://github.com/apache/pulsar/issues/4956 Master Issue: https://github.com/apache/pulsar/issues/4956 ### Motivation In the latest version of pyyaml, using load will issue a warning and explain some reasons here https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation. ### Modifications * Use yaml.FullLoader as default loader (cherry picked from commit d49e61e0907c30d71184da4eb769c9761456a80f) --- docker/pulsar/scripts/gen-yml-from-env.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docker/pulsar/scripts/gen-yml-from-env.py b/docker/pulsar/scripts/gen-yml-from-env.py index 1aa16b47b44..6bddcdf938a 100755 --- a/docker/pulsar/scripts/gen-yml-from-env.py +++ b/docker/pulsar/scripts/gen-yml-from-env.py @@ -46,7 +46,7 @@ if len(sys.argv) < 2: conf_files = sys.argv[1:] for conf_filename in conf_files: - conf = yaml.load(open(conf_filename)) + conf = yaml.load(open(conf_filename), Loader=yaml.FullLoader) # update the config modified = False -- GitLab