Folio Analytics only partially renamed holdings_record_id to holdings_id.

I’m not sure what project is the best to report this in, so I went with mod-ldp.
Please direct me to the correct place if this is incorrect.

The items_holdings_instances derived table has removed holdings_record_id in favor of holdings_id as per:

• https://github.com/folio-org/folio-analytics/issues/693

• https://github.com/folio-org/folio-analytics/pull/700

I refactored some LDP related SQL scripts that I use where I used holdings_record_id to now instead be holdings_id and discovered that folio_reporting.item_ext still uses holdings_record_id rather than holdings_id.

see: https://github.com/folio-org/folio-analytics/blob/main/reports/item_ext.sql

This makes working with LDP / Folio Analytics a bit more confusing and error prone.

It would be really great if some future version would make the references to the holdings record id / holdings id consistent across the board.