#561 [fix] Cache 동시성 이슈 해결

Mile-Writings · Oct 30, 2024 · 14672a3 · 14672a3
2 parents ca90f86 + b274155
commit 14672a3
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 3 deletions.
diff --git a/module-domain/src/main/java/com/mile/moim/domain/popular/MoimCuriousPost.java b/module-domain/src/main/java/com/mile/moim/domain/popular/MoimCuriousPost.java
@@ -7,19 +7,24 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 
+import java.util.Objects;
+
 @Embeddable
 @Getter
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 @AllArgsConstructor(access = AccessLevel.PRIVATE)
 public class MoimCuriousPost {
+    private Long postId;
     private String title;
     private String idUrl;
     private String contents;
     private String imgUrl;
     private String topic;
     private boolean isContainPhoto;
+
     public static MoimCuriousPost of(final Post post) {
         return new MoimCuriousPost(
+                post.getId(),
                 post.getTitle(),
                 post.getIdUrl(),
                 post.getContent(),
@@ -28,4 +33,16 @@ public static MoimCuriousPost of(final Post post) {
                 post.isContainPhoto()
         );
     }
+
+    @Override
+    public int hashCode() {
+        return postId.intValue();
+    }
+
+    @Override
+    public boolean equals(Object target) {
+        MoimCuriousPost targetPost = (MoimCuriousPost) target;
+
+        return Objects.equals(this.postId, targetPost.postId);
+    }
 }
diff --git a/module-domain/src/main/java/com/mile/moim/domain/popular/MoimCuriousWriter.java b/module-domain/src/main/java/com/mile/moim/domain/popular/MoimCuriousWriter.java
@@ -6,17 +6,34 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 
+import java.util.Objects;
+
 @Embeddable
 @Getter
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 public class MoimCuriousWriter {
+    Long writerNameId;
     String name;
 
-    private MoimCuriousWriter(final String name) {
+
+    private MoimCuriousWriter(final Long writerNameId, final String name) {
+        this.writerNameId = writerNameId;
         this.name = name;
     }
 
     public static MoimCuriousWriter of(final WriterName writerName) {
-        return new MoimCuriousWriter(writerName.getName());
+        return new MoimCuriousWriter(writerName.getId(), writerName.getName());
+    }
+
+    @Override
+    public int hashCode() {
+        return writerNameId.intValue();
+    }
+
+    @Override
+    public boolean equals(Object target) {
+        MoimCuriousWriter targetWriter = (MoimCuriousWriter) target;
+
+        return Objects.equals(this.name, targetWriter.name) && Objects.equals(this.writerNameId, targetWriter.writerNameId);
     }
 }
diff --git a/module-domain/src/main/java/com/mile/moim/service/popular/MoimPopularInfoService.java b/module-domain/src/main/java/com/mile/moim/service/popular/MoimPopularInfoService.java
@@ -17,7 +17,6 @@ public class MoimPopularInfoService {
 
 
     @Cacheable(value = "moimPopularInfo", key = "#moim.id")
-    @AtomicValidateMoimPopulerInfo
     public MoimPopularInfo getMoimPopularInfo(final Moim moim) {
         return moimPopularInfoRepository.findByMoimId(moim.getId()).orElseGet(
                 () -> moimPopularInfoRegister.setMostPopularInfoOfMoim(moim)