)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"e610d0f71fcf6bca63e41902e4093ebb47181eb6","unresolved":false,"context_lines":[{"line_number":16,"context_line":"\u003cmap object at 0x021FFD90\u003e"},{"line_number":17,"context_line":"[111, 112, 113]"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Partially-implements blueprint py3-compatibility"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Change-Id: I1b4d84388adb03939918750c6596c3db4c88a330"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_b92f8df4","line":19,"updated":"2016-07-01 10:05:59.000000000","message":"it should be: blueprint trove-python3","commit_id":"6fcc22e7a69f641523b30c1156bbfbade922da47"}],"trove/common/strategies/cluster/experimental/redis/taskmanager.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"aafce6d57e7e18740a295773570c5a66ae5c54cd","unresolved":false,"context_lines":[{"line_number":125,"context_line":"            LOG.debug(\"All members ready, proceeding for cluster setup.\")"},{"line_number":126,"context_line":"            new_insts \u003d [Instance.load(context, instance_id)"},{"line_number":127,"context_line":"                         for instance_id in new_instance_ids]"},{"line_number":128,"context_line":"            new_guests \u003d list(map(self.get_guest, new_insts))"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"            # Connect nodes to the cluster head"},{"line_number":131,"context_line":"            for guest in new_guests:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3aaa91ec_39f6fdd6","line":128,"updated":"2016-07-01 10:05:28.000000000","message":"Oh yes, it looks like this change fixes a bug since new_guests is consumed twice.","commit_id":"6fcc22e7a69f641523b30c1156bbfbade922da47"}],"trove/common/stream_codecs.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"aafce6d57e7e18740a295773570c5a66ae5c54cd","unresolved":false,"context_lines":[{"line_number":314,"context_line":"                if not key.strip().startswith(self._comment_markers):"},{"line_number":315,"context_line":"                    items \u003d self._string_converter.to_objects("},{"line_number":316,"context_line":"                        [v if v else None for v in"},{"line_number":317,"context_line":"                         list(map(self._strip_comments, row[1:]))])"},{"line_number":318,"context_line":"                    current \u003d data_dict.get(key)"},{"line_number":319,"context_line":"                    if current is not None:"},{"line_number":320,"context_line":"                        current.append(trove_utils.unpack_singleton(items)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3aaa91ec_f9263565","line":317,"updated":"2016-07-01 10:05:28.000000000","message":"this change looks useless","commit_id":"6fcc22e7a69f641523b30c1156bbfbade922da47"}],"trove/guestagent/datastore/experimental/redis/service.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"aafce6d57e7e18740a295773570c5a66ae5c54cd","unresolved":false,"context_lines":[{"line_number":342,"context_line":"            group_size \u003d 200"},{"line_number":343,"context_line":"            while slots:"},{"line_number":344,"context_line":"                cmd \u003d ([system.REDIS_CLI, \u0027cluster\u0027, \u0027addslots\u0027]"},{"line_number":345,"context_line":"                       + slots[0:group_size])"},{"line_number":346,"context_line":"                out, err \u003d utils.execute_with_timeout(*cmd, run_as_root\u003dTrue,"},{"line_number":347,"context_line":"                                                      root_helper\u003d\u0027sudo\u0027)"},{"line_number":348,"context_line":"                if \u0027OK\u0027 not in out:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3aaa91ec_39cb9d73","line":345,"updated":"2016-07-01 10:05:28.000000000","message":"Ah yes, this change also fixes a bug, because map() object doesn\u0027t support indexing (obj[int]).","commit_id":"6fcc22e7a69f641523b30c1156bbfbade922da47"}],"trove/tests/fakes/swift.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"aafce6d57e7e18740a295773570c5a66ae5c54cd","unresolved":false,"context_lines":[{"line_number":474,"context_line":"        def side_effect_func(*args, **kwargs):"},{"line_number":475,"context_line":"            if (args[0] in self._containers and"},{"line_number":476,"context_line":"                args[1] in list(map(lambda x: x[\u0027name\u0027],"},{"line_number":477,"context_line":"                                    self._objects[args[0]]))):"},{"line_number":478,"context_line":"                return ("},{"line_number":479,"context_line":"                    {\u0027content-length\u0027: len(contents), \u0027accept-ranges\u0027: \u0027bytes\u0027,"},{"line_number":480,"context_line":"                     \u0027last-modified\u0027: \u0027Mon, 10 Mar 2013 01:06:34 GMT\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3aaa91ec_39927d57","line":477,"updated":"2016-07-01 10:05:28.000000000","message":"this change also fixes a bug.\n\nI suggest to use a list-comprehension for readability:\n\n[x[\u0027name\u0027] for x in self._objects[args[0]]]","commit_id":"6fcc22e7a69f641523b30c1156bbfbade922da47"}]}