)]}'
{"swift/common/db.py":[{"author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"change_message_id":"68c818fd6fe2ded84091cd2b2057cca83ba884e6","unresolved":false,"context_lines":[{"line_number":179,"context_line":"        self.conn \u003d None"},{"line_number":180,"context_line":"        self.db_file \u003d db_file"},{"line_number":181,"context_line":"        self.pending_file \u003d self.db_file + \u0027.pending\u0027"},{"line_number":182,"context_line":"        self.pending_timeout \u003d pending_timeout or 10"},{"line_number":183,"context_line":"        self.stale_reads_ok \u003d stale_reads_ok"},{"line_number":184,"context_line":"        self.db_dir \u003d os.path.dirname(db_file)"},{"line_number":185,"context_line":"        self.timeout \u003d timeout"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FRx8%3D","line":182,"updated":"2013-08-02 17:32:22.000000000","message":"it\u0027s perhaps out of the scope from this patch but I wonder if there is some use cases to have this time-out as configuration for busy system with a lot of container operations that can afford to wait for the file being available.","commit_id":"f242311b209d9aa634622545b1818ac81521ed72"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"cffc05ecb737a17237d4259d1d40f0d59674ba6d","unresolved":false,"context_lines":[{"line_number":179,"context_line":"        self.conn \u003d None"},{"line_number":180,"context_line":"        self.db_file \u003d db_file"},{"line_number":181,"context_line":"        self.pending_file \u003d self.db_file + \u0027.pending\u0027"},{"line_number":182,"context_line":"        self.pending_timeout \u003d pending_timeout or 10"},{"line_number":183,"context_line":"        self.stale_reads_ok \u003d stale_reads_ok"},{"line_number":184,"context_line":"        self.db_dir \u003d os.path.dirname(db_file)"},{"line_number":185,"context_line":"        self.timeout \u003d timeout"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FQqY%3D","line":182,"in_reply_to":"AAAAQn%2F%2FRx8%3D","updated":"2013-08-03 03:10:11.000000000","message":"The default of 10 allows Proxy to cycle over broken nodes within a typical client timeout of 30.","commit_id":"f242311b209d9aa634622545b1818ac81521ed72"},{"author":{"_account_id":6198,"name":"Peter Portante","email":"peter.a.portante@gmail.com","username":"peter-a-portante"},"change_message_id":"985503ac4fd22143d7cfd8105fe09799c691b8fd","unresolved":false,"context_lines":[{"line_number":928,"context_line":"        if pending_size \u003e PENDING_CAP:"},{"line_number":929,"context_line":"            self._commit_puts([record])"},{"line_number":930,"context_line":"        else:"},{"line_number":931,"context_line":"            with lock_parent_directory(self.pending_file,"},{"line_number":932,"context_line":"                                       self.pending_timeout):"},{"line_number":933,"context_line":"                with open(self.pending_file, \u0027a+b\u0027) as fp:"},{"line_number":934,"context_line":"                    # Colons aren\u0027t used in base64 encoding; so they are our"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FYf4%3D","line":931,"updated":"2013-08-01 02:01:42.000000000","message":"Did this line violate pep8?","commit_id":"f242311b209d9aa634622545b1818ac81521ed72"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"5c188af87cdd17d87c99aec6a2b0626f428dd94a","unresolved":false,"context_lines":[{"line_number":928,"context_line":"        if pending_size \u003e PENDING_CAP:"},{"line_number":929,"context_line":"            self._commit_puts([record])"},{"line_number":930,"context_line":"        else:"},{"line_number":931,"context_line":"            with lock_parent_directory(self.pending_file,"},{"line_number":932,"context_line":"                                       self.pending_timeout):"},{"line_number":933,"context_line":"                with open(self.pending_file, \u0027a+b\u0027) as fp:"},{"line_number":934,"context_line":"                    # Colons aren\u0027t used in base64 encoding; so they are our"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FYX4%3D","line":931,"in_reply_to":"AAAAQn%2F%2FYf4%3D","updated":"2013-08-01 03:20:38.000000000","message":"No, it didn\u0027t violate PEP8, at least tox -e pep8 didn\u0027t complain. It\u0027s just that the code has another one almost exactly like this and I made them identical.","commit_id":"f242311b209d9aa634622545b1818ac81521ed72"}]}