Skip to content

docs: tell the Purple search case as the steps actually taken - #12

Merged
labyrinth30 merged 1 commit into
mainfrom
docs/search-case-process
Oct 5, 2026
Merged

labyrinth30 merged 1 commit into
mainfrom
docs/search-case-process

Conversation

@labyrinth30

Copy link
Copy Markdown
Owner

변경

  • Purple CASE 01 "내 기여와 해결" → "해결 과정" 4단계로 재구성 (PA-3408 PR #9025 기록 기준)
    1. EXPLAIN: 주문 테이블 전체 스캔 + DEPENDENT SUBQUERY 7개
    2. 인덱스 확인: 생성일·유형 인덱스 없음, 하지만 정렬 비용이 거의 없어(0.55초 vs 0.49초) 추가하지 않음
    3. 한 SQL 안 IN (SELECT … UNION …): 다시 DEPENDENT로 실행, 4.55초 그대로
    4. 쿼리 분리: 0.33초 + 0.56초 = 0.89초
  • "선택과 이유" 요약도 인덱스를 넣지 않은 판단을 포함하도록 수정

검증

  • build 에러 0, 두 PDF 생성, e2e 32/32 통과, 1440px 스크린샷 확인

🤖 Generated with Claude Code

EXPLAIN, index check, single-SQL attempt, then the split query, following
the PA-3408 PR notes.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 5, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 1 minute.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 54db6c95-9114-4115-be31-57e9552fa9dc
📥 Commits

Reviewing files that changed from the base of the PR and between f5dbdd3 and 64fbfc1.

📒 Files selected for processing (1)
  • src/content/cases/purple-search.mdx
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@labyrinth30
labyrinth30 merged commit 67776ec into main Oct 5, 2026
3 checks passed
@labyrinth30
labyrinth30 deleted the docs/search-case-process branch October 5, 2026 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant