)]}'
{"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_common.py":[{"author":{"_account_id":25648,"name":"wang yong","email":"yongwc@foxmail.com","username":"wangyong"},"change_message_id":"acaaffea40fdeed04920dcb18ebcf13bc95f4583","unresolved":false,"context_lines":[{"line_number":1712,"context_line":"    def _get_flashcopy_mapping_attributes(self, fc_map_id):"},{"line_number":1713,"context_line":"        try:"},{"line_number":1714,"context_line":"            resp \u003d self.ssh.lsfcmap(fc_map_id)"},{"line_number":1715,"context_line":"            if not len(resp):"},{"line_number":1716,"context_line":"                return None"},{"line_number":1717,"context_line":"            return resp[0]"},{"line_number":1718,"context_line":"        except exception.VolumeBackendAPIException as ex:"},{"line_number":1719,"context_line":"            LOG.warning(\"Failed to get fcmap %(fcmap)s info. \""},{"line_number":1720,"context_line":"                        \"Exception: %(ex)s.\", {\u0027fcmap\u0027: fc_map_id,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f485f77_e4fb4176","line":1717,"range":{"start_line":1715,"start_character":0,"end_line":1717,"end_character":26},"updated":"2017-11-16 07:06:29.000000000","message":"The logical could be more clear. return resp[0] if resp else None","commit_id":"127f649c09d733b04351fc31a777327b4ee446c0"},{"author":{"_account_id":17045,"name":"xiaoqin","email":"xiaoqin.li@hotmail.com","username":"xiaoqin"},"change_message_id":"a488a0bab02d8297b98afed2d9d4b607b09d9695","unresolved":false,"context_lines":[{"line_number":1712,"context_line":"    def _get_flashcopy_mapping_attributes(self, fc_map_id):"},{"line_number":1713,"context_line":"        try:"},{"line_number":1714,"context_line":"            resp \u003d self.ssh.lsfcmap(fc_map_id)"},{"line_number":1715,"context_line":"            if not len(resp):"},{"line_number":1716,"context_line":"                return None"},{"line_number":1717,"context_line":"            return resp[0]"},{"line_number":1718,"context_line":"        except exception.VolumeBackendAPIException as ex:"},{"line_number":1719,"context_line":"            LOG.warning(\"Failed to get fcmap %(fcmap)s info. \""},{"line_number":1720,"context_line":"                        \"Exception: %(ex)s.\", {\u0027fcmap\u0027: fc_map_id,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f485f77_13b69d8b","line":1717,"range":{"start_line":1715,"start_character":0,"end_line":1717,"end_character":26},"in_reply_to":"1f485f77_e4fb4176","updated":"2017-11-16 10:24:44.000000000","message":"Hi Yong, thanks a lot for your comments. It\u0027s addressed in new patch.","commit_id":"127f649c09d733b04351fc31a777327b4ee446c0"}]}
