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

BUGFIX: Improve performance on hierarchy relations #5273

Draft
wants to merge 4 commits into
base: 9.0
Choose a base branch
from

Conversation

dlubitz
Copy link
Contributor

@dlubitz dlubitz commented Oct 2, 2024

Followup of #5268

It seems we found the culprit. The reason for this obsolete join is history. Originally the nodename was stored in the hierarchy relation table. And with #5018 as stated:

Node names are now stored in the contentgraph's node table

The node name is now in the node table.

There are actually more hidden places that can seemingly be optimised we found at least 4 places where a join was annotated being made for the legacy reason:

// we need to join with the hierarchy relation, because we need the node name.

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.

1 participant