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.

Sign up for GitHub

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

Jump to bottom

IBX-153: Added indices to workflow relation to optimize queries #76

Merged
merged 1 commit into from
Apr 14, 2021
Merged

IBX-153: Added indices to workflow relation to optimize queries #76

merged 1 commit into from
Apr 14, 2021

Conversation

adamwojs
Copy link
Member

@adamwojs adamwojs commented Apr 13, 2021 •
edited
Loading

JIRA: https://issues.ibexa.co/browse/IBX-153

Description

Added missing indexes to ezeditorialworkflow_workflows table. See https://github.com/ezsystems/ezplatform-workflow/pull/181

Upgrade

Upgrade script as PR target is 2.5

CREATE INDEX idx_workflow_co_id_ver ON ezeditorialworkflow_workflows(content_id, version_no);
CREATE INDEX idx_workflow_name ON ezeditorialworkflow_workflows(workflow_name);

@adamwojs adamwojs changed the base branch from master to 2.5 April 13, 2021 13:09
@adamwojs adamwojs self-assigned this Apr 13, 2021
@adamwojs adamwojs requested a review from a team April 13, 2021 13:09
@adamwojs adamwojs added the Doc needed The changes require some documentation label Apr 13, 2021
@ViniTou ViniTou requested a review from a team April 14, 2021 06:44
@alongosz alongosz changed the title IBX-153: Workflow queries optimization IBX-153: Added indices to workflow relation to optimize queries Apr 14, 2021
alongosz
alongosz requested changes Apr 14, 2021
View reviewed changes
Copy link
Member

@alongosz alongosz left a comment

Choose a reason for hiding this comment

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

The proper commit title could be IBX-153: Added indices to workflow relation to optimize queries. Changed PR title, but it's gonna be merged using commit title as it's the only one commit.

@tomaszszopinski tomaszszopinski self-assigned this Apr 14, 2021
tomaszszopinski
tomaszszopinski approved these changes Apr 14, 2021
View reviewed changes
Copy link

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

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

QA approved on eZPlatform 2.5-dev.

@alongosz alongosz merged commit dcf5095 into 2.5 Apr 14, 2021
@alongosz alongosz deleted the ibx_153 branch April 14, 2021 13:09
@lserwatka
Copy link
Member

Could you merge it up?

@adamwojs
Copy link
Member Author

Done.

@DominikaK DominikaK removed the Doc needed The changes require some documentation label May 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alongosz alongosz alongosz requested changes

@tomaszszopinski tomaszszopinski tomaszszopinski approved these changes

@ViniTou ViniTou ViniTou approved these changes

Assignees

@adamwojs adamwojs

@tomaszszopinski tomaszszopinski

Labels
QA approved
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@adamwojs @lserwatka @alongosz @ViniTou @tomaszszopinski @DominikaK