)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"12c9e089190aad41f40696dd26a705339caf3991","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In python 2.7.5 the query is extracted from the path and can be found in"},{"line_number":10,"context_line":"in the \u0027query\u0027 member of ParseResult  while in python 2.6 it is left"},{"line_number":11,"context_line":"unparsed in the path member."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ic6afbe7b431e3eb9ff28d3c9b3fa35a52efb3fb6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAP3%2F%2Fgco%3D","line":11,"updated":"2013-07-17 07:50:39.000000000","message":"This should say: \"where in python 2.6, query will not be parsed for all schemes which include the 2 we are interested in here\".\n\nThis will make it clear that the issue is related to the url scheme","commit_id":"96a71a65f0e0130976b1aa878e710f4baf1e6939"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"51c5a038024c562e2e3eaab54e95ae04f72291ef","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In python 2.7.5 the query is extracted from the path and can be found in"},{"line_number":10,"context_line":"in the \u0027query\u0027 member of ParseResult  while in python 2.6 it is left"},{"line_number":11,"context_line":"unparsed in the path member."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ic6afbe7b431e3eb9ff28d3c9b3fa35a52efb3fb6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAP3%2F%2Fepk%3D","line":11,"in_reply_to":"AAAAP3%2F%2Fgco%3D","updated":"2013-07-17 16:41:23.000000000","message":"I\u0027ll comment that I tested this on Python 2.7 and 2.7.3, and both of those versions acted identically to the behavior in Python 2.6.","commit_id":"96a71a65f0e0130976b1aa878e710f4baf1e6939"}],"nova/tests/cells/test_cells_rpc_driver.py":[{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"6b44539c2e6b11955285b5e27b50f96690080e2c","unresolved":false,"context_lines":[{"line_number":230,"context_line":"        url \u003d \"rabbit://u:p@h:10/virtual?ssl\u003d1\""},{"line_number":231,"context_line":"        self.assertRaises(ValueError, rpc_driver.parse_transport_url, url)"},{"line_number":232,"context_line":""},{"line_number":233,"context_line":"    def test_query_string_old_urlparse(self):"},{"line_number":234,"context_line":"        url \u003d \"rabbit://u:p@h:10/virtual?ssl\u003d1\""},{"line_number":235,"context_line":""},{"line_number":236,"context_line":"        python_old_parse_result \u003d urlparse.ParseResult("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAP3%2F%2Ffvs%3D","line":233,"updated":"2013-07-17 12:25:43.000000000","message":"Just a suggestion - might be good to add the bug number to the name as this test is very specific and weird and people will wander what the...","commit_id":"1b0e79d1d4aa0028ff5508d3ed6ea160a317802d"},{"author":{"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},"change_message_id":"77947914c37f815bab7a91791256b1aeafb6edb3","unresolved":false,"context_lines":[{"line_number":245,"context_line":"        self.assertRaises(ValueError, rpc_driver.parse_transport_url, url)"},{"line_number":246,"context_line":""},{"line_number":247,"context_line":"    def test_query_string_new_urlparse(self):"},{"line_number":248,"context_line":"        url \u003d \"rabbit://u:p@h:10/virtual?ssl\u003d1\""},{"line_number":249,"context_line":""},{"line_number":250,"context_line":"        python_new_parse_result \u003d urlparse.ParseResult("},{"line_number":251,"context_line":"            scheme\u003d\u0027rabbit\u0027, netloc\u003d\u0027u:p@h:10\u0027, path\u003d\u0027/virtual\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2F%2FuM%3D","line":248,"updated":"2013-07-17 20:05:43.000000000","message":"There should be a note here to explain what the purpose of the \"new\" versus \"old\" test is.","commit_id":"febe711c9cd76f151e072002333766affc04747f"},{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"0309bc8fd059ca59fa3504415ade0920ddef3108","unresolved":false,"context_lines":[{"line_number":245,"context_line":"        self.assertRaises(ValueError, rpc_driver.parse_transport_url, url)"},{"line_number":246,"context_line":""},{"line_number":247,"context_line":"    def test_query_string_new_urlparse(self):"},{"line_number":248,"context_line":"        url \u003d \"rabbit://u:p@h:10/virtual?ssl\u003d1\""},{"line_number":249,"context_line":""},{"line_number":250,"context_line":"        python_new_parse_result \u003d urlparse.ParseResult("},{"line_number":251,"context_line":"            scheme\u003d\u0027rabbit\u0027, netloc\u003d\u0027u:p@h:10\u0027, path\u003d\u0027/virtual\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2F%2BIc%3D","line":248,"in_reply_to":"AAAAQn%2F%2F%2FuM%3D","updated":"2013-07-18 09:07:17.000000000","message":"I had the same comment on the previous patch. Thanks for updating Xavier","commit_id":"febe711c9cd76f151e072002333766affc04747f"}]}