2019-10-21 03:44:05 UTC - Jerry Peng: Seems like there is a problem with running integration tests: ``` Python 2.7.16 Traceback (most recent call last): File "bin/gen-yml-from-env.py", line 49, in <module> conf = yaml.load(open(conf_filename), Loader=yaml.FullLoader) AttributeError: 'module' object has no attribute 'FullLoader' Python Version ... ``` ---- 2019-10-21 03:44:25 UTC - Jerry Peng: due to this change: <https://github.com/apache/pulsar/pull/4974>? ---- 2019-10-21 03:58:20 UTC - Ali Ahmed: unlikely ---- 2019-10-21 04:03:04 UTC - Jerry Peng: its more likely this change: <https://github.com/apache/pulsar/pull/5335> ---- 2019-10-21 04:03:09 UTC - Jerry Peng: that changed the env ---- 2019-10-21 05:42:46 UTC - Retardust: @Retardust has joined the channel ---- 2019-10-21 05:48:20 UTC - Retardust: hi, I've wondering why " if (log.isDebugEnabled()) " still used? majority of logger implementation including log4j handle that inside debug method. Thats pattern produce very noisy codebase:( Any arguments to save the lines with it? :slightly_smiling_face: Or I will remove that in files affected by my future pull-requests ----