-
Notifications
You must be signed in to change notification settings - Fork 0
feat: Zipkin 분산 추적 및 Order-User 서비스 간 호출 추적 구현 #11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Changes from all commits
Commits
Show all changes
7 commits
Select commit
Hold shift + click to select a range
24c75fb
feat: Zipkin 분산 추적 구현 및 테스트 완료
yesrin 3cbf57c
feat: Zipkin 분산 추적 및 Order-User 서비스 간 호출 추적 구현 (#10)
yesrin 19fd7a1
docs: Zipkin 분산 추적 가이드 문서 추가
yesrin 7fe9cf2
docs: Q3 B3 vs W3C Propagation 설명 개선
yesrin d58caf5
docs: README 업데이트 - 기술 스택 및 최근 개선사항 추가
yesrin a79c3a1
fix: TimeLimiter 설정 적용 및 @TimeLimiter 어노테이션 추가
yesrin fddfd13
refactor: OrderController 엔드포인트 분리 및 타입 안정성 개선
yesrin File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TimeLimiter 설정이 userClient에 적용되지 않습니다.
timeoutDuration만 정의되어 있어서는 Feign 호출에 시간 제한이 걸리지 않습니다.resilience4j.timelimiter.instances.userClient를 선언하고OrderService.getOrdersWithUserInfo등 실제 호출부에@TimeLimiter(name = "userClient")를 붙여야 3초 제한이 동작합니다. 그렇지 않으면 User Service가 응답 지연/정지 상태일 때 호출 스레드가 계속 대기해 회로 차단이나 fallback이 제때 작동하지 못합니다. 아래처럼 인스턴스를 추가해 주세요.timelimiter: configs: default: timeoutDuration: 3s + instances: + userClient: + baseConfig: default그리고 해당 메서드에
@TimeLimiter(name = "userClient")를 추가하는 것도 잊지 말아 주세요.🤖 Prompt for AI Agents