Skip to content

JDBC 스팬 원격 서비스명을 piki-db 로 명시 - #930

Merged
m-a-king merged 2 commits into
devfrom
chore/929-jdbc-span-remote-name
Aug 12, 2026
Merged

JDBC 스팬 원격 서비스명을 piki-db 로 명시#930
m-a-king merged 2 commits into
devfrom
chore/929-jdbc-span-remote-name

Conversation

@m-a-king

@m-a-king m-a-king commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Situation

  • 파싱 실패 알림(파싱 FAILED 종결 로그 전수화 + 실패 span error 마킹 #902 트랙)으로 트레이스를 실제로 읽기 시작하자, waterfall 의 JDBC 스팬이 원격 서비스 piki 로 표시돼 pikiquery 처럼 붙어 보이는 혼동이 드러났다. 서비스 식별자(piki-core·piki-extractor)와 한 글자 차이라 서비스로 오독하기 쉽다.
  • 원인: datasource-micrometer 는 JDBC client 스팬의 원격 서비스명으로 명시된 Hikari pool 이름을 1순위로 쓰고, 기본값("HikariPool-*")이면 DB catalog 이름으로 fallback 한다(2.2.1 DefaultDataSourceNameResolver 역어셈블로 확인). 우리 DB 이름이 piki 라 그게 그대로 노출된 것.

Task

  • JDBC 스팬의 원격 서비스명을 DB 임이 한눈에 드러나는 이름(piki-db)으로 명시한다.

Action

  • spring.datasource.hikari.pool-name: piki-db 한 줄. resolver 1순위가 명시된 pool 이름이라 이것만으로 표시가 바뀐다.
waterfall 표기 piki query · result-set piki-db query · result-set
HikariCP 메트릭 pool 라벨 HikariPool-1 piki-db

Result

  • 부작용 점검 완료: 대시보드 2개(app-overview·fleet-overview) 모두 pool 라벨을 하드코딩하지 않아 쿼리 영향 없음. 시계열은 라벨 값만 바뀌고 수는 불변이라 무료 티어 한도와도 무관.
  • 표시 개선일 뿐 판정 능력 변화는 없다 — JDBC 스팬은 여전히 duration 클 때만 주인공(풀 대기·락)이라는 읽기 기준 그대로.

연관 이슈

Summary by CodeRabbit

  • 개선 사항
    • 데이터베이스 연결 풀 설정을 추가했습니다.
    • 연결 풀 이름을 piki-db로 지정해 연결 관리 식별성을 높였습니다.

- 트레이스 waterfall 의 JDBC 스팬이 원격 서비스 "piki"(DB catalog 이름)로 표시돼 서비스 식별자 piki-core 와 혼동됐다
- datasource-micrometer 2.2.1 의 DefaultDataSourceNameResolver 를 역어셈블해 확인: 명시된 Hikari pool 이름이 1순위("HikariPool-*" 기본값은 무시), 없으면 catalog fallback — 그래서 pool-name 한 줄로 해결
- 부작용 점검: HikariCP 메트릭 pool 라벨이 HikariPool-1 → piki-db 로 바뀌지만 대시보드 두 개 모두 pool 라벨을 하드코딩하지 않아 영향 없음(시계열 수 불변)
@m-a-king m-a-king added the chore 그 외 (빌드·deps·ci·도구·설정·잡일) label Aug 12, 2026
@m-a-king m-a-king linked an issue Aug 12, 2026 that may be closed by this pull request
@m-a-king m-a-king self-assigned this Aug 12, 2026
@github-actions

Copy link
Copy Markdown

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.

@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro Plus

Run ID: c68477c6-e878-44ba-a484-450b2b7e5a78

📥 Commits

Reviewing files that changed from the base of the PR and between 421d85e and ec2df2d.

📒 Files selected for processing (1)
  • src/main/resources/application.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/main/resources/application.yml

Walkthrough

spring.datasource.hikari.pool-namepiki-db로 설정했다. JDBC 스팬의 원격 서비스명과 Hikari 메트릭의 풀 라벨이 piki-db를 사용한다.

Changes

JDBC 스팬 서비스명 설정

Layer / File(s) Summary
Hikari 풀 이름 설정
src/main/resources/application.yml
spring.datasource.hikari.pool-namepiki-db를 지정했다. JDBC 스팬과 Hikari 메트릭이 해당 풀 이름을 사용한다.

Estimated code review effort: 1 (Trivial) | ~2 minutes

연결 이슈 평가

Objective Addressed Explanation
[929] JDBC 스팬의 원격 서비스명을 piki-db로 지정
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/929-jdbc-span-remote-name

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.

@m-a-king
m-a-king merged commit 31bacb7 into dev Aug 12, 2026
10 checks passed
@m-a-king
m-a-king deleted the chore/929-jdbc-span-remote-name branch August 12, 2026 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 그 외 (빌드·deps·ci·도구·설정·잡일)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JDBC 스팬 원격 서비스명을 piki-db 로 명시

1 participant