)]}'
{"subunit2sql/migrations/versions/36f33ef8b283_add_attrs_column_to_test_runs.py":[{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"809fab300c687b59eab440d2e3680d7fd36a303b","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        api.update_test_run({\u0027attrs\u0027: \u0027,\u0027.join(update_dict[test_run])},"},{"line_number":57,"context_line":"                            test_run, session)"},{"line_number":58,"context_line":"    session.close()"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"def downgrade():"},{"line_number":62,"context_line":"    op.delete_column(\u0027test_runs\u0027, \u0027attrs\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"3a961159_2b24f5c4","line":59,"updated":"2015-01-14 15:59:37.000000000","message":"shouldn\u0027t we also remove attrs\u0027s rows from test_run_metadata table?\nIf so downgrade also requires that you move attrs from test_run back to test_run_metadata","commit_id":"0564ba8b575f53d8411039bfb88615335d691208"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"a67798fbb5bb8505c0d57e68f70f5a00aa0e3ac9","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        api.update_test_run({\u0027attrs\u0027: \u0027,\u0027.join(update_dict[test_run])},"},{"line_number":57,"context_line":"                            test_run, session)"},{"line_number":58,"context_line":"    session.close()"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"def downgrade():"},{"line_number":62,"context_line":"    op.delete_column(\u0027test_runs\u0027, \u0027attrs\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1a930d6b_a16ab87e","line":59,"in_reply_to":"3a961159_2b24f5c4","updated":"2015-01-25 02:58:32.000000000","message":"That was my original thought here too, but afterthinking  a while I figured that since it\u0027s a key value store it doesn\u0027t really matter if we delete it or not. But, that was me probably just being lazy. (ie making the downgrade simpler)  If you think it\u0027s worth the effort I can respin this to add a row delete.","commit_id":"0564ba8b575f53d8411039bfb88615335d691208"}]}