)]}'
{"cinder/db/sqlalchemy/models.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"e325b2200fe22c530a7d002a9b461764e4ba3d47","unresolved":false,"context_lines":[{"line_number":647,"context_line":"    \"\"\""},{"line_number":648,"context_line":""},{"line_number":649,"context_line":"    __tablename__ \u003d \u0027quota_classes\u0027"},{"line_number":650,"context_line":"    __table_args__ \u003d (Index(\u0027quota_classes_class_name_idx\u0027, \u0027class_name\u0027),"},{"line_number":651,"context_line":"                      CinderBase.__table_args__)"},{"line_number":652,"context_line":""},{"line_number":653,"context_line":"    id \u003d Column(Integer, primary_key\u003dTrue)"},{"line_number":654,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_a31a1f7a","line":651,"range":{"start_line":650,"start_character":2,"end_line":651,"end_character":48},"updated":"2017-09-26 01:54:51.000000000","message":"Is this a duplicated one? See line 655","commit_id":"1fb1a808cd01ecdfcf72228ce2a7f3e27f0d4127"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"c8d360924bf2d8ef2addd8806bbc9ded99969623","unresolved":false,"context_lines":[{"line_number":647,"context_line":"    \"\"\""},{"line_number":648,"context_line":""},{"line_number":649,"context_line":"    __tablename__ \u003d \u0027quota_classes\u0027"},{"line_number":650,"context_line":"    __table_args__ \u003d (Index(\u0027quota_classes_class_name_idx\u0027, \u0027class_name\u0027),"},{"line_number":651,"context_line":"                      CinderBase.__table_args__)"},{"line_number":652,"context_line":""},{"line_number":653,"context_line":"    id \u003d Column(Integer, primary_key\u003dTrue)"},{"line_number":654,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_92140ba2","line":651,"range":{"start_line":650,"start_character":2,"end_line":651,"end_character":48},"in_reply_to":"7f515b1d_92052bc5","updated":"2017-09-26 05:11:01.000000000","message":"OK, according to the answer here:\n\nhttps://stackoverflow.com/questions/6626810/multiple-columns-index-when-using-the-declarative-orm-extension-of-sqlalchemy\n\nit does appear you are correct that either way works so this is duplicating things. Looks like I should probably use the __table_args__ method of declaration for multi-column indexes, and just use the index\u003dTrue kwarg method for individual columns.","commit_id":"1fb1a808cd01ecdfcf72228ce2a7f3e27f0d4127"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"0778f15a2a5f1e5972148234f46694825634becf","unresolved":false,"context_lines":[{"line_number":647,"context_line":"    \"\"\""},{"line_number":648,"context_line":""},{"line_number":649,"context_line":"    __tablename__ \u003d \u0027quota_classes\u0027"},{"line_number":650,"context_line":"    __table_args__ \u003d (Index(\u0027quota_classes_class_name_idx\u0027, \u0027class_name\u0027),"},{"line_number":651,"context_line":"                      CinderBase.__table_args__)"},{"line_number":652,"context_line":""},{"line_number":653,"context_line":"    id \u003d Column(Integer, primary_key\u003dTrue)"},{"line_number":654,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_92052bc5","line":651,"range":{"start_line":650,"start_character":2,"end_line":651,"end_character":48},"in_reply_to":"7f515b1d_a31a1f7a","updated":"2017-09-26 04:49:12.000000000","message":"I don\u0027t think so, even though that had \"index\u003dTrue\". My understanding was this was still needed on __table_args__, but I will see if I can find more details to back that up.\n\nGood eye!","commit_id":"1fb1a808cd01ecdfcf72228ce2a7f3e27f0d4127"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"e325b2200fe22c530a7d002a9b461764e4ba3d47","unresolved":false,"context_lines":[{"line_number":662,"context_line":"    \"\"\"Represents the current usage for a given resource.\"\"\""},{"line_number":663,"context_line":""},{"line_number":664,"context_line":"    __tablename__ \u003d \u0027quota_usages\u0027"},{"line_number":665,"context_line":"    __table_args__ \u003d (Index(\u0027quota_usage_project_id_idx\u0027, \u0027project_id\u0027),"},{"line_number":666,"context_line":"                      Index(\u0027quota_usage_project_resource_idx\u0027, \u0027project_id\u0027,"},{"line_number":667,"context_line":"                            \u0027resource\u0027),"},{"line_number":668,"context_line":"                      CinderBase.__table_args__)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_6314274f","line":665,"range":{"start_line":665,"start_character":4,"end_line":665,"end_character":72},"updated":"2017-09-26 01:54:51.000000000","message":"ditto","commit_id":"1fb1a808cd01ecdfcf72228ce2a7f3e27f0d4127"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"e325b2200fe22c530a7d002a9b461764e4ba3d47","unresolved":false,"context_lines":[{"line_number":688,"context_line":"    __tablename__ \u003d \u0027reservations\u0027"},{"line_number":689,"context_line":"    __table_args__ \u003d (Index(\u0027reservations_allocated_id_idx\u0027, \u0027allocated_id\u0027),"},{"line_number":690,"context_line":"                      Index(\u0027reservations_usage_id_idx\u0027, \u0027usage_id\u0027),"},{"line_number":691,"context_line":"                      Index(\u0027reservations_project_id_idx\u0027, \u0027project_id\u0027),"},{"line_number":692,"context_line":"                      Index(\u0027reservations_deleted_expire_idx\u0027,"},{"line_number":693,"context_line":"                            \u0027deleted\u0027, \u0027expire\u0027),"},{"line_number":694,"context_line":"                      CinderBase.__table_args__)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_230e2f3b","line":691,"range":{"start_line":691,"start_character":16,"end_line":691,"end_character":73},"updated":"2017-09-26 01:54:51.000000000","message":"ditto.","commit_id":"1fb1a808cd01ecdfcf72228ce2a7f3e27f0d4127"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"e325b2200fe22c530a7d002a9b461764e4ba3d47","unresolved":false,"context_lines":[{"line_number":894,"context_line":"    __tablename__ \u003d \u0027driver_initiator_data\u0027"},{"line_number":895,"context_line":"    __table_args__ \u003d ("},{"line_number":896,"context_line":"        schema.UniqueConstraint(\"initiator\", \"namespace\", \"key\"),"},{"line_number":897,"context_line":"        Index(\u0027driver_initiator_data_initiator_idx\u0027, \u0027initiator\u0027),"},{"line_number":898,"context_line":"        CinderBase.__table_args__)"},{"line_number":899,"context_line":""},{"line_number":900,"context_line":"    id \u003d Column(Integer, primary_key\u003dTrue, nullable\u003dFalse)"},{"line_number":901,"context_line":"    initiator \u003d Column(String(255), index\u003dTrue, nullable\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_86029108","line":898,"range":{"start_line":897,"start_character":6,"end_line":898,"end_character":34},"updated":"2017-09-26 01:54:51.000000000","message":"ditto.","commit_id":"1fb1a808cd01ecdfcf72228ce2a7f3e27f0d4127"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"e325b2200fe22c530a7d002a9b461764e4ba3d47","unresolved":false,"context_lines":[{"line_number":931,"context_line":"class ImageVolumeCacheEntry(BASE, models.ModelBase):"},{"line_number":932,"context_line":"    \"\"\"Represents an image volume cache entry\"\"\""},{"line_number":933,"context_line":"    __tablename__ \u003d \u0027image_volume_cache_entries\u0027"},{"line_number":934,"context_line":"    __table_args__ \u003d (Index(\u0027image_volume_cache_entries_host_idx\u0027, \u0027host\u0027),"},{"line_number":935,"context_line":"                      Index(\u0027image_volume_cache_entries_image_id_idx\u0027,"},{"line_number":936,"context_line":"                            \u0027image_id\u0027),"},{"line_number":937,"context_line":"                      CinderBase.__table_args__)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_6609d52a","line":934,"range":{"start_line":934,"start_character":3,"end_line":934,"end_character":75},"updated":"2017-09-26 01:54:51.000000000","message":"ditto","commit_id":"1fb1a808cd01ecdfcf72228ce2a7f3e27f0d4127"}]}
