Poppy upgrade: Investigate using instance.complete_updated_date to replace instance/holdings/items join

[cbeer]

Possibly fixes #980

[hudajkhan]

Seems like: Poppy added field for timestamp on instance reflecting whether holdings/items have changed.
We need to assess if that is in fact true, and then see if we can use the Poppy info to save some indexing work.

May also require assessing impact on indexing memory issues.

[dnoneill]

See comment #1310

[dnoneill]

Tests completed

• changing properties on the instance
• changing properties on a holdings record
• changing properties on an item
• transferring an item between holdings
  with caveat. The record with the moved item does not update. Alissa has informed me this is a know issue with folio currently and record editors know to update the record with the moved item so that it does trigger.
• transferring a holdings record between instances
  Same caveat. The record with the moved holding doesn't update.
• changing marc records
• changing course reserves information (been tested, did not work correctly)

Based on the caveats this does not fix #980 but it does allow us to clean up some code.

[dnoneill]

Couple of things that came up with exploring the data:

table	Number of rows per table	How to access instance
sul_mod_inventory_storage.instance	10,284,710	n/a
sul_mod_inventory_storage.holdings_record	12,366,458	instanceid field
sul_mod_inventory_storage.items	11,429,559	holdingsrecordid field

One odd thing about the holdings_record is there doesn't seem to be any empty instanceid fields (assuming I queried correctly). Some holdings have same instanceids (see WHERE instanceid = '6b68aea8-2230-52a9-b008-ce8f21309f90')