Investigate missing foreign keys on messagecategory table.

[squeaky-pl]

Currently the table looks like this:

CREATE TABLE `messagecategory` (
  `created_at` datetime NOT NULL DEFAULT CURRENT_TIMESTAMP,
  `id` bigint(20) NOT NULL AUTO_INCREMENT,
  `message_id` bigint(20) NOT NULL,
  `category_id` bigint(20) NOT NULL,
  PRIMARY KEY (`id`),
  KEY `ix_messagecategory_category_id` (`category_id`),
  KEY `ix_messagecategory_created_at` (`created_at`),
  KEY `message_category_ids` (`message_id`,`category_id`)
);

Notice that there are no foreign key constraints on message_id and category_id columns. This can lead to to data corruption and exceptions on Python side. We should investigate if they were omitted on purpose. If not we should add them.

[squeaky-pl]

It looks like they were dropped on purpose

sync-engine/migrations/versions/225_drop_messagecategory_foreign_keys.py

Line 20 in 9022236

text("ALTER TABLE messagecategory" " DROP FOREIGN KEY messagecategory_ibfk_1")

[squeaky-pl]

The commit that deletes them is this one: ff8b0bd