Dashboard

« Fusionner cette pull request pourrait dégrader la production. » Polylane relit chaque pull request face à la production réelle.

Rejoindre la liste d'attente
github.com/coreplane/orders-api/pull/482

Add trigram index for order search #482

Open rvidal wants to merge 1 commit into main from order-search-trgm
Conversation 1 Commits 1 Checks 2 Files changed 1
polylane bot commented 2 minutes ago ···
Caution

Merging this pull request may degrade production (high impact).

Merging this blocks every write to orders while the index builds. migrations/0114_order_search_trgm.sql:3 adds CREATE INDEX … USING gin (search_text gin_trgm_ops) without CONCURRENTLY, and a plain CREATE INDEX takes a full write lock on orders for the whole build. Checkout sustains ~38 writes/s on that table; each one queues behind the lock until the build finishes.

To make this safe: build the index with CREATE INDEX CONCURRENTLY outside the transactional migration.

orders-db · writes per second · last 48h projected lock window
02040
-48h -24h now
every one of these writes blocks while the index builds

Polylane analysed de91b47 for production impact.

Some checks were not successful 1 failing and 1 successful check
ci / test Successful in 2m 4s Details
Polylane production impact Non-concurrent index build locks writes on orders Details
Merging is blocked

Ton linter ne voit pas la production. Ce relecteur, si.

  • Relue face à ce qui tourne

    Le diff est vérifié contre les services sur lesquels il se déploie : leur config, leur trafic, leurs incidents récents.

  • Validé par défaut

    Un refus doit prouver exactement comment une ligne du diff casse la production. « Pourrait » ne compte pas.

  • Un seul commentaire

    Mis à jour sur place à chaque push, jamais d'empilement. Les changements de docs et de tests passent directement.

  • Des dents, si tu en veux

    Active la vérification bloquante et un refus retient la fusion jusqu'à ce qu'un push ultérieur le résolve.

L'incident le moins cher est celui qui n'est jamais fusionné. Polylane l'attrape en revue.

Rejoindre la liste d'attente