Skip to content

[node/3251858] Add index for depth.#28

Open
simesy wants to merge 1 commit into
previousnext:mainfrom
simesy:patch-1
Open

[node/3251858] Add index for depth.#28
simesy wants to merge 1 commit into
previousnext:mainfrom
simesy:patch-1

Conversation

@simesy

@simesy simesy commented Nov 30, 2021

Copy link
Copy Markdown
Contributor

Add index for the subquery generated by the Top relationship.
https://www.drupal.org/project/entity_hierarchy/issues/3251858

Add index for the subquery generated by the Top relationship.
@larowlan

Copy link
Copy Markdown
Member

ping @kimpepper

@kimpepper kimpepper left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks ok to me. Are there any other columns that should be added to this index?

@simesy

simesy commented Dec 1, 2021

Copy link
Copy Markdown
Contributor Author

I'll get @nterbogt to comment as he did the sleuthing.

@nterbogt

nterbogt commented Dec 1, 2021

Copy link
Copy Markdown

I'd like to continue with a test for depth, left_pos, right_pos and see what comes out of the explain.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants