)]}'
{"cloudpulse/common/rpc.py":[{"author":{"_account_id":25587,"name":"Shubhendu Poothia","email":"poothia.shubhendu@gmail.com","username":"shubhendu"},"change_message_id":"1a0503591a83fda60f3e701b7dac79463c15a1e5","unresolved":false,"context_lines":[{"line_number":32,"context_line":"from cloudpulse.common import exception"},{"line_number":33,"context_line":"from oslo_config import cfg"},{"line_number":34,"context_line":"from oslo_serialization import jsonutils"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"from oslo_messaging.rpc import dispatcher"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_fd9129f3","line":35,"updated":"2017-06-21 03:45:08.000000000","message":"why is there a blank line here? and please move line 36 to line 34 and line 34 to line 35 to arrange it alphabetically.","commit_id":"c13b3ef0d33b1746bda48fe84e0a283ec58557ce"}],"cloudpulse/common/rpc_service.py":[{"author":{"_account_id":25587,"name":"Shubhendu Poothia","email":"poothia.shubhendu@gmail.com","username":"shubhendu"},"change_message_id":"1a0503591a83fda60f3e701b7dac79463c15a1e5","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from cloudpulse.common import context as cloudpulse_context"},{"line_number":22,"context_line":"from cloudpulse.common import rpc"},{"line_number":23,"context_line":"from cloudpulse.objects import base as objects_base"},{"line_number":24,"context_line":"from oslo_messaging.rpc import dispatcher"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"# NOTE(paulczar):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_dde0ad99","line":24,"updated":"2017-06-21 03:45:08.000000000","message":"oslo_messaging, as a module has been imported above on line 19. Python will optimize your import, but it looks redundant, \n\naccess_policy \u003d messaging.dispatcher.DefaultRPCAccessPolicy\n\n\nEither do this or move line 24 to line 20, i.e, all third party libraries must be in one contiguous section, arranged alphabetically. However, i would suggest not reimporting.","commit_id":"c13b3ef0d33b1746bda48fe84e0a283ec58557ce"}]}