)]}'
{"ceilometer/storage/impl_sqlalchemy.py":[{"author":{"_account_id":7368,"name":"Thomas Maddox","email":"me@thomasem.net","username":"thomasm"},"change_message_id":"ae1c30fee0b61cccde5a930daea8332e758b343e","unresolved":false,"context_lines":[{"line_number":690,"context_line":"        with session.begin():"},{"line_number":691,"context_line":"            session.merge(User(id\u003dalarm_change[\u0027user_id\u0027]))"},{"line_number":692,"context_line":"            session.merge(Project(id\u003dalarm_change[\u0027project_id\u0027]))"},{"line_number":693,"context_line":"            session.merge(Project(id\u003dalarm_change[\u0027on_behalf_of\u0027]))"},{"line_number":694,"context_line":"            alarm_change_row \u003d AlarmChange(event_id\u003dalarm_change[\u0027event_id\u0027])"},{"line_number":695,"context_line":"            alarm_change_row.update(alarm_change)"},{"line_number":696,"context_line":"            session.add(alarm_change_row)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAASn%2F%2FK18%3D","line":693,"updated":"2013-09-10 17:51:36.000000000","message":"Newbie question: why did you choose to use session.merge(...) here rather than session.add(...) or just applying the attributes to the AlarmChange model and letting the FK constraint do the work of relating those?","commit_id":"e393935eda4fa9f4d560d3710826f79ab0b876d7"}],"ceilometer/storage/sqlalchemy/migrate_repo/versions/015_add_alarm_history_table.py":[{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"2399a5404d9e3b3323e97785eb78e801495b463a","unresolved":false,"context_lines":[{"line_number":25,"context_line":"def upgrade(migrate_engine):"},{"line_number":26,"context_line":"    meta.bind \u003d migrate_engine"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    project \u003d Table(\u0027project\u0027, meta, autoload\u003dTrue)"},{"line_number":29,"context_line":"    user \u003d Table(\u0027user\u0027, meta, autoload\u003dTrue)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    alarm_history \u003d Table("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FVOI%3D","line":28,"updated":"2013-09-06 07:21:10.000000000","message":"Extra load of table.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"1c8477fbaa9d515ce20aa73e114f8a53da2787f4","unresolved":false,"context_lines":[{"line_number":25,"context_line":"def upgrade(migrate_engine):"},{"line_number":26,"context_line":"    meta.bind \u003d migrate_engine"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    project \u003d Table(\u0027project\u0027, meta, autoload\u003dTrue)"},{"line_number":29,"context_line":"    user \u003d Table(\u0027user\u0027, meta, autoload\u003dTrue)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    alarm_history \u003d Table("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FTeU%3D","line":28,"in_reply_to":"AAAASn%2F%2FUmY%3D","updated":"2013-09-06 17:31:37.000000000","message":"Without autoloading the project \u0026 user tables, I see \u0027AttributeError: id\u0027 failures when the ForeignKeyConstraint is created.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"52d665737c56dd704a69fac5e1ee2ddee0a2749b","unresolved":false,"context_lines":[{"line_number":25,"context_line":"def upgrade(migrate_engine):"},{"line_number":26,"context_line":"    meta.bind \u003d migrate_engine"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    project \u003d Table(\u0027project\u0027, meta, autoload\u003dTrue)"},{"line_number":29,"context_line":"    user \u003d Table(\u0027user\u0027, meta, autoload\u003dTrue)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    alarm_history \u003d Table("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FUmY%3D","line":28,"in_reply_to":"AAAASn%2F%2FUvY%3D","updated":"2013-09-06 11:40:44.000000000","message":"You can use ForeignKeyConstraint for this.\nfkey \u003d ForeignKeyConstraint(columns\u003d[table.c.column1, table.c.column2], \n                                    refcolumns\u003d[ref_table.c.column1, ref_table.c.column2])\nfkey.create(engine\u003dmigrate_engine)\nIn this case project and user table should be loaded.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"b5d5c06bd4c3b3fada261cc7f56f559b56d2dbb4","unresolved":false,"context_lines":[{"line_number":25,"context_line":"def upgrade(migrate_engine):"},{"line_number":26,"context_line":"    meta.bind \u003d migrate_engine"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    project \u003d Table(\u0027project\u0027, meta, autoload\u003dTrue)"},{"line_number":29,"context_line":"    user \u003d Table(\u0027user\u0027, meta, autoload\u003dTrue)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    alarm_history \u003d Table("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FUvY%3D","line":28,"in_reply_to":"AAAASn%2F%2FVOI%3D","updated":"2013-09-06 11:05:01.000000000","message":"Hmmm, I thought that loading the table would be required as it\u0027s referred to below in the ForeignKey declarations. Is that not the case?\n\n(I found that not both declaring and loading the table caused the FK creation to fail with NoReferenced{Table|Column}Error).","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"2399a5404d9e3b3323e97785eb78e801495b463a","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    meta.bind \u003d migrate_engine"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    project \u003d Table(\u0027project\u0027, meta, autoload\u003dTrue)"},{"line_number":29,"context_line":"    user \u003d Table(\u0027user\u0027, meta, autoload\u003dTrue)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    alarm_history \u003d Table("},{"line_number":32,"context_line":"        \u0027alarm_history\u0027, meta,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FVOE%3D","line":29,"updated":"2013-09-06 07:21:10.000000000","message":"The same.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"2399a5404d9e3b3323e97785eb78e801495b463a","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    alarm_history \u003d Table("},{"line_number":32,"context_line":"        \u0027alarm_history\u0027, meta,"},{"line_number":33,"context_line":"        Column(\u0027event_id\u0027, String(255), primary_key\u003dTrue, index\u003dTrue),"},{"line_number":34,"context_line":"        Column(\u0027alarm_id\u0027, String(255), index\u003dTrue),"},{"line_number":35,"context_line":"        Column(\u0027on_behalf_of\u0027, String(255),"},{"line_number":36,"context_line":"               ForeignKey(\u0027%s.id\u0027 % project.name), index\u003dTrue),"},{"line_number":37,"context_line":"        Column(\u0027project_id\u0027, String(255),"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FVNU%3D","line":34,"updated":"2013-09-06 07:21:10.000000000","message":"In index\u003dTrue case we will have different name of index in different backends.\nIt will be better to use Index(\u0027ix_alarm_history_alarm_id\u0027, alarm_history.alarm_id).","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"52d665737c56dd704a69fac5e1ee2ddee0a2749b","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    alarm_history \u003d Table("},{"line_number":32,"context_line":"        \u0027alarm_history\u0027, meta,"},{"line_number":33,"context_line":"        Column(\u0027event_id\u0027, String(255), primary_key\u003dTrue, index\u003dTrue),"},{"line_number":34,"context_line":"        Column(\u0027alarm_id\u0027, String(255), index\u003dTrue),"},{"line_number":35,"context_line":"        Column(\u0027on_behalf_of\u0027, String(255),"},{"line_number":36,"context_line":"               ForeignKey(\u0027%s.id\u0027 % project.name), index\u003dTrue),"},{"line_number":37,"context_line":"        Column(\u0027project_id\u0027, String(255),"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FUnw%3D","line":34,"in_reply_to":"AAAASn%2F%2FUt0%3D","updated":"2013-09-06 11:40:44.000000000","message":"You should create alarm_history at first, and then you should do\nIndex(\u0027ix_alarm_history_alarm_id\u0027, alarm_history.alarm_id).create(migrate_engine)","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"b5d5c06bd4c3b3fada261cc7f56f559b56d2dbb4","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    alarm_history \u003d Table("},{"line_number":32,"context_line":"        \u0027alarm_history\u0027, meta,"},{"line_number":33,"context_line":"        Column(\u0027event_id\u0027, String(255), primary_key\u003dTrue, index\u003dTrue),"},{"line_number":34,"context_line":"        Column(\u0027alarm_id\u0027, String(255), index\u003dTrue),"},{"line_number":35,"context_line":"        Column(\u0027on_behalf_of\u0027, String(255),"},{"line_number":36,"context_line":"               ForeignKey(\u0027%s.id\u0027 % project.name), index\u003dTrue),"},{"line_number":37,"context_line":"        Column(\u0027project_id\u0027, String(255),"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FUt0%3D","line":34,"in_reply_to":"AAAASn%2F%2FVNU%3D","updated":"2013-09-06 11:05:01.000000000","message":"I\u0027m unsure as to what change exactly you\u0027re suggesting here.\n\nReplacing:\n\n  alarm_history \u003d Table(\n      \u0027alarm_history\u0027, meta,\n      Column(\u0027event_id\u0027, String(255), primary_key\u003dTrue, index\u003dTrue),\n      Column(\u0027alarm_id\u0027, String(255), index\u003dTrue),\n      ...))\n\nwith:\n\n\n  alarm_history \u003d Table(\n      \u0027alarm_history\u0027, meta,\n      Column(\u0027event_id\u0027, String(255), primary_key\u003dTrue, index\u003dTrue),\n      Column(\u0027alarm_id\u0027, String(255),\n                  Index(\u0027ix_alarm_history_alarm_id\u0027, alarm_history.alarm_id)),\n       ...))\n\nis clearly not what you meant, as that refers to alarm_history before it\u0027s assigned.\n\nWhereas simply creating the index later, e.g:\n\n  alarm_history \u003d Table(\n      \u0027alarm_history\u0027, meta,\n      Column(\u0027event_id\u0027, String(255), primary_key\u003dTrue, index\u003dTrue),\n      Column(\u0027alarm_id\u0027, String(255)),\n      ...))\n   Index(\u0027ix_alarm_history_alarm_id\u0027, alarm_history.alarm_id)\n\ndoes not appear to have the desired effect.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"2399a5404d9e3b3323e97785eb78e801495b463a","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        Column(\u0027event_id\u0027, String(255), primary_key\u003dTrue, index\u003dTrue),"},{"line_number":34,"context_line":"        Column(\u0027alarm_id\u0027, String(255), index\u003dTrue),"},{"line_number":35,"context_line":"        Column(\u0027on_behalf_of\u0027, String(255),"},{"line_number":36,"context_line":"               ForeignKey(\u0027%s.id\u0027 % project.name), index\u003dTrue),"},{"line_number":37,"context_line":"        Column(\u0027project_id\u0027, String(255),"},{"line_number":38,"context_line":"               ForeignKey(\u0027%s.id\u0027 % project.name), index\u003dTrue),"},{"line_number":39,"context_line":"        Column(\u0027user_id\u0027, String(255),"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FVN8%3D","line":36,"updated":"2013-09-06 07:21:10.000000000","message":"Instead of \u0027%s.id\u0027 % project.name just use \u0027project.id\u0027. Extra Index\u003dTrue.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"b5d5c06bd4c3b3fada261cc7f56f559b56d2dbb4","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        Column(\u0027event_id\u0027, String(255), primary_key\u003dTrue, index\u003dTrue),"},{"line_number":34,"context_line":"        Column(\u0027alarm_id\u0027, String(255), index\u003dTrue),"},{"line_number":35,"context_line":"        Column(\u0027on_behalf_of\u0027, String(255),"},{"line_number":36,"context_line":"               ForeignKey(\u0027%s.id\u0027 % project.name), index\u003dTrue),"},{"line_number":37,"context_line":"        Column(\u0027project_id\u0027, String(255),"},{"line_number":38,"context_line":"               ForeignKey(\u0027%s.id\u0027 % project.name), index\u003dTrue),"},{"line_number":39,"context_line":"        Column(\u0027user_id\u0027, String(255),"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FUtk%3D","line":36,"in_reply_to":"AAAASn%2F%2FVN8%3D","updated":"2013-09-06 11:05:01.000000000","message":"Fair enough, that was just my ham-fisted way of avoiding a pep8 warning .. F841 local variable \u0027project\u0027 is assigned to but never used. But I can simply suppress that with #noqa.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"2399a5404d9e3b3323e97785eb78e801495b463a","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        Column(\u0027on_behalf_of\u0027, String(255),"},{"line_number":36,"context_line":"               ForeignKey(\u0027%s.id\u0027 % project.name), index\u003dTrue),"},{"line_number":37,"context_line":"        Column(\u0027project_id\u0027, String(255),"},{"line_number":38,"context_line":"               ForeignKey(\u0027%s.id\u0027 % project.name), index\u003dTrue),"},{"line_number":39,"context_line":"        Column(\u0027user_id\u0027, String(255),"},{"line_number":40,"context_line":"               ForeignKey(\u0027%s.id\u0027 % user.name), index\u003dTrue),"},{"line_number":41,"context_line":"        Column(\u0027type\u0027, String(20)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FVN4%3D","line":38,"updated":"2013-09-06 07:21:10.000000000","message":"The same.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"2399a5404d9e3b3323e97785eb78e801495b463a","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        Column(\u0027project_id\u0027, String(255),"},{"line_number":38,"context_line":"               ForeignKey(\u0027%s.id\u0027 % project.name), index\u003dTrue),"},{"line_number":39,"context_line":"        Column(\u0027user_id\u0027, String(255),"},{"line_number":40,"context_line":"               ForeignKey(\u0027%s.id\u0027 % user.name), index\u003dTrue),"},{"line_number":41,"context_line":"        Column(\u0027type\u0027, String(20)),"},{"line_number":42,"context_line":"        Column(\u0027detail\u0027, String(255)),"},{"line_number":43,"context_line":"        Column(\u0027timestamp\u0027, DateTime(timezone\u003dFalse)))"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FVNw%3D","line":40,"updated":"2013-09-06 07:21:10.000000000","message":"The same.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"7cbaf51981c27a35695b89eeaaf789c4a0db04b2","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    alarm_history.create()"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    if migrate_engine.name \u003d\u003d \u0027mysql\u0027 or migrate_engine.name \u003d\u003d \u0027postgresql\u0027:"},{"line_number":47,"context_line":"        indices \u003d [Index(\u0027ix_alarm_history_alarm_id\u0027,"},{"line_number":48,"context_line":"                         alarm_history.c.alarm_id),"},{"line_number":49,"context_line":"                   Index(\u0027ix_alarm_history_on_behalf_of\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAASn%2F%2FTIk%3D","line":46,"updated":"2013-09-06 19:25:51.000000000","message":"nit: migrate_engine.name in [\u0027mysql\u0027, \u0027postgresql\u0027] (for short)","commit_id":"c19ae17d2297ee38ea1c59fbfa4b50bca6adcd19"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"f533225e1a90c305de320c9a937dca25ccd758ac","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    alarm_history.create()"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    if migrate_engine.name \u003d\u003d \u0027mysql\u0027 or migrate_engine.name \u003d\u003d \u0027postgresql\u0027:"},{"line_number":47,"context_line":"        indices \u003d [Index(\u0027ix_alarm_history_alarm_id\u0027,"},{"line_number":48,"context_line":"                         alarm_history.c.alarm_id),"},{"line_number":49,"context_line":"                   Index(\u0027ix_alarm_history_on_behalf_of\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAASn%2F%2FTHI%3D","line":46,"in_reply_to":"AAAASn%2F%2FTIk%3D","updated":"2013-09-06 19:31:14.000000000","message":"Done","commit_id":"c19ae17d2297ee38ea1c59fbfa4b50bca6adcd19"}],"ceilometer/storage/sqlalchemy/models.py":[{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"2399a5404d9e3b3323e97785eb78e801495b463a","unresolved":false,"context_lines":[{"line_number":209,"context_line":"    __tablename__ \u003d \u0027alarm_history\u0027"},{"line_number":210,"context_line":"    __table_args__ \u003d ("},{"line_number":211,"context_line":"        Index(\u0027ix_alarm_history_alarm_id\u0027, \u0027alarm_id\u0027),"},{"line_number":212,"context_line":"        Index(\u0027ix_alarm_history_on_behalf_of\u0027, \u0027on_behalf_of\u0027),"},{"line_number":213,"context_line":"        Index(\u0027ix_alarm_history_project_id\u0027, \u0027project_id\u0027),"},{"line_number":214,"context_line":"        Index(\u0027ix_alarm_history_user_id\u0027, \u0027user_id\u0027),"},{"line_number":215,"context_line":"    )"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FVOY%3D","line":212,"updated":"2013-09-06 07:21:10.000000000","message":"Extra index (for ForeignKey index will be created automatically).","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"52d665737c56dd704a69fac5e1ee2ddee0a2749b","unresolved":false,"context_lines":[{"line_number":209,"context_line":"    __tablename__ \u003d \u0027alarm_history\u0027"},{"line_number":210,"context_line":"    __table_args__ \u003d ("},{"line_number":211,"context_line":"        Index(\u0027ix_alarm_history_alarm_id\u0027, \u0027alarm_id\u0027),"},{"line_number":212,"context_line":"        Index(\u0027ix_alarm_history_on_behalf_of\u0027, \u0027on_behalf_of\u0027),"},{"line_number":213,"context_line":"        Index(\u0027ix_alarm_history_project_id\u0027, \u0027project_id\u0027),"},{"line_number":214,"context_line":"        Index(\u0027ix_alarm_history_user_id\u0027, \u0027user_id\u0027),"},{"line_number":215,"context_line":"    )"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FUmA%3D","line":212,"in_reply_to":"AAAASn%2F%2FUtM%3D","updated":"2013-09-06 11:40:44.000000000","message":"Add please this index in migrations in this case.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"b5d5c06bd4c3b3fada261cc7f56f559b56d2dbb4","unresolved":false,"context_lines":[{"line_number":209,"context_line":"    __tablename__ \u003d \u0027alarm_history\u0027"},{"line_number":210,"context_line":"    __table_args__ \u003d ("},{"line_number":211,"context_line":"        Index(\u0027ix_alarm_history_alarm_id\u0027, \u0027alarm_id\u0027),"},{"line_number":212,"context_line":"        Index(\u0027ix_alarm_history_on_behalf_of\u0027, \u0027on_behalf_of\u0027),"},{"line_number":213,"context_line":"        Index(\u0027ix_alarm_history_project_id\u0027, \u0027project_id\u0027),"},{"line_number":214,"context_line":"        Index(\u0027ix_alarm_history_user_id\u0027, \u0027user_id\u0027),"},{"line_number":215,"context_line":"    )"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FUtM%3D","line":212,"in_reply_to":"AAAASn%2F%2FVOY%3D","updated":"2013-09-06 11:05:01.000000000","message":"Yes, but the naming of that index needs to explicitly controlled also, right?\n\nJust relying on the index being created automatically seems to result in it being named for the column in mysql (e.g. \u0027on_behalf_of\u0027 as opposed to \u0027ix_alarm_history_on_behalf_of\u0027).","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"2399a5404d9e3b3323e97785eb78e801495b463a","unresolved":false,"context_lines":[{"line_number":210,"context_line":"    __table_args__ \u003d ("},{"line_number":211,"context_line":"        Index(\u0027ix_alarm_history_alarm_id\u0027, \u0027alarm_id\u0027),"},{"line_number":212,"context_line":"        Index(\u0027ix_alarm_history_on_behalf_of\u0027, \u0027on_behalf_of\u0027),"},{"line_number":213,"context_line":"        Index(\u0027ix_alarm_history_project_id\u0027, \u0027project_id\u0027),"},{"line_number":214,"context_line":"        Index(\u0027ix_alarm_history_user_id\u0027, \u0027user_id\u0027),"},{"line_number":215,"context_line":"    )"},{"line_number":216,"context_line":"    event_id \u003d Column(String(255), primary_key\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FVOQ%3D","line":213,"updated":"2013-09-06 07:21:10.000000000","message":"The same.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"2399a5404d9e3b3323e97785eb78e801495b463a","unresolved":false,"context_lines":[{"line_number":211,"context_line":"        Index(\u0027ix_alarm_history_alarm_id\u0027, \u0027alarm_id\u0027),"},{"line_number":212,"context_line":"        Index(\u0027ix_alarm_history_on_behalf_of\u0027, \u0027on_behalf_of\u0027),"},{"line_number":213,"context_line":"        Index(\u0027ix_alarm_history_project_id\u0027, \u0027project_id\u0027),"},{"line_number":214,"context_line":"        Index(\u0027ix_alarm_history_user_id\u0027, \u0027user_id\u0027),"},{"line_number":215,"context_line":"    )"},{"line_number":216,"context_line":"    event_id \u003d Column(String(255), primary_key\u003dTrue)"},{"line_number":217,"context_line":"    alarm_id \u003d Column(String(255))"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAASn%2F%2FVOM%3D","line":214,"updated":"2013-09-06 07:21:10.000000000","message":"The same.","commit_id":"cd186da610e11ab8e6bcf75278ad1c84442f6288"}]}