)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"a15029111218a10a2999877ea6cfbe3314e5ce6a","unresolved":false,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2017-08-04 10:50:24 +0800"},{"line_number":4,"context_line":"Commit:     yanghuichan \u003cyanghc@fiberhome.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-08-04 10:52:09 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replacing six.iter usages substituting .items(), .keys()"},{"line_number":8,"context_line":"and .values() in place of six.iteritems/dict.iteritems ,"},{"line_number":9,"context_line":"six.iterkeys/dict.iterkeys and six.itervalues/itervalues"},{"line_number":10,"context_line":"respectively."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"1.As mentioned in [1], we should avoid using six.iteritems"},{"line_number":13,"context_line":"to achieve iterators. We can use dict.items instead, as it"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df3967d1_14d8a31b","line":10,"range":{"start_line":6,"start_character":0,"end_line":10,"end_character":13},"updated":"2017-08-04 06:06:14.000000000","message":"Please add short header for your commit message.","commit_id":"40fba7f15502b4125fe3c1335cf86c21eab11841"}],"cinder/volume/drivers/ibm/ibm_storage/proxy.py":[{"author":{"_account_id":23630,"name":"Tovin Seven","email":"tovin07@gmail.com","username":"tovin07"},"change_message_id":"1fef497c86dea44b9bea3b9b23c4844d581ecc37","unresolved":false,"context_lines":[{"line_number":360,"context_line":"        if not self.targets:"},{"line_number":361,"context_line":"            return None"},{"line_number":362,"context_line":"        try:"},{"line_number":363,"context_line":"            target \u003d (self.targets).keys().next()"},{"line_number":364,"context_line":"            return target"},{"line_number":365,"context_line":"        except Exception:"},{"line_number":366,"context_line":"            return None"}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_8586a012","line":363,"range":{"start_line":363,"start_character":21,"end_line":363,"end_character":22},"updated":"2017-08-04 04:10:25.000000000","message":"redundant","commit_id":"40fba7f15502b4125fe3c1335cf86c21eab11841"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"a15029111218a10a2999877ea6cfbe3314e5ce6a","unresolved":false,"context_lines":[{"line_number":360,"context_line":"        if not self.targets:"},{"line_number":361,"context_line":"            return None"},{"line_number":362,"context_line":"        try:"},{"line_number":363,"context_line":"            target \u003d (self.targets).keys().next()"},{"line_number":364,"context_line":"            return target"},{"line_number":365,"context_line":"        except Exception:"},{"line_number":366,"context_line":"            return None"}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_94fdb36f","line":363,"range":{"start_line":363,"start_character":21,"end_line":363,"end_character":22},"in_reply_to":"df3967d1_8586a012","updated":"2017-08-04 06:06:14.000000000","message":"+1","commit_id":"40fba7f15502b4125fe3c1335cf86c21eab11841"},{"author":{"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},"change_message_id":"97815f96b8e698601bb12c59e5ad5dc64c8d4a97","unresolved":false,"context_lines":[{"line_number":360,"context_line":"        if not self.targets:"},{"line_number":361,"context_line":"            return None"},{"line_number":362,"context_line":"        try:"},{"line_number":363,"context_line":"            target \u003d self.targets.iterkeys().next()"},{"line_number":364,"context_line":"            return target"},{"line_number":365,"context_line":"        except Exception:"},{"line_number":366,"context_line":"            return None"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf3e73b7_41c15254","line":363,"range":{"start_line":363,"start_character":34,"end_line":363,"end_character":42},"updated":"2017-08-07 22:05:49.000000000","message":"Shouldn\u0027t this be keys()?","commit_id":"e699221983b45a33e9ea10dfd5a53a8fe895f008"}],"cinder/volume/drivers/ibm/ibm_storage/xiv_proxy.py":[{"author":{"_account_id":23630,"name":"Tovin Seven","email":"tovin07@gmail.com","username":"tovin07"},"change_message_id":"1fef497c86dea44b9bea3b9b23c4844d581ecc37","unresolved":false,"context_lines":[{"line_number":1580,"context_line":"            self.meta[\u0027stat\u0027][\u0027rpo\u0027] \u003d repl.Replication.get_supported_rpo()"},{"line_number":1581,"context_line":"            self.meta[\u0027stat\u0027][\u0027replication_count\u0027] \u003d len(self.targets)"},{"line_number":1582,"context_line":"            self.meta[\u0027stat\u0027][\u0027replication_targets\u0027] \u003d [target for target in"},{"line_number":1583,"context_line":"                                                        (self.targets).keys()]"},{"line_number":1584,"context_line":""},{"line_number":1585,"context_line":"        self.meta[\u0027stat\u0027][\u0027timestamp\u0027] \u003d datetime.datetime.utcnow()"},{"line_number":1586,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_657f3c15","line":1583,"range":{"start_line":1583,"start_character":56,"end_line":1583,"end_character":57},"updated":"2017-08-04 04:10:25.000000000","message":"redundant","commit_id":"40fba7f15502b4125fe3c1335cf86c21eab11841"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"a15029111218a10a2999877ea6cfbe3314e5ce6a","unresolved":false,"context_lines":[{"line_number":1580,"context_line":"            self.meta[\u0027stat\u0027][\u0027rpo\u0027] \u003d repl.Replication.get_supported_rpo()"},{"line_number":1581,"context_line":"            self.meta[\u0027stat\u0027][\u0027replication_count\u0027] \u003d len(self.targets)"},{"line_number":1582,"context_line":"            self.meta[\u0027stat\u0027][\u0027replication_targets\u0027] \u003d [target for target in"},{"line_number":1583,"context_line":"                                                        (self.targets).keys()]"},{"line_number":1584,"context_line":""},{"line_number":1585,"context_line":"        self.meta[\u0027stat\u0027][\u0027timestamp\u0027] \u003d datetime.datetime.utcnow()"},{"line_number":1586,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_74024f71","line":1583,"range":{"start_line":1583,"start_character":56,"end_line":1583,"end_character":57},"in_reply_to":"df3967d1_657f3c15","updated":"2017-08-04 06:06:14.000000000","message":"+1","commit_id":"40fba7f15502b4125fe3c1335cf86c21eab11841"}]}