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

Import from other channels - Constantly getting "Failed to load search results" #3186

Closed
pcenov opened this issue Jun 2, 2021 · 11 comments · Fixed by #3399
Closed

Import from other channels - Constantly getting "Failed to load search results" #3186

pcenov opened this issue Jun 2, 2021 · 11 comments · Fixed by #3399

Comments

@pcenov
Copy link
Member

pcenov commented Jun 2, 2021

Observed behavior

I'm constantly getting a "Failed to load search results" regardless of the search term that I've entered. There's a 524 error in the console followed by other timeout errors.

Expected behavior

Fully operational Search functionality.

Steps to reproduce the issue

  1. Go to https://hotfixes.studio.learningequality.org/en/accounts/#/ and sign in.
  2. Click on a channel to open the channel editor.
  3. Click Add and select Import from channels.
  4. Enter a search term like 'math' and click the Search button.

Additional information

2021-06-02_18-26-59

Errors and logs

Failed to load resource: the server responded with a status of 524 ()

Usage Details

@radinamatic radinamatic added bug P0 - critical Priority: Release blocker or regression labels Jun 2, 2021
@radinamatic
Copy link
Member

Experiencing the same issue consistently, search for resources to import into channels does not work.

Importar de otros canales - PTBR-test - Kolibri Studio - Google Chrome_035

Note that searching the Content Library directly works without issues.

@pcenov
Copy link
Member Author

pcenov commented Aug 11, 2021

Retested today and this issue is still happening at both hotfixes and https://studio.learningequality.org

https://learningequality.testquality.com defect D7.

@bjester
Copy link
Member

bjester commented Mar 1, 2022

@vkWeb @d0sadata Another search related issue that's more actionable is #3181

@bjester
Copy link
Member

bjester commented Mar 1, 2022

Note the request is timing out due to the amount of data and our database performance in production and staging (hotfixes), so this isn't a "bug" per se but a need for optimization.

@vkWeb
Copy link
Member

vkWeb commented Mar 11, 2022

@bjester how do you suggest we should go about optimizing this?

@bjester
Copy link
Member

bjester commented Mar 11, 2022

@vkWeb Please take a look at implementing #3181 first. I'm offline next week but we can collaborate on this afterwards.

@rtibbles
Copy link
Member

One initial suggestion I had given for this was to create a separate search endpoint purely for the public channels - this might help reduce the query complexity by removing some of the permissions checking logic from these searches. However, I have not validated whether removing those aspects of the queries would resolve this issue.

@vkWeb
Copy link
Member

vkWeb commented Mar 13, 2022

@bjester okay, we'll look into this when you return.

@rtibbles we'll definitely try that as well to see if perf improves.

@vkWeb vkWeb assigned vkWeb and d0sadata and unassigned vkWeb and d0sadata Apr 29, 2022
@ozer550
Copy link
Member

ozer550 commented Apr 29, 2022

I would like to help @vkWeb on this.

@vkWeb
Copy link
Member

vkWeb commented Apr 29, 2022

I had a talk with @d0sadata about this, he's fine with me and @ozer550 handling this.

@ozer550
Copy link
Member

ozer550 commented May 6, 2022

I and @vkWeb tried to look into this but got lost on our way looking into SearchContentNodeViewSet. Can you provide some pointers on how should we proceed? @bjester @rtibbles

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

Successfully merging a pull request may close this issue.

7 participants