)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"295d329c00c80fdd636f5a118e28d6076dc4ce0d","unresolved":false,"context_lines":[{"line_number":10,"context_line":"and recreated in 047 migration script[1]. After 047, we should make"},{"line_number":11,"context_line":"sure the change to these two tables has been addressed."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This patch add the check in 048 contract script to ensure the change"},{"line_number":14,"context_line":"is adopted indeed."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[1]: Ibb408758466ff367f57bafbd4b8c9213499f8dc3"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"bfdaf3ff_e48dadd3","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":18},"updated":"2019-01-15 21:15:13.000000000","message":"If we don\u0027t do this does it cause failures with testing subsequent patches?","commit_id":"252aedf7fde2a314ac0b2a0ba373b1fffc5bb770"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"2424dcae0206f32839a83fe3bb1d819fc0230d85","unresolved":false,"context_lines":[{"line_number":10,"context_line":"and recreated in 047 migration script[1]. After 047, we should make"},{"line_number":11,"context_line":"sure the change to these two tables has been addressed."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This patch add the check in 048 contract script to ensure the change"},{"line_number":14,"context_line":"is adopted indeed."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[1]: Ibb408758466ff367f57bafbd4b8c9213499f8dc3"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"bfdaf3ff_c4abc91a","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":18},"in_reply_to":"bfdaf3ff_e48dadd3","updated":"2019-01-15 21:16:09.000000000","message":"Nevermind - I answered my own question when I realized I asked something similar on another patch.","commit_id":"252aedf7fde2a314ac0b2a0ba373b1fffc5bb770"}],"keystone/common/sql/expand_repo/versions/048_expand_add_registered_limit_id_column_for_limit.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"3b5adbfb14094bc8464aec78be48e7a42adfc274","unresolved":false,"context_lines":[{"line_number":25,"context_line":"        sql.ForeignKey(registered_limit_table.c.id))"},{"line_number":26,"context_line":"    limit_table.create_column(registered_limit_id)"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    # This is the new table created in 047 migration script for SQLite case."},{"line_number":29,"context_line":"    try:"},{"line_number":30,"context_line":"        limit_table_new \u003d sql.Table(\u0027limit_new\u0027, meta, autoload\u003dTrue)"},{"line_number":31,"context_line":"        registered_limit_id \u003d sql.Column("}],"source_content_type":"text/x-python","patch_set":10,"id":"9fdfeff1_32a67355","line":28,"updated":"2019-02-11 20:09:13.000000000","message":"For safety, should we check to make sure we\u0027re actually operating on a SQLite database before running this code?","commit_id":"556532abfc1b1750c32d078e33d61838935619a0"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"c1006c19f39d329cc542cfe42ca46279add72987","unresolved":false,"context_lines":[{"line_number":25,"context_line":"        sql.ForeignKey(registered_limit_table.c.id))"},{"line_number":26,"context_line":"    limit_table.create_column(registered_limit_id)"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    # This is the new table created in 047 migration script for SQLite case."},{"line_number":29,"context_line":"    try:"},{"line_number":30,"context_line":"        limit_table_new \u003d sql.Table(\u0027limit_new\u0027, meta, autoload\u003dTrue)"},{"line_number":31,"context_line":"        registered_limit_id \u003d sql.Column("}],"source_content_type":"text/x-python","patch_set":10,"id":"9fdfeff1_a53d3a8f","line":28,"in_reply_to":"9fdfeff1_32a67355","updated":"2019-02-11 23:27:42.000000000","message":"Yes we should. We will also need a new meta afaik, just to be sure.","commit_id":"556532abfc1b1750c32d078e33d61838935619a0"}]}