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

[14.0][IMP] product_import: reorganize for inheritance to import by batch #1076

Open
wants to merge 3 commits into
base: 14.0
Choose a base branch
from

Conversation

florentx
Copy link

@florentx florentx commented Nov 29, 2024

Some catalogs contain 10'000 references and import does not complete due to other tasks running at the same time, and locking table.

Splitting product import into individual jobs should mitigate this issue.

[ EDIT ]
This PR reorganizes code in order to easily be able to split import by batch with method split() of queue_job add-on, or any similar machinery.

@florentx florentx force-pushed the product_import_job branch 6 times, most recently from 96ad6db to 4f485a1 Compare November 29, 2024 11:07
@florentx florentx requested a review from simahawk December 3, 2024 16:53
@florentx florentx force-pushed the product_import_job branch 2 times, most recently from 9bed83c to 967c912 Compare December 4, 2024 09:11
Copy link

@twalter-c2c twalter-c2c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@florentx florentx force-pushed the product_import_job branch 3 times, most recently from c246786 to 1df34b7 Compare December 9, 2024 08:23
@florentx florentx force-pushed the product_import_job branch 4 times, most recently from 16f6447 to ca36212 Compare January 28, 2025 08:10
@florentx florentx changed the title [14.0][IMP] product_import: Import products asynchronously [14.0][IMP] product_import: Reorganize for inheritance to import by batch Jan 28, 2025
@florentx florentx changed the title [14.0][IMP] product_import: Reorganize for inheritance to import by batch [14.0][IMP] product_import: reorganize for inheritance to import by batch Jan 28, 2025
@florentx florentx requested a review from simahawk January 28, 2025 08:23
@florentx florentx requested a review from twalter-c2c January 28, 2025 10:00
Copy link

@twalter-c2c twalter-c2c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

It is needed in order to have a safe recomputation of fields of other models.
There was an issue in a third-party add-on which has a field recomputed
when a new 'product.supplierinfo' is added to 'seller_ids' of the product.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants