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
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