แดชบอร์ด

"การ merge pull request นี้อาจทำให้ production เสื่อมประสิทธิภาพ" Polylane รีวิวทุก pull request เทียบกับ production จริง

ลงชื่อเข้า waitlist
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

linter ของคุณมองไม่เห็น production ผู้รีวิวคนนี้เห็น

  • รีวิวเทียบกับสิ่งที่รันอยู่จริง

    diff ถูกตรวจเทียบกับ service ที่มันจะ deploy ไป: การตั้งค่า ทราฟฟิก และ incident ล่าสุดของ service เหล่านั้น

  • ผ่านโดยค่าเริ่มต้น

    การไม่ผ่านต้องพิสูจน์ให้ชัดว่าบรรทัดใดใน diff ทำให้ production พังอย่างไร "อาจจะ" ไม่นับ

  • คอมเมนต์เดียว

    อัปเดตในที่เดิมทุกครั้งที่ push ไม่มีการถมทับ การเปลี่ยนเอกสารและเทสต์ผ่านไปตรง ๆ

  • มีเขี้ยว ถ้าคุณต้องการ

    เปิด check แบบบล็อก แล้วการไม่ผ่านจะกั้น merge ไว้จนกว่า push ครั้งถัดไปจะแก้มันได้

incident ที่ถูกที่สุดคือ incident ที่ไม่มีวันถูก merge Polylane จับมันได้ตอนรีวิว

ลงชื่อเข้า waitlist