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

[REQUEST] COUNT-query is executed twice in feed #123

Closed
bertterheide opened this issue Sep 13, 2021 · 5 comments
Closed

[REQUEST] COUNT-query is executed twice in feed #123

bertterheide opened this issue Sep 13, 2021 · 5 comments
Labels
Investigating We are working on this issue together with the customer.

Comments

@bertterheide
Copy link

Channable needs to know the product collection size on two occasions:

  • to validate and count the amount of available pages
  • to pass the total number of products in the feed summary

To do this, the COUNT-query is executed twice. This introduces a performance issue when using a large product set.

@jorgb90
Copy link

jorgb90 commented Jan 3, 2022

And this as well?

@bertterheide
Copy link
Author

Yes, I’m using this in production.

@jorgb90
Copy link

jorgb90 commented Jan 6, 2022

@Marvin-Magmodules We applied this pull request and can confirm it gives performance improvement and works correctly.

@evs-xsarus
Copy link
Contributor

@Marvin-Magmodules when will this be in a release?

@Frank-Magmodules Frank-Magmodules added the Investigating We are working on this issue together with the customer. label Feb 27, 2024
@Frank-Magmodules Frank-Magmodules changed the title COUNT-query is executed twice in feed [REQUEST] COUNT-query is executed twice in feed Feb 27, 2024
@Frank-Magmodules
Copy link
Member

Hi @jorgb90, @evs-xsarus and @bertterheide , Thanks a lot for your patience regarding this request. I'm happy to inform you that we've incorporated your request into our latest release. Your initiative in opening this issue and your patience are much appreciated. I'm closing this issue for now, but please feel free to reopen it or add comments whenever necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Investigating We are working on this issue together with the customer.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants