Skip to content

check the import_status field before doing SQL operations to check the import url

Gregory Stark requested to merge optimize-projects-for-imported-projects into master

What does this MR do?

Before we do SQL to look up import urls first do the fast check of checking if the import state machine is "started"

Are there points in the code the reviewer needs to double check?

Do we actually need to check the import URL at all? All the other state checks just check the state machine and trust it. Why is this one special?

Why was this MR needed?

One-time imports, e.g. from the github importer, were doing extra SQL work on every project view to check if the import was in progress. That's pointless since they'll never be started again once they're finished. We could even clear all the import data but this is just as good.

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

#37714 (closed)

Edited by Yorick Peterse

Merge request reports