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

Improving search efficiency and speed #1368

Open
skenow opened this issue Jan 1, 2023 · 4 comments
Open

Improving search efficiency and speed #1368

skenow opened this issue Jan 1, 2023 · 4 comments
Milestone

Comments

@skenow
Copy link
Contributor

skenow commented Jan 1, 2023

As reported in the forums by MadFish - the current search strategy carries a lot of overhead, especially when there are a lot of results. He provided a proof of concept for reducing the query time significantly in this post - https://www.impresscms.org/modules/iforum/viewtopic.php?post_id=47912#forumpost47912

@skenow skenow added this to the 1.5.1 milestone Jan 1, 2023
@skenow
Copy link
Contributor Author

skenow commented Jan 2, 2023

Maybe this isn't so much a core thing as it is a module thing. Where should it go?

@MekDrop
Copy link
Member

MekDrop commented Jan 2, 2023

From my understanding - both. This place needs refactoring into something that modules gives parameters where to search and core uses them. Right now module asks all modules for search results and then merges. And that uses a lot of resources.

I don't see how we could make something good for 1.5.x. Only maybe for 2.0.x.

@fiammybe
Copy link
Member

fiammybe commented Jan 2, 2023

I agree that the focus for this should be on 2.x. 2.0 if it won't delay that version too much, otherwise a follow up release. First thing is now to get 2.0 stable on shared hosting.

@github-actions
Copy link

github-actions bot commented Mar 6, 2023

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or assign sombody or this will be closed in 5 days.

@github-actions github-actions bot added the stale label Mar 6, 2023
@MekDrop MekDrop removed the stale label Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants