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

Add missing index (update spy_queue.schema.xml) #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

berndalter-txb
Copy link

Add missing index to spy_queue_process.process_pid

Queries to find processes by process id in Spryker's queue process manager use condition SELECT/DELETE FROM spy_queue_process WHERE process_pid IN (...), but column process_pid doesn't have an index, so DB performance of those queries is worse than it needs to be, especially when the table size grows (which should not, but might happen).

->filterByProcessPid($processIds, Criteria::IN);

add missing index to spy_queue_process

- queries to find processes by process id in Spryker's queue process manager use condition 'WHERE process_pid IN (...)', but column process_pid doesn't have an index
@berndalter-txb berndalter-txb force-pushed the missing-index-on-process_pid branch from 9db0172 to af99eed Compare July 16, 2024 14:28
@berndalter-txb berndalter-txb changed the title Update spy_queue.schema.xml Add missing index (update spy_queue.schema.xml) Jul 17, 2024
@gechetspr
Copy link
Contributor

Dear contributor,

Your pull request will be reviewed by our team. In the meantime, please ensure that the PR complies with Spryker's coding guidelines and that your changes are covered by tests.

Thank you very much for your contribution to the development of Spryker and the growth of the Spryker community!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants