GroupContactCache: Drop temp table *after* releasing lock

[mattwire]

Overview

Currently we drop the temporary table before updating the cache time and before releasing the locks. This cleanup is not critical and should be outside the locked section for performance.

Before

Temporary table dropped within lock before updating cache time.

After

Temporary table dropped after lock is released.

Technical Details

Comments

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

this seems fine to me @eileenmcnaughton ?

[eileenmcnaughton]

Yeah - it makes sense