Skip to content

IBX-8125: Added *link* index to ezurlalias_ml #535

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

Merged
merged 1 commit into from
Jun 6, 2025

Conversation

reithor
Copy link
Contributor

@reithor reithor commented May 14, 2025

🎫 Issue IBX-8125

Related PRs:

Description:

LocationService::moveSubtree can result in such a query being executed:

SELECT parent, text_md5 FROM `ezurlalias_ml` WHERE
    (is_alias = ?) AND
    (is_original = ?) AND
    (action_type = ?)
    AND (link = ?)

Running:

EXPLAIN SELECT parent, text_md5 FROM `ezurlalias_ml` WHERE
    (is_alias = 0) AND
    (is_original = 0) AND
    (action_type = 'eznode')
    AND (link = 67);

shows that link column is not part of the index and number of checked rows is unnecessarily high.

@reithor reithor marked this pull request as ready for review May 15, 2025 07:07
@reithor reithor requested a review from a team May 15, 2025 07:07
@ezrobot ezrobot requested review from adamwojs, mikadamczyk, Steveb-p, alongosz, ciastektk, ViniTou, barw4, wiewiurdp, konradoboza and tbialcz and removed request for a team May 15, 2025 07:07
@adamwojs adamwojs force-pushed the ibx-8125_added_link_index_to_ezurlalias_ml branch from 7ffca90 to 039b835 Compare May 15, 2025 09:50
@tomaszszopinski tomaszszopinski force-pushed the ibx-8125_added_link_index_to_ezurlalias_ml branch from 039b835 to 4be3917 Compare June 4, 2025 13:46
Copy link

sonarqubecloud bot commented Jun 4, 2025

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 Ibexa DXP 4.6 commerce.

@Steveb-p Steveb-p merged commit 902584d into 4.6 Jun 6, 2025
24 checks passed
@Steveb-p Steveb-p deleted the ibx-8125_added_link_index_to_ezurlalias_ml branch June 6, 2025 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants