Don't specify character set for civicrm_relationship_cache table

[mattwire]

Overview

The upgrader in 5.29 specified utf8 for character set. It should not be specified as it should use the database default (eg. utf8mb4).

How did I find this? Keep finding civicrm_relationship_cache set to utf8 on client databases when everything else is utf8mb4.

Before

Hardcoded to utf8

After

Uses database default (eg. utf8mb4)

Technical Details

Comments

I don't think a new table was added to core for a long time and then this was added in 5.29. I'd like to fix the upgrader step even though it won't affect many (ie. they've probably already run the upgrade or will install new) because it leaves a bad example or how to create a new table in core.

[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).

[eileenmcnaughton]

@seamuslee001 this looks right to me - agree?

[seamuslee001]

Yes I think so