Skip to content
This repository was archived by the owner on Nov 13, 2024. It is now read-only.

extremely slow execution of sighting.get_pipeline_status() #942

Open
naknomum opened this issue Nov 30, 2023 · 0 comments
Open

extremely slow execution of sighting.get_pipeline_status() #942

naknomum opened this issue Nov 30, 2023 · 0 comments
Labels
bug Something isn't working elasticsearch

Comments

@naknomum
Copy link
Member

source unknown, but very likely related to needing to block while querying sage. maybe multiple times? on zebra, can take 1-2 minutes to get a result. this is especially bad during full Sighting elasticsearch re-indexing, as it hits this method 35000 times.

possibly solutions:

  • sighting.get_pipeline_state() (used by ES index) calls get_pipeline_status() - this is inefficient to get the whole complex status when the state can possibly short circuit early. some optimization possible here.
  • get_pipeline_status() might have some optimizations available in the methods it calls; in particular, some sort of sage timeout improvement or (better still) avoidance
  • probably the best solution is to store the state as an attribute on Sighting. (and try to only use get_pipeline_state() and avoid get_pipeline_status() e.g. in the api Sighting schema) this is a little more work, but computing on-the-fly is probably best avoided
@naknomum naknomum self-assigned this Nov 30, 2023
@TanyaStere42 TanyaStere42 added bug Something isn't working elasticsearch labels Nov 30, 2023
@TanyaStere42 TanyaStere42 added this to the Codex Release 2.2 milestone Nov 30, 2023
@TanyaStere42 TanyaStere42 moved this from Todo to In Progress in Codex 2.2.0 Dec 11, 2023
@TanyaStere42 TanyaStere42 removed this from the Codex Release 2.2 milestone Jan 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working elasticsearch
Projects
None yet
Development

No branches or pull requests

2 participants