feat : Long 타입 반환 메서드에 Query 추가 (#364)

* feat : Long 타입 반환 메서드에 Query 추가

* refactor : 쿼리문 개선

Author: crocusia

cs25-batch/src/main/java/com/example/cs25batch/aop/MailLogAspect.java

@@ -48,18 +48,19 @@ public Object logMailSend(ProceedingJoinPoint joinPoint) throws Throwable {
             caused = e.getMessage();
             throw new CustomMailException(MailExceptionCode.EMAIL_SEND_FAILED_ERROR);
         } finally {
-            MailLog log = MailLog.builder()
+            MailLog mailLog = MailLog.builder()
                 .subscription(subscription)
                 .quiz(quiz)
                 .sendDate(LocalDateTime.now())
                 .status(status)
                 .caused(caused)
                 .build();
 
-            mailLogRepository.save(log);
+            mailLogRepository.save(mailLog);
             mailLogRepository.flush();
 
             if (status == MailStatus.FAILED) {
+                log.info("메일 발송 실패 : subscriptionId - {}, cause - {}", subscription.getId(), caused);
                 Map<String, String> retryMessage = Map.of(
                     "email", subscription.getEmail(),
                     "subscriptionId", subscription.getId().toString(),

cs25-entity/src/main/java/com/example/cs25entity/domain/mail/repository/MailLogRepository.java

@@ -7,6 +7,8 @@
 import java.util.Optional;
 import java.util.Set;
 import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.data.jpa.repository.Query;
+import org.springframework.data.repository.query.Param;
 import org.springframework.stereotype.Repository;
 
 @Repository
@@ -23,5 +25,11 @@ default MailLog findByIdOrElseThrow(Long id) {
 
     void deleteAllByIdIn(Collection<Long> ids);
 
-    Set<Long> findDistinctQuiz_IdBySubscription_Id(Long subscriptionId);
+    @Query("""
+          select distinct ml.quiz.id
+          from MailLog ml
+          where ml.subscription.id = :subscriptionId
+            and ml.status = com.example.cs25entity.domain.mail.enums.MailStatus.SENT
+        """)
+    Set<Long> findDistinctQuiz_IdBySubscription_Id(@Param("subscriptionId") Long subscriptionId);
 }