)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"e15ae2805ab710f3ae7f4323431b48463f792f5e","unresolved":false,"context_lines":[{"line_number":10,"context_line":"NotImplementedError."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Instead of of using NotImplementedError the abc class."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Partial-Bug: #1346797"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia9a75a53f68c28594654a07146a2a30c1ae3c412"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_7db36f4e","line":13,"updated":"2014-08-13 09:35:59.000000000","message":"- \"of\" is twice in the commit message\n- Is there a word missing? ..like: \"it\u0027s using the ABC class?","commit_id":"d993b413e522bbd4a4ae664ae8a0fc30c7cc5a53"}],"tempest/stress/stressaction.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"5630954b44f43ad1a528fd8cb804a5f626382cc0","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import abc"},{"line_number":16,"context_line":"import signal"},{"line_number":17,"context_line":"import six"},{"line_number":18,"context_line":"import sys"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from tempest.openstack.common import log as logging"}],"source_content_type":"text/x-python","patch_set":2,"id":"7ac7a964_4b797e2e","line":17,"updated":"2014-08-14 18:41:40.000000000","message":"So like your other patch, can you move this import to a separate block. It will get thrown out by pep8 once https://review.openstack.org/#/c/98918/ lands","commit_id":"5f279c09f17b3ac0162dc77a17b548c0e63df91e"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"cbecf136bc4bcb564d34df07f81adccb9ba5d6f7","unresolved":false,"context_lines":[{"line_number":90,"context_line":"    @abc.abstractmethod"},{"line_number":91,"context_line":"    def run(self):"},{"line_number":92,"context_line":"        \"\"\"This method is where the stress test code runs.\"\"\""},{"line_number":93,"context_line":"        return"}],"source_content_type":"text/x-python","patch_set":3,"id":"1abeadc6_ef41780c","line":93,"updated":"2014-08-18 18:48:45.000000000","message":"Why not \u0027pass\u0027 instead of return ?","commit_id":"fd247f592229984bccac2600483058750d0024fd"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"c69b6592245c7be4d6969eae70696a2475fbb69a","unresolved":false,"context_lines":[{"line_number":90,"context_line":"    @abc.abstractmethod"},{"line_number":91,"context_line":"    def run(self):"},{"line_number":92,"context_line":"        \"\"\"This method is where the stress test code runs.\"\"\""},{"line_number":93,"context_line":"        return"}],"source_content_type":"text/x-python","patch_set":3,"id":"1abeadc6_de43ecac","line":93,"in_reply_to":"1abeadc6_ef41780c","updated":"2014-08-22 08:43:32.000000000","message":"I guess the code is here does not matter to much as it\u0027s declared as abstract. pass would be nicer, but I\u0027m going to +A anyways.","commit_id":"fd247f592229984bccac2600483058750d0024fd"}]}