Comment 7 for bug 1599634

Revision history for this message
Galen Charlton (gmc) wrote :

Looks very promising. One suggestion I have is to change the id column in the view to just be the ID from the source tables (promoted to BIGINT in some cases). This would gain us two things:

- it removes a redundancy, as circ_type already specifies the source table
- it would make it easier to join this view against the circulation tables when writing SQL queries