Page Comparison

Summary

Adds the ability to configure custom (granular) deletion policies for posts for specific teams and/or channels. When the daily retention job runs it selectively deletes posts by age based on the configuration.

...

TBD: Does ChannelId need to be indexed?

RetentionPoliciesTeams table

TBD: Does TeamId need to be indexed?

Model

• Rename DataRetentionPolicy to GlobalDataRetentionPolicy

• Add RetentionPolicy representing a record in the RetentionPolicies table.

• Add RetentionPolicyChannel representing a record in the RetentionPoliciesChannels table.

• Add RetentionPolicyTeam representing a record in the RetentionPoliciesTeams table.

...

-- untested
SELECT
    Posts.Id
FROM
    Posts
    JOIN Channels ON Channels.Id = Posts.ChannelId
    JOIN Teams ON Teams.Id = Channels.TeamId
WHERE
    (
        Posts.CreateAt > :data_retention_channel1_timestamp
        AND ChannelId = :data_retion_channel1_channel_id
    )
    OR (
        Posts.CreateAt > :data_retention_channel2_timestamp
        AND ChannelId = :data_retion_channel2_channel_id
    )
    OR (
        Posts.CreateAt > :data_retention_team1_timestamp
        AND Teams.Id = :data_retion_team1_team_id
        AND ChannelId NOT IN (:data_retention_channels_channel_ids)
    )
    OR (
        Posts.CreateAt > :data_retention_team2_timestamp
        AND Teams.Id = :data_retion_team2_team_id
        AND ChannelId NOT IN (:data_retention_channels_channel_ids)
    )
    OR (
        Posts.CreateAt > :global_retention_timestamp
        AND ChannelId NOT IN (:data_retention_channels_channel_ids)
        AND Teams.Id NOT IN (:data_retention_teams_team_ids)
    );

The post ids returned from that query can be used to delete:

• Reactions

• Preferences (of Category ‘flagged_post’)

• Threads

• ThreadMemberships

• LinkMetadata TBD: these aren’t currently purged via data retention, but they probably should be.

FileInfo (and their associated files on disk) and ChannelMemberHistory continue to be deleted system-wide with no new granularity.

TBD: Why is ChannelMemberHistory currently deleted as part of message retention days settings?

Mobile

No changes. Existing file-deleted and post-deleted UX covers all.