)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"8276f33e050e7f1c83621f5e1c425d9dd699c09b","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2014-08-20 20:37:52 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make some sqlite UPDATE calls only to records that have changed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I057a5f8971f06171ed966505e3b5092324f16ec9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1abeadc6_9e18cf1f","line":8,"updated":"2014-08-20 21:21:00.000000000","message":"You should put the info in your comment (below) here in the commit message and also reference the bug that you are addressing with the last line in your msg being \"Closes-Bug: xxxxxxx\"","commit_id":"5acc81a64f51c9a48ce4fc258b97528c1c0673d2"},{"author":{"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},"change_message_id":"64c29e97f6da3b32f981ba048fa2a02e12188d5d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Mark Seger \u003cMark.Seger@hp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-08-21 13:55:08 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make some sqlite UPDATE calls only to records that have changed"},{"line_number":8,"context_line":"Closes-Bug: 1332025"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I057a5f8971f06171ed966505e3b5092324f16ec9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1abeadc6_5fa8d576","line":7,"updated":"2014-08-21 16:08:19.000000000","message":"Not sure...but I think you need a blank line after the first line in a commit message","commit_id":"de122d373294f7cdbca49bb5c75b0a8ca2e593b8"}],"swift/common/db.py":[{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"8276f33e050e7f1c83621f5e1c425d9dd699c09b","unresolved":false,"context_lines":[{"line_number":456,"context_line":"        \"\"\""},{"line_number":457,"context_line":"        with self.get() as conn:"},{"line_number":458,"context_line":"            old_status \u003d self._is_deleted(conn)"},{"line_number":459,"context_line":""},{"line_number":460,"context_line":"            # get current times and only do UPDATE if one changed"},{"line_number":461,"context_line":"            row \u003d conn.execute(\u0027\u0027\u0027"},{"line_number":462,"context_line":"                SELECT created_at, put_timestamp, delete_timestamp from %s_stat"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_7ecdb384","line":459,"updated":"2014-08-20 21:21:00.000000000","message":"rather than do 2 conn.execute() statements, do:\ncurs \u003d conn.cursor()\nand then\nrow \u003d curs.execute() and later just curr.execute() to save one temporary cursor when you are going to execute both.","commit_id":"5acc81a64f51c9a48ce4fc258b97528c1c0673d2"},{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"8276f33e050e7f1c83621f5e1c425d9dd699c09b","unresolved":false,"context_lines":[{"line_number":474,"context_line":"                    timestamp \u003d Timestamp(time.time())"},{"line_number":475,"context_line":"                    self._update_status_changed_at(conn, timestamp.internal)"},{"line_number":476,"context_line":""},{"line_number":477,"context_line":"            conn.commit()"},{"line_number":478,"context_line":""},{"line_number":479,"context_line":"    def get_items_since(self, start, count):"},{"line_number":480,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_1e9e7f07","line":477,"updated":"2014-08-20 21:21:00.000000000","message":"pretty sure you wanted this within the conditional you just added","commit_id":"5acc81a64f51c9a48ce4fc258b97528c1c0673d2"},{"author":{"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},"change_message_id":"64c29e97f6da3b32f981ba048fa2a02e12188d5d","unresolved":false,"context_lines":[{"line_number":456,"context_line":"        \"\"\""},{"line_number":457,"context_line":"        with self.get() as conn:"},{"line_number":458,"context_line":"            old_status \u003d self._is_deleted(conn)"},{"line_number":459,"context_line":""},{"line_number":460,"context_line":"            # get current times and only do UPDATE if one changed"},{"line_number":461,"context_line":"            cur \u003d conn.execute(\u0027\u0027\u0027"},{"line_number":462,"context_line":"                SELECT created_at, put_timestamp, delete_timestamp from %s_stat"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_622a5a91","line":459,"updated":"2014-08-21 16:08:19.000000000","message":"This function is used in two places: in the account/container-server (as a result of sync) and in container-replicator. This patch fixes the second case (container-replicator), but at the expense of adding an extra SELECT for the account/container-servers. They call _get_synced_replicator_info() and check the timestamps before calling merge_timestamps()","commit_id":"de122d373294f7cdbca49bb5c75b0a8ca2e593b8"}],"swift/container/backend.py":[{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"8276f33e050e7f1c83621f5e1c425d9dd699c09b","unresolved":false,"context_lines":[{"line_number":771,"context_line":"        \u0027\u0027\u0027"},{"line_number":772,"context_line":"        with self.get() as conn:"},{"line_number":773,"context_line":"            try:"},{"line_number":774,"context_line":"                # only do update if sync point changed"},{"line_number":775,"context_line":"                if conn.execute(\u0027\u0027\u0027"},{"line_number":776,"context_line":"                    SELECT reconciler_sync_point from container_stat"},{"line_number":777,"context_line":"                    \u0027\u0027\u0027).fetchone()[0] !\u003d point:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_5e971748","line":774,"updated":"2014-08-20 21:21:00.000000000","message":"same comment as earlier, setup a single cursor so you don\u0027t use 2 when executing both commands","commit_id":"5acc81a64f51c9a48ce4fc258b97528c1c0673d2"}]}