Skip to content

Clean up attention request related todos

Sincheol (David) Kim requested to merge 366442-remove-mr_attention_requests-todos into master

What does this MR do and why?

This MR deletes Todo entries for attention request feature as we are no longer releasing that feature. We're removing these todos first before we remove attention request related backend changes to avoid possible issues since the feature has already been disabled.

Related to #366442 (closed)

Database

Up

main: == 20220803042352 CleanupMrAttentionRequestTodos: migrating ===================
main: -- execute("DELETE from todos WHERE action = 10")
main:    -> 0.0027s
main: == 20220803042352 CleanupMrAttentionRequestTodos: migrated (0.0029s) ==========

Down

main: == 20220803042352 CleanupMrAttentionRequestTodos: reverting ===================
main: == 20220803042352 CleanupMrAttentionRequestTodos: reverted (0.0017s) ==========

Query plan

https://console.postgres.ai/gitlab/gitlab-production-tunnel-pg12/sessions/11311/commands/40529

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #366442 (closed)

Edited by Sincheol (David) Kim

Merge request reports