-
Notifications
You must be signed in to change notification settings - Fork 7
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
Alert list timing out #319
Comments
That URL is working for me now. Is it still showing 504 for you? |
Sorry, that URL was for the list. It's the alert associated with that list that is failing... I can't get a URL directly to the alert itself to work sorry. |
Looking at logs, these errors are resulting:
Testing that URL manually, resulted in I'm guessing the spatial_object is still to blame (too complex) and resulting in SOLR timing out or running out of memory. UPDATE: I think the I think the intended result should use: UPDATE 2: Reminder: the spatial object should be tested independently before using in a https://biocache.ala.org.au/ws/occurrences/search?q=spatialObject:9433227 results in
FYI, we advise that you do not combine |
Thanks for investigating Nick! Adding some other relevant background here, spatialObject:9433227 was one of the original shapefiles that was too complex and needed optimising. There's more info on ticket #246 but essentially there is an optimised version of it that I created, spatialObject:9439588. So that should be the one used in alerts. Can be viewed and tested at: |
@kylie-m @nickdos I just updated that spatial object from spatialObject:9433227 to https://spatial.ala.org.au/?pid=9439588. However, the alert still failed. I note that the list actually runs of 3 shapefiles, so is one of the other two causing this issue too? Or is having multiple shapefiles itself causing the issue? |
Hi @turley85 - I saw this in the logs:
So I think you had the page loaded from earlier and then clicked the "Preview" or "Notify" but your login had expired. So try reloading the page and see if you're prompted to login again. And then try running it again. |
@nickdos Hmm, I just tried again. Closed the windows, logged out and then back into ALA and used "Preview" to test the alert again. It failed again sorry :( Let me know if there's something else I should have done to test! |
I just tested https://biocache.ala.org.au/ws/occurrences/search?q=spatialObject:9433219 as well, so that spatialObject should be ok. I didn't spot a third one on the list though? |
Same error again: Will look into it more. |
Seems the timeouts are causing the DB to error (as described in other ticket), so the DB lookup for the query ID subsequently fails. So fix is to remove all but one @turley85, we strongly recommend you take a copy of the list over to lists-test.ala.org.au and do the testing on alerts-test.ala.org.au., before making changes on production servers. |
@nickdos would a good additional workaround here be to combine the 2 spatial layers into one layer in QGIS first? No guarantees but I can give that a try, have done so for other work previously |
@kylie-m - I think so. Combining spatialObjects adds an extra level of complexity and depending on how they are combined, could be worse than a single object. So simpler/safer to stick with a single spatialObject, as recommended by Adam. |
Thanks Nick! @turley85 I have merged the 2 layers in QGIS, then uploaded to Spatial portal. In ala-test: is returning records within the new spatial object above, though the equivalent alert on test is not yet working - I'll keep trying, but @nickdos if you have any ideas, let me know! In production: Alert is working for this test list: https://lists.ala.org.au/speciesListItem/list/dr28737 Other Docs:
|
hmm I wonder if the spatialObject is too complex when in combination with a more complex query, but just ok with a simpler query, @nickdos ? |
@kylie-m I wondered the same thing - I think the additional terms for the OR'ed names might be pushing us over some threshold value. Only way to know is run the alert and look at logs, I think.
|
I'm getting a 504 Gateway Time-out the previewing the alert:
BioSecurity alert for NSW_NPWS_Western_Weeds_list
The text was updated successfully, but these errors were encountered: