)]}'
{"cinder/volume/drivers/emc/vnx/adapter.py":[{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"2f9aaeead15de184146aaa3712958a43aa949135","unresolved":false,"context_lines":[{"line_number":613,"context_line":"            LOG.info(_LI(\u0027No storage pool is configured. This host will \u0027"},{"line_number":614,"context_line":"                         \u0027manage all the pools on the VNX system.\u0027))"},{"line_number":615,"context_line":""},{"line_number":616,"context_line":"        return list(pool for pool in array_pools if pool.name in pool_names)"},{"line_number":617,"context_line":""},{"line_number":618,"context_line":"    def get_enabler_stats(self):"},{"line_number":619,"context_line":"        stats \u003d dict()"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_405c9a25","line":616,"range":{"start_line":616,"start_character":8,"end_line":616,"end_character":76},"updated":"2016-09-22 12:40:29.000000000","message":"Maybe you should use [] instead of list()? It seems it will be closer to this function code style and faster to execute","commit_id":"39f1e51b70dfb68c46136a43d498157cce6c06bb"},{"author":{"_account_id":22059,"name":"zheng yin","email":"yin__zheng@163.com","username":"zy751713126"},"change_message_id":"e55e5dff801b108d61cdbf18ec238606cee2ac21","unresolved":false,"context_lines":[{"line_number":613,"context_line":"            LOG.info(_LI(\u0027No storage pool is configured. This host will \u0027"},{"line_number":614,"context_line":"                         \u0027manage all the pools on the VNX system.\u0027))"},{"line_number":615,"context_line":""},{"line_number":616,"context_line":"        return list(pool for pool in array_pools if pool.name in pool_names)"},{"line_number":617,"context_line":""},{"line_number":618,"context_line":"    def get_enabler_stats(self):"},{"line_number":619,"context_line":"        stats \u003d dict()"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_97b0a98d","line":616,"range":{"start_line":616,"start_character":8,"end_line":616,"end_character":76},"in_reply_to":"5a8bc5a2_405c9a25","updated":"2016-09-22 13:24:04.000000000","message":"Done","commit_id":"39f1e51b70dfb68c46136a43d498157cce6c06bb"}],"cinder/volume/drivers/emc/vnx/client.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"8757f022d44c07c9490b4d9120a80e8d2e9217bb","unresolved":false,"context_lines":[{"line_number":477,"context_line":"        wwns \u003d set()"},{"line_number":478,"context_line":"        ports_with_all_info \u003d self.vnx.get_fc_port()"},{"line_number":479,"context_line":"        for po in ports:"},{"line_number":480,"context_line":"            online_list \u003d list("},{"line_number":481,"context_line":"                filter(lambda p: (p \u003d\u003d po and p.link_status \u003d\u003d \u0027Up\u0027"},{"line_number":482,"context_line":"                                  and p.port_status \u003d\u003d \u0027Online\u0027),"},{"line_number":483,"context_line":"                       ports_with_all_info))"},{"line_number":484,"context_line":"            online_list \u003d list("},{"line_number":485,"context_line":"                p for p in ports_with_all_info if p \u003d\u003d po and"},{"line_number":486,"context_line":"                p.link_status \u003d\u003d \u0027Up\u0027 and p.port_status \u003d\u003d \u0027Online\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a8bc5a2_92552a1f","line":483,"range":{"start_line":480,"start_character":11,"end_line":483,"end_character":44},"updated":"2016-09-22 08:22:25.000000000","message":"please remove this","commit_id":"0a497b1f7e1a8911e2f974d38daf3e8e2fccca66"},{"author":{"_account_id":22059,"name":"zheng yin","email":"yin__zheng@163.com","username":"zy751713126"},"change_message_id":"94ef0fd0dd479d09352f6e7731b1516202f2f222","unresolved":false,"context_lines":[{"line_number":477,"context_line":"        wwns \u003d set()"},{"line_number":478,"context_line":"        ports_with_all_info \u003d self.vnx.get_fc_port()"},{"line_number":479,"context_line":"        for po in ports:"},{"line_number":480,"context_line":"            online_list \u003d list("},{"line_number":481,"context_line":"                filter(lambda p: (p \u003d\u003d po and p.link_status \u003d\u003d \u0027Up\u0027"},{"line_number":482,"context_line":"                                  and p.port_status \u003d\u003d \u0027Online\u0027),"},{"line_number":483,"context_line":"                       ports_with_all_info))"},{"line_number":484,"context_line":"            online_list \u003d list("},{"line_number":485,"context_line":"                p for p in ports_with_all_info if p \u003d\u003d po and"},{"line_number":486,"context_line":"                p.link_status \u003d\u003d \u0027Up\u0027 and p.port_status \u003d\u003d \u0027Online\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a8bc5a2_7a922759","line":483,"range":{"start_line":480,"start_character":11,"end_line":483,"end_character":44},"in_reply_to":"5a8bc5a2_92552a1f","updated":"2016-09-22 12:18:10.000000000","message":"Done","commit_id":"0a497b1f7e1a8911e2f974d38daf3e8e2fccca66"}]}