HeuJung
2ae1972a3b
공개 이슈 gnuboard/g7 이 지목한 병목은 "같은 일을 반복 실행한다" 축이다. 응답·행의 무게를 다룬 · 과 겹치지 않는다. 공통 계약 — 총 건수 상한과 페이지 이동 범위는 별개 결정이다. 묶으면 필요 없이 기능이 깎인다. 총 건수만 상한을 받고(파생 테이블 COUNT), "다음" 이동은 per_page + 1 실측으로 끝까지 열어 둔다. 계산이 불가능해지는 것은 마지막 페이지 번호 하나뿐이며 그 사실은 last_page: null 이 알린다. 최신순처럼 실제 컬럼으로 정렬하는 목록은 커서로 전환해 깊이와 무관하게 일정 속도로 이동한다. 관련도순은 계산값 정렬이라 커서 키로 쓸 수 없어 offset 을 유지한다. 계약의 입구는 표준 paginate 와 같은 폭이어야 한다. 관계·쿼리 빌더를 받지 못하면 그 좁은 만큼이 그대로 운영 500 이 되고, 실제로 관리자 알림 목록에서 그렇게 터졌다. 응답 조립도 컬렉션마다 손으로 하면 형태가 늘어나는 순간 없는 값을 부르거나 새 필드를 흘리므로, 형태 판정을 paginationMeta 한 곳에 모았다. 표준 paginate 응답은 필드 단위로 이전과 동일하다. 요청당 반복 비용 — 훅 구독마다 남기던 로그 400줄, 요청당 스무 번 넘던 설정 파일 재읽기, 이미 캐시된 목록의 DB 재조회를 없앴다. 권한 판정은 요청 스코프 메모를 두어 화면 요소마다 나가던 조회를 한 번으로 줄였고, 크로스 요청 캐시는 두지 않아 권한 변경이 종전처럼 다음 요청에 반영된다. 검색 질의는 활성 엔진이 만든다. 저장소가 구체 엔진을 지목하면 플러그인이 등록한 엔진은 호출될 기회 자체를 잃고 오류 없이 다른 방식으로 동작한다. 해석기를 두어 활성 엔진에 위임하고, 전문검색이 없는 DBMS 의 부분일치 폴백도 드라이버명 하드코딩 대신 선언형 config 로 옮겼다. 총 건수가 잘린 목록에서 순번을 역산하면 0 과 음수가 나온다. 지어내지 않고 null 을 돌려주며, 그 원칙을 last_page 와 동일하게 적용했다.
82 lines
2.8 KiB
PHP
82 lines
2.8 KiB
PHP
<?php
|
|
|
|
namespace Tests\Concerns;
|
|
|
|
use Illuminate\Database\Events\QueryExecuted;
|
|
use Illuminate\Support\Facades\DB;
|
|
|
|
/**
|
|
* 실행된 쿼리 수를 세는 테스트 트레이트
|
|
*
|
|
* N+1 은 "행 수가 늘면 쿼리도 는다" 는 형태로 나타난다. 특정 시점의 쿼리 수를 숫자로
|
|
* 박아 두면 관계 하나만 추가해도 테스트가 깨져 유지되지 않고, 반대로 상한만 두면
|
|
* 행 수에 비례해 늘어나는 진짜 회귀를 놓친다.
|
|
*
|
|
* 그래서 이 트레이트는 **같은 화면을 행 수만 바꿔 두 번 재고 그 차이를 본다**.
|
|
* 행을 두 배로 늘려도 쿼리 수가 그대로면 조회 구조가 행 수와 무관하다는 뜻이다.
|
|
*/
|
|
trait CountsQueries
|
|
{
|
|
/**
|
|
* 클로저를 실행하며 발생한 쿼리를 모두 기록합니다.
|
|
*
|
|
* @param \Closure $callback 측정 대상
|
|
* @return array<int, string> 실행된 SQL 목록
|
|
*/
|
|
protected function captureQueries(\Closure $callback): array
|
|
{
|
|
$queries = [];
|
|
|
|
DB::listen(function (QueryExecuted $query) use (&$queries) {
|
|
$queries[] = $query->sql;
|
|
});
|
|
|
|
try {
|
|
$callback();
|
|
} finally {
|
|
// 리스너는 커넥션에 누적되므로, 다음 측정이 이전 기록을 물려받지 않도록 끊는다.
|
|
DB::getEventDispatcher()?->forget(QueryExecuted::class);
|
|
}
|
|
|
|
return $queries;
|
|
}
|
|
|
|
/**
|
|
* 클로저를 실행하며 발생한 쿼리 수를 셉니다.
|
|
*
|
|
* @param \Closure $callback 측정 대상
|
|
* @return int 실행된 쿼리 수
|
|
*/
|
|
protected function countQueries(\Closure $callback): int
|
|
{
|
|
return count($this->captureQueries($callback));
|
|
}
|
|
|
|
/**
|
|
* 데이터 규모를 바꿔도 쿼리 수가 늘지 않는지 단언합니다.
|
|
*
|
|
* 행을 늘리는 일(`$grow`)은 측정 밖에서 수행하고, 측정은 조회(`$measure`)만 감쌉니다.
|
|
*
|
|
* @param \Closure $measure 측정할 조회 (쿼리 수를 재는 대상)
|
|
* @param \Closure $grow 데이터를 늘리는 작업 (측정에 포함되지 않음)
|
|
* @param string $context 실패 메시지에 붙일 설명
|
|
*/
|
|
protected function assertQueryCountStableAsDataGrows(\Closure $measure, \Closure $grow, string $context = ''): void
|
|
{
|
|
$before = $this->countQueries($measure);
|
|
|
|
$grow();
|
|
|
|
$after = $this->countQueries($measure);
|
|
|
|
$label = $context !== '' ? $context.': ' : '';
|
|
|
|
$this->assertLessThanOrEqual(
|
|
$before,
|
|
$after,
|
|
$label."행 수를 늘렸더니 쿼리 수가 {$before} → {$after} 로 늘었다. "
|
|
.'조회가 행마다 쿼리를 내고 있다(N+1) — 관계를 eager load 하거나 일괄 조회로 바꾼다'
|
|
);
|
|
}
|
|
}
|