)]}'
{"keystone/common/sql/migrate_repo/versions/019_fixup_role.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"d77f7818be1749231725f86293358400f9eb874b","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    # blank \u0027extra\u0027 field should be \"{}\""},{"line_number":23,"context_line":"    update \u003d role_table.update().where(role_table.c.extra \u003d\u003d None).values("},{"line_number":24,"context_line":"        {role_table.c.extra: \"{}\"})"},{"line_number":25,"context_line":"    migrate_engine.execute(update)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F2MU%3D","line":24,"updated":"2013-03-09 00:15:42.000000000","message":"this looks like the first patch that you showed me -- is the only thing that\u0027s different about it that you had to provide references (role_table.c.extra) to the column in both cases?","commit_id":"9b13664d09e46ee87d4bd8054cd4c2990f369fd2"}],"tests/test_sql_upgrade.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"d77f7818be1749231725f86293358400f9eb874b","unresolved":false,"context_lines":[{"line_number":544,"context_line":"        self.upgrade(1)"},{"line_number":545,"context_line":"        self.insert_dict(session, \"role\", {\"id\": \"test\", \"name\": \"test\"})"},{"line_number":546,"context_line":"        self.upgrade(18)"},{"line_number":547,"context_line":"        self.insert_dict(session, \"role\", {\"id\": \"test2\", \"name\": \"test2\", \"extra\": None})"},{"line_number":548,"context_line":"        r \u003d session.execute(\u0027select count(*) as c from role where extra is null\u0027)"},{"line_number":549,"context_line":"        self.assertEqual(r.fetchone()[\u0027c\u0027], 2)"},{"line_number":550,"context_line":"        self.upgrade(19)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F2L4%3D","line":547,"updated":"2013-03-09 00:15:42.000000000","message":"should this one have some valid \"extra\" data so that you can test to ensure you\u0027re not overwriting it with \"{}\" as well?","commit_id":"9b13664d09e46ee87d4bd8054cd4c2990f369fd2"}]}