Problems with 0864.MVF_CRA-upgrade.sql and reporter.classic_current_circ

Bug #1287967 reported by Ben Shum
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fix Released

Bug Description

Evergreen master

When attempting to run 0864.MVF_CRA-upgrade.sql (and presumably a problem in the upgrade SQL for 2.5-2.6 too), it will fail to run if your system makes use of the custom reporter.classic_current_circ view, which depends on metabib.rec_descriptor (which is one of the views dropped and recreated with 0864).

The "solution" is likely to consist of the following:

1) Put a DROP VIEW IF EXISTS reporter.classic_current_circ; before the DROP VIEW metabib.rec_descriptor; in the script. And then make a note somewhere about running the example reporter scripts to recreate this later.

2) Rewrite the reporter.classic_current_circ view to use a different way of getting that information.

Note discussion from IRC logs:

Patch and ideas forthcoming, but this should be resolved before we release 2.6-rc1

Tags: pullrequest
Revision history for this message
Ben Shum (bshum) wrote :

Only changing the 0864 file since it will fail to apply completely if this issue is encountered.

Fix here: user/bshum/lp1287967-fix-0864-upgrade;a=shortlog;h=refs/heads/user/bshum/lp1287967-fix-0864-upgrade

tags: added: pullrequest
Changed in evergreen:
milestone: none → 2.6.0-rc1
importance: Undecided → High
status: New → Confirmed
Bill Erickson (berick)
Changed in evergreen:
assignee: nobody → Bill Erickson (erickson-esilibrary)
status: Confirmed → In Progress
Revision history for this message
Bill Erickson (berick) wrote :

Confirmed the issue and the fix. In addition to my sign-off, I pushed a small change to the release notes for composite-attr-defs to mention the lost view there as well (belt + suspenders).;a=shortlog;h=refs/heads/user/berick/lp1287967-fix-0864-upgrade-continued

Changed in evergreen:
assignee: Bill Erickson (erickson-esilibrary) → nobody
status: In Progress → Confirmed
Revision history for this message
Ben Shum (bshum) wrote :

Oh good call Bill on the release note addition. Done and pushed to master, with thanks!

Changed in evergreen:
status: Confirmed → Fix Committed
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.