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

Performance issues after loading 50 articles #37

Open
mohanzeal opened this issue Mar 24, 2016 · 1 comment
Open

Performance issues after loading 50 articles #37

mohanzeal opened this issue Mar 24, 2016 · 1 comment

Comments

@mohanzeal
Copy link

Hi shameerc,

This application is good for small blogs i.e for about 20-30 articles blog. If we keep on writing articles and if exceeds more than 50 articles then the performance is getting degraded(issue 1). I think this is the issue with the loadArticles() function in Textpress class that loads all the articles in the articles directory in the function that's why the object of the Textpress class is occupying more memory and more time if there are more articles processing them.

One more thing is that cache is not working(issue 2) for the get parameter strings in the url. I have implemented pagination for the index page like localhost/textpress/?page=3 and for each page the cache is displaying only the first visit page.

Can you please fix this 2 issues.

And this application is very fast and loads with in 0.1 second if the articles are less than 10 in number. So if you can figure out a work around for optimizing Textpress object then this application will be the fastest application that can be used with out any problem.

@shameerc
Copy link
Owner

shameerc commented Jun 1, 2016

Hey @mohanzeal
We need to load and process all articles to get the categories and tags. But once a page is cached, IMO the performance won't be an issue.

The second one is a bug but which need to be fixed. It's probably because the cache key we are using is not considering the query string. I'm happy to accept a PR on this.

I'm planning for version 3 of TextPress, which should be much better than what we have now. But it's getting delayed due to other high priority stuff :(

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

No branches or pull requests

2 participants