JDBC 스팬 원격 서비스명을 piki-db 로 명시 - #930
Merged
Merged
Conversation
- 트레이스 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 라벨을 하드코딩하지 않아 영향 없음(시계열 수 불변)
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Walkthrough
ChangesJDBC 스팬 서비스명 설정
Estimated code review effort: 1 (Trivial) | ~2 minutes 연결 이슈 평가
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Situation
piki로 표시돼pikiquery처럼 붙어 보이는 혼동이 드러났다. 서비스 식별자(piki-core·piki-extractor)와 한 글자 차이라 서비스로 오독하기 쉽다.DefaultDataSourceNameResolver역어셈블로 확인). 우리 DB 이름이piki라 그게 그대로 노출된 것.Task
piki-db)으로 명시한다.Action
spring.datasource.hikari.pool-name: piki-db한 줄. resolver 1순위가 명시된 pool 이름이라 이것만으로 표시가 바뀐다.pool라벨Result
pool라벨을 하드코딩하지 않아 쿼리 영향 없음. 시계열은 라벨 값만 바뀌고 수는 불변이라 무료 티어 한도와도 무관.연관 이슈
Summary by CodeRabbit
piki-db로 지정해 연결 관리 식별성을 높였습니다.