TransactionalListener을 통해서 연관관계의 객체를 save하려고 하는데 궁금한게 있습니다...

Question

public class AccountCreatedEvent extends ApplicationEvent { private final Long id; public AccountCreatedEvent(Object source, Long id) { super(source); this.id &#61; id; }AccountService &#64;Transactional public void create(NewAccountRequest newAccountRequest) { Account account &#61; NewAccountRequest.toEntity(newAccountRequest); account &#61; accountRepository.save(account); eventPublisher.publishEvent(new AccountCreatedEvent(this, account.getId())); }WalletService&#64;Transactional(propagation &#61; Propagation.REQUIRES_NEW) &#64;TransactionalEventListener(phase &#61; TransactionPhase.AFTER_COMMIT) public void handleAccountCreated(AccountCreatedEvent event) { Account account &#61; accountRepository.findById(event.getId()) .orElseThrow(() -> new IllegalArgumentException(&#34;해당 계정이 존재하지 않습니다.&#34;)); walletRepository.save(CashWallet.createWallet(account)); }현재 위와 같이 이벤트에는 id만 넘기고 서비스로직에서 다시 해당 Account를 가져와 wallet을 save하는 구조로 되어있습니다.근데 이러면 괜히 db조회가 하나 더 추가되니까 db부하만 늘어나는거같은데 고민입니다.그래서 생각한건 아예 이벤트에 Account의 객체를 넘겨주는건데 그러면 이벤트가 Account에 대해 의존하게되는데 마음에 안드네요;어떤 방법이 좋은걸까요ㅠㅠ

집킨 · Answer

이벤트는 특정 도메인 모델에 종속적일 수 밖에 없습니다이벤트에 accountId 가 들어가도 이 용어 자체가 어카운트 도메인 모델의 용어죠그럼 이벤트 핸들러쪽도 필연적으로 이벤트를 발행한 쪽의 도메인에 종속적이게 되는데, 보통 핸들러는 인프라스트럭처 영역에서 구현하고, 핸들러가 받은 이벤트의 데이터를 핸들러가 포함된 바운디드 컨텍스트의 도메인에 맞게 변환하는 방식으로 도메인 모델간 의존성을 제거합니다본문의 경우는 완벽하게 월렛에서 어카운트를 의존하는 거라서 월렛의 모델 설계가 잘못됐을 가능성이 높습니다월렛에선 어카운트 도메인의 객체를 쓰지말고 account id같은 식별자만 사용하도록 수정해보세요

charlatan · Answer

Account 과 Wallet 이 어떤 관계인지 모르겠지만, 아마도 OneToOne 일 듯 싶은데, 자식 데이터를 넣으면서 FK에 부모의 키를 넣을 때 부모 엔티티를 가져오는 것은 비효율적입니다. 그런 경우에 getReferenceById 를 쓰면 엔티티를 lazy 로드하므로 추가 select 문이 나가지 않습니다. findById 대신에 써보세요.

TransactionalListener을 통해서 연관관계의 객체를 save하려고 하는데 궁금한게 있습니다...

2개의 답변

Q&A 목록