[obs-commits] [openSUSE/open-build-service] c73046: Create notified_projects join table
Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: c730462f265035b54ac07ed8a3c223e2a7c65262 https://github.com/openSUSE/open-build-service/commit/c730462f265035b54ac07e... Author: Eduardo Navarro <enavarro@suse.com> Date: 2020-04-24 (Fri, 24 Apr 2020) Changed paths: A src/api/db/migrate/20200421115317_create_join_table_notified_projects.rb M src/api/db/structure.sql Log Message: ----------- Create notified_projects join table Retrieving the projects of a notification, requires querying for comments, requests and reviews. We want to avoid n+1 queries instead of going through the polymorphic associations notifiable and notifiable.commentable for comments. We need to have an ID in the table since we'll be using a `has_many through:` association Co-authored-by: Dany Marcoux <dmarcoux@suse.com> Co-authored-by: David Kang <dkang@suse.com> Co-authored-by: Saray Cabrera Padrón <scabrerapadron@suse.de> Commit: e85d1203ab053fe605b8f24ed220de344810353c https://github.com/openSUSE/open-build-service/commit/e85d1203ab053fe605b8f2... Author: Dany Marcoux <dmarcoux@posteo.de> Date: 2020-04-24 (Fri, 24 Apr 2020) Changed paths: M src/api/app/models/notification.rb A src/api/app/models/notified_project.rb M src/api/app/models/project.rb Log Message: ----------- Add model and associations for join table notified_projects Commit: cef8f358b6fa12704ef765d71dbc96059a20ad12 https://github.com/openSUSE/open-build-service/commit/cef8f358b6fa12704ef765... Author: Eduardo Navarro <enavarro@suse.com> Date: 2020-04-24 (Fri, 24 Apr 2020) Changed paths: A src/api/app/services/notified_projects.rb Log Message: ----------- Create service to get the projects of a notification This service retrieves the associated projects of a notification. It will be used either by the notification creation from events, or from a data migration that will fill the notifications' projects after the notified_projects table was introduced. Co-authored-by: Saray Cabrera Padrón <scabrerapadron@suse.de> Commit: de3eba3b9d8e765364b50a95eef2396bf5a54107 https://github.com/openSUSE/open-build-service/commit/de3eba3b9d8e765364b50a... Author: Dany Marcoux <dmarcoux@posteo.de> Date: 2020-04-24 (Fri, 24 Apr 2020) Changed paths: A src/api/db/data/20200421121610_backfill_notified_projects.rb A src/api/spec/cassettes/BackfillNotifiedProjects/up/backfills_the_notifications_projects_table_with_all_projects_from_existing_notifications.yml A src/api/spec/db/data/backfill_notified_projects_spec.rb Log Message: ----------- Add migration to backfill notified_projects table Commit: 2392177bccc6908cb002fee54e998a446fce0830 https://github.com/openSUSE/open-build-service/commit/2392177bccc6908cb002fe... Author: Victor Pereira <vpereira@suse.de> Date: 2020-04-24 (Fri, 24 Apr 2020) Changed paths: M src/api/app/models/notification.rb A src/api/app/models/notified_project.rb M src/api/app/models/project.rb A src/api/app/services/notified_projects.rb A src/api/db/data/20200421121610_backfill_notified_projects.rb A src/api/db/migrate/20200421115317_create_join_table_notified_projects.rb M src/api/db/structure.sql A src/api/spec/cassettes/BackfillNotifiedProjects/up/backfills_the_notifications_projects_table_with_all_projects_from_existing_notifications.yml A src/api/spec/db/data/backfill_notified_projects_spec.rb Log Message: ----------- Merge pull request #9433 from dmarcoux/data-migration-notified-projects Create and backfill notified_projects table Compare: https://github.com/openSUSE/open-build-service/compare/f4bc345a71ad...239217...
participants (1)
-
Victor Pereira