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

Resources dashboard loading remarkably slow #174

Open
smanousopoulos opened this issue Oct 13, 2015 · 2 comments
Open

Resources dashboard loading remarkably slow #174

smanousopoulos opened this issue Oct 13, 2015 · 2 comments

Comments

@smanousopoulos
Copy link
Contributor

The loading of a user's resources in the dashboard (here) calls CKAN's user_show API action which in turn calls package_show to fetch all user packages (here).

As has been proposed and implemented here package_search could be used instead of package_show to allow passing limit and offset parameters to the query.

Maybe we should implement this logic in our CKAN version as pagination is meaningless as is.

@drmalex07
Copy link
Member

I agree.

The only thing we have to ensure is that the same authorization checks are performed (e.g. a proper NotAuthorized exception will be raised) in either cases (show/search).

Also, we should adapt our templates, since search results have a completely different structure than ordinary package_show results.

Another approach (but only in case the solution proposed by @smanousopoulos does not yield significant improvements), would be to bypass user_show and write our own lightweight version. This is because user_show gathers several pieces of information (such as followers and activity history) which are irrelevant to a dashboard for resources.

@kalxas
Copy link
Member

kalxas commented Oct 13, 2015

+1

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