Skip to content

Remove code for zoekt_indexed_namespaces

Background

This work implements the schema design defined here. The plan is to replace Zoekt::IndexedNamespace class and zoekt_indexed_namespaces table with two new classes/tables Search::Zoekt::EnabledNamespace (zoekt_enabled_namespaces and Search::Zoekt::Index (zoekt_indices)

This work is broken up into multiple iterations

MR What it does
!138992 (merged) Add database migrations and model classes
!140252 (merged) Migrate data and use new models
!140906 (merged) Remove rails code
!142829 (merged) Drop tables migration

What does this MR do and why?

Related to #424121 (closed)

The MR

  • Removes all references to the code backed by the table zoekt_indexed_namespaces
  • Adds a post deployment migration to drop the table

Note: All of the data was moved to zoekt_indices and zoekt_enabled_namespaces. Additionally this feature behind a feature flag on GitLab.com and not released for self-managed which reduces the risk for data impacts for self-managed.

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

N/A

How to set up and validate locally

Removal only

Edited by Terri Chu

Merge request reports