)]}'
{"nova/db/sqlalchemy/migrate_repo/versions/133_aggregate_delete_fix.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"38d4d34af926253ae98970f5b24bdb866b7b6df4","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    aggregates \u003d Table(\u0027aggregates\u0027, meta, autoload\u003dTrue)"},{"line_number":30,"context_line":"    if dialect.startswith(\u0027sqlite\u0027):"},{"line_number":31,"context_line":"        aggregates.drop_column(\u0027host\u0027)"},{"line_number":32,"context_line":"        aggregates.create_column(Column(\u0027host\u0027, String(255)))"},{"line_number":33,"context_line":"    elif dialect.startswith(\u0027postgres\u0027):"},{"line_number":34,"context_line":"        ucon \u003d UniqueConstraint(\u0027host\u0027,"},{"line_number":35,"context_line":"                                name\u003d\u0027aggregates_host_key\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAALH%2F%2FW0A%3D","line":32,"updated":"2012-09-19 00:36:10.000000000","message":"shouldn\u0027t there be some attempt to save and move that data?","commit_id":"e206e795ddfc5e689a466fe2855d25b4662a77fd"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"e4e781eb8e4f44571f4322d042811778fe38699b","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    aggregates \u003d Table(\u0027aggregates\u0027, meta, autoload\u003dTrue)"},{"line_number":30,"context_line":"    if dialect.startswith(\u0027sqlite\u0027):"},{"line_number":31,"context_line":"        aggregates.drop_column(\u0027host\u0027)"},{"line_number":32,"context_line":"        aggregates.create_column(Column(\u0027host\u0027, String(255)))"},{"line_number":33,"context_line":"    elif dialect.startswith(\u0027postgres\u0027):"},{"line_number":34,"context_line":"        ucon \u003d UniqueConstraint(\u0027host\u0027,"},{"line_number":35,"context_line":"                                name\u003d\u0027aggregates_host_key\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAALH%2F%2FWzc%3D","line":32,"in_reply_to":"AAAALH%2F%2FW0A%3D","updated":"2012-09-19 00:58:37.000000000","message":"Sean,   This is the sqlite only migration which *should* never be used in production.  That being said do have a recommendation on how to save the data?  My sqlalchemy is weak","commit_id":"e206e795ddfc5e689a466fe2855d25b4662a77fd"},{"author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"change_message_id":"1e7b56871f8298375a0f5bfa44b47032e6d60c51","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    if dialect.startswith(\u0027sqlite\u0027):"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"        aggregates.drop_column(\u0027name\u0027)"},{"line_number":33,"context_line":"        aggregates.create_column(Column(\u0027name\u0027, String(255)))"},{"line_number":34,"context_line":"    elif dialect.startswith(\u0027postgres\u0027):"},{"line_number":35,"context_line":"        ucon \u003d UniqueConstraint(\u0027name\u0027,"},{"line_number":36,"context_line":"                                name\u003d\u0027aggregates_name_key\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAALH%2F%2FWws%3D","line":33,"updated":"2012-09-19 04:36:47.000000000","message":"this will delete the existing aggregate data which is probably not what is intended. I see that we do the same in an earlier migration so I guess we are just assuming that no one is actually using sqlite in production.","commit_id":"231aa0e469502f6ad933181775401b400c713b71"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"fbc19295136f0ec7d0d7689da66ebc45d72ad094","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    if dialect.startswith(\u0027sqlite\u0027):"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"        aggregates.drop_column(\u0027name\u0027)"},{"line_number":33,"context_line":"        aggregates.create_column(Column(\u0027name\u0027, String(255)))"},{"line_number":34,"context_line":"    elif dialect.startswith(\u0027postgres\u0027):"},{"line_number":35,"context_line":"        ucon \u003d UniqueConstraint(\u0027name\u0027,"},{"line_number":36,"context_line":"                                name\u003d\u0027aggregates_name_key\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAALH%2F%2FWaw%3D","line":33,"in_reply_to":"AAAALH%2F%2FWws%3D","updated":"2012-09-19 15:18:08.000000000","message":"I remain uncomfortable with that as an assumption. Especially after Brian\u0027s findings on migrations from Essex. Can\u0027t we grab the data first, then do an update after the create.","commit_id":"231aa0e469502f6ad933181775401b400c713b71"}]}