)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"62c50ea6e124d7c783d018214a06cc5d1a4d9bd1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix for bug 887712"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Check weather the instance_id passed into instance_update() is a model id"},{"line_number":10,"context_line":"or a UUID."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic361299cdbfa5c3786815753c47a8e4f3ba6d494"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAACn%2F%2F8QY%3D","line":9,"updated":"2011-11-28 21:28:18.000000000","message":"Minor nit—\"whether\"","commit_id":"168d9e9ae7c3aabc33f78dfe40abdbc54717468f"}],"nova/db/sqlalchemy/api.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"62c50ea6e124d7c783d018214a06cc5d1a4d9bd1","unresolved":false,"context_lines":[{"line_number":1581,"context_line":"    metadata \u003d values.get(\u0027metadata\u0027)"},{"line_number":1582,"context_line":"    if metadata is not None:"},{"line_number":1583,"context_line":"        instance_metadata_update(context,"},{"line_number":1584,"context_line":"                                 instance_ref.id,"},{"line_number":1585,"context_line":"                                 values.pop(\u0027metadata\u0027),"},{"line_number":1586,"context_line":"                                 delete\u003dTrue)"},{"line_number":1587,"context_line":"    with session.begin():"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACn%2F%2F8QI%3D","line":1584,"updated":"2011-11-28 21:28:18.000000000","message":"Aren\u0027t we using item access style elsewhere, i.e., \"instance_ref[\u0027id\u0027]\"?  We should probably strive for consistent style.","commit_id":"168d9e9ae7c3aabc33f78dfe40abdbc54717468f"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"62c50ea6e124d7c783d018214a06cc5d1a4d9bd1","unresolved":false,"context_lines":[{"line_number":1584,"context_line":"                                 instance_ref.id,"},{"line_number":1585,"context_line":"                                 values.pop(\u0027metadata\u0027),"},{"line_number":1586,"context_line":"                                 delete\u003dTrue)"},{"line_number":1587,"context_line":"    with session.begin():"},{"line_number":1588,"context_line":"        instance_ref.update(values)"},{"line_number":1589,"context_line":"        instance_ref.save(session\u003dsession)"},{"line_number":1590,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACn%2F%2F8QM%3D","line":1587,"updated":"2011-11-28 21:28:18.000000000","message":"The one thing that bothers me about this change is that it seems like we\u0027re putting less and less stuff under the protection of a transaction.  Not certain how necessary that really is, but it\u0027s something I\u0027m concerned about.","commit_id":"168d9e9ae7c3aabc33f78dfe40abdbc54717468f"}]}