)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"71ebf6662c4ebc025ef8b1a31ae8dd3feb9228ba","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fixes: Bug 1550494"},{"line_number":8,"context_line":"Converted String timestamp to int"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: Ib9ba662f1497bc0691ba1f81cf4134ca3e72acc4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a77a97e_756ce936","line":10,"range":{"start_line":10,"start_character":11,"end_line":10,"end_character":52},"updated":"2016-11-21 22:41:38.000000000","message":"Could you please add \n\n   Closes-Bug: #\u003cbug id\u003e \n\nLets launchpad track the fix","commit_id":"c54cb3d13ac234fca2ea1d546d6a16bc17876386"}],"searchlight/tests/unit/test_elasticsearch_version.py":[{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"71ebf6662c4ebc025ef8b1a31ae8dd3feb9228ba","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        self.timestamp \u003d \u00272015-12-15 06:40:55.012316\u0027"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    def get_interval(self, version1, version2):"},{"line_number":33,"context_line":"        return int(str(version1)[:9]) - int(str(version2)[:9])"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def test_use_update_with_timestamp(self):"},{"line_number":36,"context_line":"        version \u003d NotificationBase.get_version(self.payload, self.timestamp)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a77a97e_df1cfec3","line":33,"range":{"start_line":33,"start_character":23,"end_line":33,"end_character":31},"updated":"2016-11-21 22:41:38.000000000","message":"Since we know we\u0027re dealing with 18 digit numbers and we\u0027re only interested in 9 of them this can be done with integer division (and a note indicating we\u0027re only interested in the resource date, not the messaging one):\n\n   return (version1 // 10**9) - (version2 // 10**9)\n\nI would argue this is more correct since really we want to ignore the 9 least significant digits, and only take into account the most significant digits regardless of length","commit_id":"c54cb3d13ac234fca2ea1d546d6a16bc17876386"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"71ebf6662c4ebc025ef8b1a31ae8dd3feb9228ba","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    def test_use_update_with_timestamp(self):"},{"line_number":36,"context_line":"        version \u003d NotificationBase.get_version(self.payload, self.timestamp)"},{"line_number":37,"context_line":"        self.assertEqual(450161655161655012, version)"},{"line_number":38,"context_line":"        self.assertEqual(18, len(str(version)))"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        aware_version \u003d NotificationBase.get_version(self.aware_payload,"},{"line_number":41,"context_line":"                                                     self.timestamp)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a77a97e_7f768ad9","line":38,"range":{"start_line":38,"start_character":32,"end_line":38,"end_character":33},"updated":"2016-11-21 22:41:38.000000000","message":"Could also use assertGreater(a, 10**17). Not going to reject the patch for that, but since we\u0027re dealing with a number it makes sense to express the checks as one. As a bonus it\u0027ll handle the rollover properly in a few centuries time!","commit_id":"c54cb3d13ac234fca2ea1d546d6a16bc17876386"}]}