)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"4f95ecd5633967b0e2728787d14f3aa366e1f501","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The IDL change_seqno can change as a result of running idl.run(), and"},{"line_number":10,"context_line":"it returns True if there has been a change. The existing code would"},{"line_number":11,"context_line":"block without checking the return value, and therefor could block"},{"line_number":12,"context_line":"until another change occurred, or the timeout was reached."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I9b7aac2755e6e6b69165023e01583337f9d30f15"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a1a9d01_bb2f2f87","line":12,"updated":"2015-09-28 22:54:11.000000000","message":"does this warrant a bug report? Wonder if it\u0027s worth backporting...","commit_id":"6806cd16a4a67fc9b4eded988de68c8e49bcdb17"},{"author":{"_account_id":11682,"name":"Ryan Moats","email":"rmoats@us.ibm.com","username":"regXboi"},"change_message_id":"1280593ddf5c0a294c105aa37fc2ff09e03c6214","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The IDL change_seqno can change as a result of running idl.run(), and"},{"line_number":10,"context_line":"it returns True if there has been a change. The existing code would"},{"line_number":11,"context_line":"block without checking the return value, and therefor could block"},{"line_number":12,"context_line":"until another change occurred, or the timeout was reached."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I9b7aac2755e6e6b69165023e01583337f9d30f15"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a1a9d01_6949a35f","line":12,"in_reply_to":"9a1a9d01_6f3e9029","updated":"2015-09-29 20:36:04.000000000","message":"russelb, I\u0027d like to eventually change that.","commit_id":"6806cd16a4a67fc9b4eded988de68c8e49bcdb17"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"f92aebd2813a1b9e0f673fa30a0d54e266a067d2","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The IDL change_seqno can change as a result of running idl.run(), and"},{"line_number":10,"context_line":"it returns True if there has been a change. The existing code would"},{"line_number":11,"context_line":"block without checking the return value, and therefor could block"},{"line_number":12,"context_line":"until another change occurred, or the timeout was reached."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I9b7aac2755e6e6b69165023e01583337f9d30f15"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a1a9d01_6f3e9029","line":12,"in_reply_to":"9a1a9d01_bb2f2f87","updated":"2015-09-29 16:49:30.000000000","message":"I\u0027d backport this.  It\u0027s a simple low-risk fix, especially given that this code path isn\u0027t the default.","commit_id":"6806cd16a4a67fc9b4eded988de68c8e49bcdb17"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"a0c876e17acb87eefb95688ca4dbca20a2bab2cb","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The IDL change_seqno can change as a result of running idl.run(), and"},{"line_number":10,"context_line":"it returns True if there has been a change. The existing code would"},{"line_number":11,"context_line":"block without checking the return value, and therefor could block"},{"line_number":12,"context_line":"until another change occurred, or the timeout was reached."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I9b7aac2755e6e6b69165023e01583337f9d30f15"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a1a9d01_3ba23f4c","line":12,"in_reply_to":"9a1a9d01_bb2f2f87","updated":"2015-09-29 00:46:26.000000000","message":"In practice, it basically means connecting to the OVSDB takes about 2-3 seconds longer than it should. And connections *should* only happen once per import (or if we have to reconnect).\n\nSo I think it\u0027s \"yeah, this code is a bit dumb, but probably the only time people will really notice it is when running single OVS functional tests where native OVSDB looks slower than vsctl.\"\n\nWith that said, it *should* be a fairly safe change to backport.","commit_id":"6806cd16a4a67fc9b4eded988de68c8e49bcdb17"}]}