[sentry-junior]

The source map debugger shows if a project uploaded any file with a debug ID. It runs this check when no debug ID of the event matches an upload. The check queries sentry_debugidartifactbundle. This table has one row for each debug ID in each bundle. When the project has no debug IDs, the query reads all debug-ID rows of the organization.

A bundle gets its debug-ID rows only at its first upload. The rows are deleted only together with the bundle. So a boolean gives the same answer.

This PR adds the has_debug_ids column to ProjectArtifactBundle, not to ArtifactBundle. A later PR adds a partial index on project_id that contains only the links with the flag. With this index, the check is one index lookup for each project. A flag on ArtifactBundle cannot do this, because that table has no project column.

The column is NOT NULL with a database default of false. Postgres adds this column without a table rewrite, so this is a normal deployment migration:

ALTER TABLE "sentry_projectartifactbundle" ADD COLUMN "has_debug_ids" boolean DEFAULT false NOT NULL;

Nothing writes or reads the column yet.

Stack:

1. feat(sourcemaps): Add has_debug_ids flag to ProjectArtifactBundle #130421 (this PR): add the column.
2. feat(sourcemaps): Set has_debug_ids when linking an artifact bundle to a project #130422: set the flag on new links.
3. feat(sourcemaps): Backfill and index has_debug_ids on project artifact bundles #130424: backfill the flag and add the index. Both are post-deployment migrations.
4. ref(sourcemaps): Read has_debug_ids in the source map debugger #130425: read the flag in the source map debugger.

via @jan-auer.

--

View Session [Sentry]

[github-actions]

This PR has a migration; here is the generated SQL for src/sentry/migrations/1256_add_projectartifactbundle_has_debug_ids.py

for 1256_add_projectartifactbundle_has_debug_ids in sentry

--
-- Add field has_debug_ids to projectartifactbundle
--
ALTER TABLE "sentry_projectartifactbundle" ADD COLUMN "has_debug_ids" boolean DEFAULT false NOT NULL;

The stack changed: has_debug_ids moved from ArtifactBundle to ProjectArtifactBundle with a partial index on project_id, and the PRs are rebased on master. Please review again.