Skip to content

misc: oss changes - new composite primary key instead of single - #7054

Merged
prakash100198 merged 1 commit into
mainfrom
external-ci-artifact-read-fix-oss
Oct 6, 2026
Merged

prakash100198 merged 1 commit into
mainfrom
external-ci-artifact-read-fix-oss

Conversation

@prakash100198

@prakash100198 prakash100198 commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Description

Fixes #

Checklist:

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR requires documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have tested it for all user roles.
  • I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?


Summary by Bito

  • Added a new composite index `idx_ci_artifact_external_ci_pipeline_id_id` on the `ci_artifact` table to optimize queries filtering by `external_ci_pipeline_id` and ordering by `id`.
  • The index is created with a partial condition `WHERE external_ci_pipeline_id IS NOT NULL` to improve performance and reduce index size.
  • Included a corresponding down migration script to drop the index if needed.

@prakash100198
prakash100198 merged commit 721048a into main Oct 6, 2026
10 checks passed
@prakash100198
prakash100198 deleted the external-ci-artifact-read-fix-oss branch October 6, 2026 11:12
@sonarqubecloud

sonarqubecloud Bot commented Oct 6, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants