Skip to content

[ai-assisted] fix(vector): points 조회 ORDER BY SQL 조립 수정#385

Merged
donghyuck merged 1 commit into2.xfrom
codex/issue-384-vector-points-orderby
Apr 30, 2026
Merged

[ai-assisted] fix(vector): points 조회 ORDER BY SQL 조립 수정#385
donghyuck merged 1 commit into2.xfrom
codex/issue-384-vector-points-orderby

Conversation

@donghyuck
Copy link
Copy Markdown
Owner

@donghyuck donghyuck commented Apr 30, 2026

Why

  • Vector Projection points 조회 SQL에서 Java text block의 줄 끝 공백 제거 때문에 ORDER BYp.display_order 형태의 PostgreSQL 문법 오류가 발생했습니다.
  • 이 오류로 GET /api/mgmt/ai/vectors/projections/{projectionId}/points가 500을 반환하고 Vector Map 산점도 렌더링이 진행되지 않았습니다.

What

  • JdbcVectorProjectionSql.orderByDisplayOrderClause(...)를 추가해 ORDER BY와 정렬 표현식 사이 공백을 일반 문자열에서 보장했습니다.
  • JdbcVectorProjectionPointRepository.findPage()가 새 ORDER BY clause helper를 사용하도록 수정했습니다.
  • ORDER BY clause 조립 회귀 테스트를 추가했습니다.

Related Issues

Validation

  • Command: ./gradlew :starter:studio-platform-starter-ai:test --tests '*JdbcVectorProjectionSqlTest'
  • Result: PASS
  • Command: ./gradlew :starter:studio-platform-starter-ai:test :starter:studio-platform-starter-ai-web:test && git diff --check
  • Result: PASS
  • Command: 로컬 PostgreSQL smoke query
  • Result: proj-20260430044829-105b3fa0 points query가 ORDER BY 포함 상태로 5건 조회됨
  • Command: scripts/run-dev.sh via detached screen session
  • Result: 서버 재시작 완료, Tomcat started on port 8080 확인
  • Command: curl -i 'http://localhost:8080/api/mgmt/ai/vectors/projections/proj-20260430044829-105b3fa0/points?limit=5&offset=0'
  • Result: 인증 없는 요청은 401 Unauthorized로 라우팅 확인

Risk / Rollback

  • Risk: points 조회 SQL의 ORDER BY 조립 방식만 변경되므로 영향 범위는 projection points 목록 정렬에 한정됩니다.
  • Rollback: 문제가 있으면 이 커밋을 revert하면 기존 SQL 조립 방식으로 돌아갑니다.

AI / Subagent Usage

  • AI-assisted: Yes
  • Subagent used: No
  • Delegated scope: N/A
  • Main author validation: 관련 단위 테스트, AI web 테스트, git diff --check, 로컬 PostgreSQL smoke query, 서버 재시작 및 라우팅 확인을 수행했습니다.

Checklist

  • commit message follows policy
  • issue template used or exception recorded
  • AI-Assisted value is correct
  • validation recorded
  • subagent usage recorded when used
  • CI / repository verification passed
  • human review completed before merge
  • no unrelated changes included

Issue:
- #384

Why:
- Java text block의 줄 끝 공백 제거로 points 조회 SQL이 ORDER BYp.display_order 형태로 조립되어 PostgreSQL 문법 오류가 발생했다.

What:
- ORDER BY 절을 일반 문자열 helper로 생성해 ORDER BY와 정렬 표현식 사이 공백을 보장했다.
- ORDER BY clause 조립 회귀 테스트를 추가했다.

Validation:
- ./gradlew :starter:studio-platform-starter-ai:test --tests '*JdbcVectorProjectionSqlTest' : PASS
- ./gradlew :starter:studio-platform-starter-ai:test :starter:studio-platform-starter-ai-web:test && git diff --check : PASS
- 로컬 PostgreSQL smoke: proj-20260430044829-105b3fa0 points query가 ORDER BY 포함 상태로 5건 조회됨
@donghyuck donghyuck merged commit 1e0473c into 2.x Apr 30, 2026
2 checks passed
@donghyuck donghyuck deleted the codex/issue-384-vector-points-orderby branch April 30, 2026 06:00
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