Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support batches processing for synchronization queue messages #1724

Closed
sivaschenko opened this issue Aug 5, 2020 · 2 comments · Fixed by magento/magento2#29543
Closed

Support batches processing for synchronization queue messages #1724

sivaschenko opened this issue Aug 5, 2020 · 2 comments · Fixed by magento/magento2#29543
Assignees
Labels
Priority: P3 May be fixed according to the position in the backlog Severity: S3 Affects non-critical data or functionality and does not force users to employ a workaround
Projects
Milestone

Comments

@sivaschenko
Copy link
Member

sivaschenko commented Aug 5, 2020

Media content bulk sync service

  • Introduce Magento\MediaContentSynchronizationApi\Api\SynchronizeIdentitiesInterface::execute(array $contentIdentities): void accepting the array of ContentIdentityInterface as a parameter
  • Intorduce the implementation of the interface Magento\MediaContentSynchronization\Model\SynchronizeIdentities
  • Reuse the service in existing implementations of Magento\MediaContentSynchronizationApi\Api\SynchronizeInterface (i.e. Magento\MediaContentSynchronizationCatalog\Model\Synchronizer\Category::synchronizeItem)

Media content synchronization consumer

  • Update the media content queue consumer Magento\MediaContentSynchronization\Model\Consume to be able to consume the message containing information about content identities and use SynchronizeIdentitiesInterface if such information is present in the message

Media gallery synchronization consumer

  • Update the media content queue consumer Magento\MediaGallerySynchronization\Model\Consume to be able to consume the message containing information about path's to synchronize and use SynchronizeFilesInterface if such information is present in the message
@m2-assistant
Copy link

m2-assistant bot commented Aug 5, 2020

Hi @sivaschenko. Thank you for your report.
To help us process this issue please make sure that you provided sufficient information.

Please, add a comment to assign the issue: @magento I am working on this


@m2-community-project m2-community-project bot added this to Ready for Grooming in Backlog Aug 5, 2020
@sivaschenko sivaschenko added Priority: P3 May be fixed according to the position in the backlog Severity: S3 Affects non-critical data or functionality and does not force users to employ a workaround labels Aug 5, 2020
@sivaschenko sivaschenko moved this from Ready for Grooming to Ready for Development in Backlog Aug 12, 2020
@sivaschenko sivaschenko added Priority: P4 Fixing can be deferred as a logical part of more important work Progress: ready for grooming and removed Priority: P3 May be fixed according to the position in the backlog Progress: ready for dev labels Aug 12, 2020
@m2-community-project m2-community-project bot moved this from Ready for Development to Ready for Grooming in Backlog Aug 12, 2020
@sivaschenko sivaschenko moved this from Ready for Grooming to Ready for Development in Backlog Aug 12, 2020
@yolouiese
Copy link
Contributor

@magento I am working on this

@m2-community-project m2-community-project bot moved this from Ready for Development to Dev In Progress in Backlog Aug 13, 2020
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 18, 2020
…synchronization queue messages - implemented gallery synchronization functionality
@sivaschenko sivaschenko added Priority: P3 May be fixed according to the position in the backlog and removed Priority: P4 Fixing can be deferred as a logical part of more important work labels Aug 19, 2020
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 20, 2020
…synchronization queue messages - implemented media content sychronization
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 20, 2020
…synchronization queue messages - added media content sychronizer and save contents to database
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 21, 2020
…synchronization queue messages - added method to sync and save media content to database
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 21, 2020
…synchronization queue messages - fixed failed static test
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 21, 2020
…synchronization queue messages - fixed failed static test
@sivaschenko sivaschenko added this to the 2.1.1 milestone Aug 25, 2020
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 26, 2020
…synchronization queue messages - fixed failed static test and added integration test
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 27, 2020
…synchronization queue messages - updated integration test
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 27, 2020
…synchronization queue messages - fixed failed integration test
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 27, 2020
…synchronization queue messages - updated pull request with requested changes
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 28, 2020
…synchronization queue messages - updated pull request with requested changes
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 31, 2020
…synchronization queue messages - added integration test for publisher
yolouiese added a commit to yolouiese/magento2 that referenced this issue Aug 31, 2020
…synchronization queue messages - updated publisher integration test file
yolouiese added a commit to yolouiese/magento2 that referenced this issue Sep 1, 2020
…synchronization queue messages - updated publisher integration test file
yolouiese added a commit to yolouiese/magento2 that referenced this issue Sep 2, 2020
…synchronization queue messages - updated PR with requested changes
yolouiese added a commit to yolouiese/magento2 that referenced this issue Sep 3, 2020
…synchronization queue messages - updated PR with requested changes and fixed failed integration test
yolouiese added a commit to yolouiese/magento2 that referenced this issue Sep 3, 2020
…synchronization queue messages - fixed failed integration test
Backlog automation moved this from Dev In Progress to Done Sep 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: P3 May be fixed according to the position in the backlog Severity: S3 Affects non-critical data or functionality and does not force users to employ a workaround
Projects
Backlog
  
Done
Development

Successfully merging a pull request may close this issue.

2 participants