esmero / ami

Archipelago Multi Importer. A module of mass ingest made for the masses
GNU Affero General Public License v3.0
2 stars 4 forks source link

Ingest order sorting algorithm does not traverse deep enough when a single CSV contains 4 levels and just UUIDs #201

Open DiegoPino opened 4 months ago

DiegoPino commented 4 months ago

What?

ON AMI ingest we presort the ADOs to ensure Top level/parents get ingested first. But we don't traverse deep enough when sorting (specially when using UUIDs for the semantic relationships) leading to the queue having children first. We also have a safety feature for that (a retry) but on very heavily nested structures, the number of retries is not enough

The task is to make the Sorting by parent algorithm faster/better and maybe just as failsafe add an extra retry

@alliomeria as observed with scrapbooks

DiegoPino commented 4 weeks ago

Still and issue .. .