reviewGroupId가 예외 로그에만 사용되고 있네요. 그럼에도 이 함수의 인자로 전달해준 이유는 "특정 리뷰 그룹의 템플릿 아이디에 대한 템플릿이 없다"는 구체적인 예외 발생 맥락을 전달하기 위함이 맞나요?

(티키타카 오래걸릴까봐 미뤄 짐작하고 이어 답변하자면 ... )
제가 레포지토리에서 예외를 발생하는 것을 맡았는데, 그럼 이 경우에는

• template 레포지토리에서 특정 값 조회했을 때 없었다는 것
• reviewGroupId 에 해당하는 template 이었다는 것

을 담기 위해서 아래와 같이 try-catch로 구현하려는데 이런 의도가 맞나요?

try {
    Template template = templateRepository.findById(templateId);
     // ... 블라블라
} catch (NotFoundException ex) {
    throw new TemplateNotFoundByReviewGroupException(reviewGroupId, templateId);
}

1. Repository로 예외를 옮김으로써 서비스에서 반복적인 코드를 줄일 수 있다.
2. 이때, 서비스단에서 남기고 있던 추가 맥락을 로깅할 수 없게 된다 (트레이드 오프)
3. 해당 부분은 AOP 로깅 과정에서 파라미터를 로깅하는 방식으로 추적할 수 있다.

디코 대화도 남깁니다~

하지만 레포로 넘기면 엔티티단에서의 예외 처리, 서비스단에서의 코드 중복이 없는 대신 맥락이 조금 모자라지겠네요
해당 맥락은 AOP로 충당하는 방식으로 갈 건지~ 아니면 이대로 서비스에 둘 건지~가 되겠군요
아마 전자가 조금 더 합리적일 것이라는 생각입니당
AOP 로깅할 때 예외 발생했을 때에만 스택트레이스의 파라미터를 로깅할 수 있을지 확인해봐야겠어요

결론: try-catch를 사용하지 말고, 예외를 최소화한 뒤 맥락은 AOP를 통해 제공하는 것을 고려해 보자