Queue Ingest can skip reporter-oriented data

Bug #2066981 reported by Mike Rylander
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Critical
Unassigned

Bug Description

Evergreen version 3.11+

Queued Ingest restructures the ingest process in several ways, but does not currently fold in effects of the separate report-data trigger bbb_simple_rec_trigger on biblio.record_entry. We need it to do that since, while the trigger does fire on bib update, the processing of the bib has been delayed and there's no fresh data for the trigger to collect.

Branch forthcoming.

Revision history for this message
Mike Rylander (mrylander) wrote :

Branch available at: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/miker/lp-2066981-QI-vs-reporting-data

From the commit message:

    LP#2066981: Queued Ingest vs reporter data

    Queued Ingest needs to call the function behind the non-QI simple report
    data trigger. This commit updates the core QI function to make sure
    that happens.

    In order to address any outstanding records, either refresh the reporter
    data by calling the existing database function for that purpose:

      SELECT reporter.refresh_materialized_simple_record();

    or generate a script to directly process records that have been through
    QI, similar to the following psql script:

    \t
    \o /tmp/update_reporter_data.sql

    WITH last_update AS (
        SELECT record, max(ingest_time)
          FROM action.ingest_queue_entry
          WHERE fail_time IS NULL
                AND override_by IS NULL
                AND record_type = 'biblio'
          GROUP BY 1
    ), last_action AS (
        SELECT qi.record, qi.action
          FROM action.ingest_queue_entry qi,
                last_update
          WHERE qi.record = last_update.record
                AND qi.record_type = 'biblio'
                AND qi.ingest_time = last_update.max
    ) SELECT DISTINCT 'select reporter.simple_rec_update('||record||', $$delete$$=$$'||action||'$$)'
        FROM last_action
        ORDER BY 1;

    \t
    \o
    \i /tmp/update_reporter_data.sql

tags: added: pullrequest reports
Revision history for this message
Blake GH (bmagic) wrote :

Thanks Mike!

Merges clean. It does the thing. Tested that ingest_ctl still works.

I signed off and pushed to main and rel_3_13

Changed in evergreen:
status: Confirmed → Fix Committed
Changed in evergreen:
assignee: Mike Rylander (mrylander) → nobody
Andrea Neiman (aneiman)
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.