[HCR-426] 함수 시그니처 변경#72
Conversation
github-actions
Bot
added
Customer Team
deploy:intelligence-server
release:patch
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a trace_id parameter to the run_recommendation_and_publish_to_kafka function to enhance traceability during background recommendation processing and Kafka publishing. The review feedback suggests extending this traceability by passing the trace_id into the get_recommendation function and simplifying the log formatting logic to more clearly represent null trace IDs.
| try: | ||
| resp = await get_recommendation(session=None, member_id=member_id) | ||
| await publish_recommendation_to_kafka(member_id, resp) | ||
| await publish_recommendation_to_kafka(member_id, resp, trace_id=trace_id) |
There was a problem hiding this comment.
trace_id가 도입되었으나, get_recommendation 호출 시에는 전달되지 않고 있습니다. 전체 프로세스의 추적성(traceability)을 위해 get_recommendation 함수와 그 내부의 RecommendationService에서도 trace_id를 받아 로그에 포함할 수 있도록 개선하는 것이 좋습니다. (참고: get_recommendation 함수의 시그니처 변경도 함께 필요합니다.)
| logger.error( | ||
| "recommendation: 백그라운드 추천/Kafka 실패 member_id=%s trace_id=%s: %s", | ||
| member_id, | ||
| (trace_id or "").strip(), |
There was a problem hiding this comment.
trace_id가 None인 경우 (trace_id or "").strip()은 빈 문자열("")을 반환하여 로그에 trace_id=:와 같이 출력됩니다. trace_id를 직접 전달하면 None인 경우 trace_id=None:으로 출력되어 더 명확하며, 불필요한 strip() 호출도 줄일 수 있습니다.
| (trace_id or "").strip(), | |
| trace_id, |
1 participant
📝작업 내용
👀변경 사항
🎫 Jira Ticket
#️⃣관련 이슈