[BE] Refactor/#540 지연로딩의 경우, soft delete을 반영할 수 없는 문제 해결 #553

yoondgu · 2023-10-05T08:21:34Z

작업 대상

Repository 메서드에 IsDeletedFalse를 붙여 조회 기능에 soft delete을 반영함에 따라
지연 로딩으로 정보를 조회하는 경우, 삭제된 토픽/핀도 조회하게 되는 문제가 있었습니다.

이를 해결하기 위한 방법 중 엔티티에 @Where을 적용하는 것을 선택했습니다.
stream, filter는 지연로딩인 모든 연관 관계의 getter에 대해 이를 적용해주어야 한다는 문제가 있기 때문입니다.
빠른 적용이 필요한 상황에서도 오히려 시간이 더 오래 걸릴 수 있습니다.
또, 그에 비해 @Where 적용 시, 수정이 필요한 대부분은 관리자 기능 및 테스트 코드인 데에 비해
새로 엔티티가 추가될 때마다/연관관계가 변경될때마다 getter를 수정하는 작업을 하기에는 유지보수성도 떨어진다고 판단했습니다.

고려해야 할 연관관계 (테스트 작성)

topic.getPins
member.getCreatedPins
location.getPins
member.getCreatedTopics
pin.getPinImages
pin.getTopic (즉시로딩, topic이 삭제되었다면 pin도 삭제된 상황이므로 해당 X)
atlas.getTopic (즉시로딩, soft delete 해당 X)
bookmark.getTopic (즉시로딩, soft delete 해당 X)

작업 내용

📄 작업 내용

지연로딩 대상인 엔티티에 Where 적용, 테스트 작성
- Where 적용으로 인해 깨지는 테스트 수정
기존에 JPQL에 조건부를 붙인 조회 쿼리를 Where 적용으로 변경
- Where 적용으로 인해 깨지는 테스트 수정

추가 작업

Pin을 삭제할 경우 Topic의 pinCount 컬럼을 업데이트하는 로직이 없는 것을 발견하여 이를 추가했습니다. (해당 커밋)

그동안은 Pin을 삭제할 일이 없었기 때문에 놓쳤던 부분인 것 같아요.
그런데 우리는 Pin을 삭제할때 Repository를 거쳐 쿼리를 실행하므로, Pin 삭제 로직과 Topic의 pinCount를 완전히 통합하기가 어려웠습니다.
이 부분에 대해서는 TODO 주석을 다수 남겨두었는데, 같이 얘기해보면 좋을 것 같아요.

🙋🏻 주의 사항

일종의 버그 해결이기 때문에 Hotfix라고 생각하지만, 개발 서버에도 먼저 반영될 사안이라고 생각해 develop-BE로 PR 보냅니다.
개발서버 배포 후 바로 운영서버 배포하면 될 것 같습니다.

스크린샷

📎 관련 이슈

closed #540

레퍼런스

- 검토할 TODO 다수 존재

github-actions · 2023-10-05T08:24:29Z

Unit Test Results

  68 files   68 suites 24s ⏱️
302 tests 302 ✔️ 0 💤 0 ❌
311 runs 311 ✔️ 0 💤 0 ❌

Results for commit 9df7456.

♻️ This comment has been updated with latest results.

junpakPark

수고하셨습니다 도이!
질문하나 남기긴했는데 merge 되도 무방하다고 생각하여 approve 남깁니다 👍👍

junpakPark · 2023-10-05T08:24:39Z

backend/src/main/java/com/mapbefine/mapbefine/admin/application/AdminCommandService.java

        permissionRepository.deleteAllByMemberId(memberId);
        atlasRepository.deleteAllByMemberId(memberId);
        bookmarkRepository.deleteAllByMemberId(memberId);
+        entityManager.flush();


Spring Data JPA의 Repository 인터페이스에는 flush 메서드가 기본적으로 포함되어있어서
해당 메서드를 사용하는 경우, 내부적으로 EntityManager의 flush 메서드가 호출된다고 하네용!

entityManager를 주입받을 필요없이 repository에서 flush하는 건 어떨까요??

추가적인 주입이 필요없다는 면에서 굉장히 좋은 아이디어네요!

하지만 현재 코드는 Permission, Atlas, Bookmark 에 대한 hard delete 쿼리를 한꺼번에 날리는 flush 이기 때문에, flush 를 호출하는 의무를 어떠한 Repository 가 가져가야 하는지를 정하는 것도 굉장히 애매한 부분인 것 같기도 하네요.

다들 어떻게 생각하시나요?

사실 저도 entityManager가 존재한다는게 거슬리긴 해요.

준팍이 말씀해주신

entityManager를 주입받을 필요없이 repository에서 flush하는 건 어떨까요??

이 부분은 별도의 설정하지 않아도 된다는 의미일까요 ?

아마, 테스트 코드 구성을 잘못해서 발생한 문제를 해결하기 위해 위와같은 구조를 갖게 된 걸로 알고있어요. (내가 짬 ㅋ)
테스트 코드에서 //given에서 데이터를 세팅해주는 작업과 같은 트랜잭션으로 묶이기 때문에 flush를 강제로 호출해주지 않으면 수정 관련 쿼리가 쓰기 지연 저장소에 저장되었다가, 트랜잭션이 커밋되기 직전에 데이터베이스로 쿼리를 날라가게 되요.
어떻게 해결 할 수 있다는 의미인지 궁금합니다 !

맞아요! 다른 경우처럼 Repository 메서드를 호출하는 것만으로 flush를 처리하면 좋겠지만,
지금은 Transactional로 묶여있는 메서드라 서비스 메서드 호출이 끝날때 한번에 flush하지 않나요..?!

그래서 위 경우 Topic, Pin, PinImage의 soft delete 쿼리(update)를 날리기 전에
Permission, Atlas, Bookmark 에 대한 hard delete 쿼리를 먼저 날려주는 처리가 필요했어요.

하지만 저도 EntityManager에 대한 의존성을 없애는 게 좋다고 생각하는데,
그러려면 @Query를 쓰더라도 @Modifying으로 flush automatically 해주는 게 좋을까요?
아니면 더 좋은 방법을 아신다면 공유 부탁드립니다 !! 🥲

중간에 flush를 해줘야 하는 이유

72번 줄이 없을 경우, 예외는 발생하지 않았지만 hard delete 메서드 호출이 DB에 반영되지 않는 것을 확인할 수 있었어요.
(테스트, 로컬 POSTMAN 확인 모두)

제가 추측하는 플로우는 아래와 같습니다.
중간에 flush를 해주지 않고 쓰기 지연으로 한 번에 flush할 경우 아래와 같은 순서로 동작하는 것 아닐까요?

hard delete 메서드를 통해 해당 id를 가진 엔티티를 영속성 컨텍스트에서 제거한다.

soft delete 메서드를 통해 update를 호출하면서, 연관된 엔티티들이 모두 영속화된다.
-> (1)에서 delete 해도, 2에서 조회할 때 함께 불러와지는 permission, atlas, bookmark까지 다시 영속화된다.

그래서 한 번에 flush할 때 delete 쿼리가 나가지 않는다.

그래서 (1)과 (2) 사이에 직접 flush를 해야 한다고 생각했어요.
제가 아직 헷갈려서 정리하다보니 길어졌네요.. ㅎㅎ 그래도 참고차 남깁니다.

아니 제가 화면 새로고침을 안해서 쥬니 댓글 지금봤네용 하핫

@cpot5620 @yoondgu

아 저는 flush를 별도로 설정하지 말라는 의미는 아니었구요!
위에 보이는 메서드를 사용하자는 의미였습니다 ㅎㅎ
이 메서드를 사용하면 entityManager의 flush를 사용한것과 동일하게 작동한다고 하네요 ㅎㅎ

GPT 센세의 첨언...

세 레포지터리 중 어떤 것을 써도 동일하게 동작할텐데
매튜 말대로 뭐 써야될지 조금 고민되긴 하겠네요 ㅋㅋ

아 아까 트러블슈팅하다가 clear까지 해보려고 entityManager를 받아왔다가 이 생각을 못했네요 !!
근데 대신 delete 메서드에 JPQL 직접 쓰고 Modifying 으로 해결하는 건 어떠세요?
쥬니가 쓴 기술 블로그 참고

좋습니다 ㅎㅎ

kpeel5839

오우 도이!

where 절을 적용하면서 isDeletedFalse 가 떨어져버리니까 굉장히 메서드가 짧아졌군요!

고생하셨습니다!

낭낭하게 코멘트 남겼습니다~!

확인해주시고 대댓글을 달아주시기를 원하기 때문에 Request Changes 남기도록 하겠습니다 ^.^

확인하시면 바로 Approve 로 바꾸도록 할게요~!

kpeel5839 · 2023-10-05T13:02:23Z

backend/src/main/java/com/mapbefine/mapbefine/admin/application/AdminCommandService.java

+        Pin pin = pinRepository.findById(pinId)
+                .orElseThrow(() -> new PinNotFoundException(PIN_NOT_FOUND, pinId));
+
+        /// TODO: 2023/10/05 soft delete를 @Query로 구현하면서, pinCount 반영을 통합할 수 없음


update pin p, topic t set p.is_deleted = true, t.pin_count = t.pin_count - 1 where p.id = "삭제하려는 pin id" AND t.id = p.topic_id;

금단의 영역인 것 같긴하지만, 방금 되는 것을 확인하긴 했습니다...

그냥 하나의 방법이니 보고 피식 웃고 넘어가주세용~~ ㅋㅋㅋ

지금 방법대로 하면 안 되는건가요 ?
상관 없어보이는데..

사실 soft delete 의 대가인 것 같아요 ㅋㅋㅋ delete 해줘야 하는 부분에서 연관된 다른 부분들도 신경써야 하는거요

근데 @PrePost 썼던 것처럼 @PreRemove로 할 수 있을지도..?? 해보고 아니면 냅둘게요 ㅎㅎ
라고 생각했는데 JPQL로 삭제할 때는 적용 안되겠네요 ...
사실 지금도 상관없지만 유지보수 측면에서 좀 더 도메인 단에서 로직을 묶을 수 없을까 해서 고민해봤습니당

아니면 JPA 이벤트리스너로 해결하는 방법도 있을까 싶은데 이 PR 내용과는 거리가 멀기 때문에 별개로 시도해볼게요!

kpeel5839 · 2023-10-05T13:05:37Z

backend/src/main/java/com/mapbefine/mapbefine/admin/application/AdminCommandService.java

+        /// TODO: 2023/10/05 soft delete를 @Query로 구현하면서, pinCount 반영을 통합할 수 없음
+        pin.decrementTopicPinCount();
+        pinRepository.deleteById(pin.getId());
+        /// TODO: 2023/10/05  pinImage는?


pin image 도 연계적으로 지워줘야 하겠군요....

soft delete 가 참 산이 많네요

kpeel5839 · 2023-10-05T13:07:32Z

backend/src/main/java/com/mapbefine/mapbefine/auth/application/AuthService.java

@@ -31,6 +30,7 @@ public void validateMember(Long memberId) {
        }
    }

+    // TODO 테스트가 필요하긴 한데, MemberQueryService와 너무 중복되는 내용이다. 엔티티 테스트로 중복을 없앨 수 있으면 좋겠다.


오우 조금 더 자세하게 말씀해주실 수 있을까요

언젠가 MemberQueryService의 스펙이 바뀔수도 혹은 AuthService의 스펙이 바뀔 수도 있을 것 같은데요.
그런 의미에서, 중복되는 테스트라고 할지라도 수행하는 것이 좋지 않을까요 ?

맞아요 근데 AuthService 의존성 문제때문에 귀찮아서... + 서비스 단에서 테스트하다보니 발생하는 중복이 많은 것 같아서요. (이건 저희 테스트 코드 전반적인 문제인 것 같아요)

더 작은 단위에서 테스트하면 좋을텐데 엔티티 테스트 해보려니 TestEntityManager 잔뜩 써야 할 것 같아서 일단 안했습니다.
테스트 추가할게용 (왜 여태 테스트 없었죠? 😄)

허허허허

kpeel5839 · 2023-10-05T13:08:31Z

backend/src/main/java/com/mapbefine/mapbefine/bookmark/domain/BookmarkRepository.java

 import java.util.Optional;
+import org.springframework.data.jpa.repository.JpaRepository;

 public interface BookmarkRepository extends JpaRepository<Bookmark, Long> {

    boolean existsByMemberIdAndTopicId(Long memberId, Long topicId);



이건 쥬니 글에서 보았듯 @query 어노테이션이 동반되지 않으면 짜피 clearAutomatically 가 동작하지 않으니까 빼신게 맞나요 ?

코멘트 위치가 잘못된듯요

또잉또잉 클릭 잘못했어연 바로 아래줄일듯 허허허

kpeel5839 · 2023-10-05T13:12:45Z

backend/src/main/java/com/mapbefine/mapbefine/member/domain/Member.java

+    public void updateStatus(Status status) {
+        memberInfo = memberInfo.createUpdatedMemberInfo(status);
+    }
+
    public void addTopic(Topic topic) {


형태를 통일해주셨군요 굳

kpeel5839 · 2023-10-05T13:14:30Z

backend/src/main/java/com/mapbefine/mapbefine/topic/application/TopicCommandService.java

+                .orElseThrow(
+                        () -> new NoSuchElementException("findCreatorByAuthMember; member not found; id=" + memberId));
    }


.orElseThrow( () -> new NoSuchElementException("findCreatorByAuthMember; member not found; id=" + memberId) );

도 괜찮아보입니다 ^.^

반영할게용

kpeel5839 · 2023-10-05T13:17:20Z

backend/src/main/java/com/mapbefine/mapbefine/topic/application/TopicCommandService.java

        pinRepository.deleteAllByTopicId(topicId);
+        /// TODO: 2023/10/05  topic의 pinCount는?
        topicRepository.deleteById(topicId);
    }


soft delete 의 대가인 것 같기도..

kpeel5839 · 2023-10-05T13:19:47Z

...end/src/test/java/com/mapbefine/mapbefine/location/application/LocationQueryServiceTest.java

+                createAndSaveTopic("준팍의 또간집", 2),
+                createAndSaveTopic("도이의 또간집", 3),
+                createAndSaveTopic("패트릭의 또간집", 4)
        );


왜 저는 안 넣어주시죠?

아 ㅋㅋ 나도 없네
이 아래에 있는 코드 리뷰 안함 ㅂ2

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 귀요미 ㅎㅎ

kpeel5839 · 2023-10-05T13:20:49Z

...end/src/test/java/com/mapbefine/mapbefine/location/application/LocationQueryServiceTest.java

+        for (int i = 0; i < deleteCounts; i++) {
+            Pin delete = topic.getPins().get(i);
+            delete.decrementTopicPinCount();
+            pinRepository.deleteById(delete.getId());
+        }
+    }


update pin p, topic t set p.is_deleted = true, t.pin_count = t.pin_count - 1 where p.id = "삭제하려는 pin id" AND t.id = p.topic_id;

역시 SQL 천재시네요

반영안할거잖아요!

kpeel5839 · 2023-10-05T13:25:13Z

backend/src/test/java/com/mapbefine/mapbefine/admin/application/AdminCommandServiceTest.java

-        PinImage deletedPinImage = pinImageRepository.findById(pinImage.getId()).get();
-
-        assertThat(deletedPinImage.isDeleted()).isTrue();
+        assertThat(pinImageRepository.findById(pinImage.getId())).isEmpty();


바로 위 테스트의 then 절과 해당 테스트의 then 절의 삭제 유무 확인 방식이 조금 다르네요!

킹시 의도하신걸까요 ?

바로 위 테스트가 뭘까요 ? existById와의 차이라면 pinImage는 List라서 이렇게 했습니다 !

글 헛 군 요!

cpot5620

도이 고생 많았어요 !!!

TODO 부분에 의견도 달아두었고, 코드 관련해서 제안드리는 부분이 있어서 RC 하겠습니당.

cpot5620 · 2023-10-05T15:12:13Z

backend/src/main/java/com/mapbefine/mapbefine/admin/application/AdminCommandService.java

        permissionRepository.deleteAllByMemberId(memberId);
        atlasRepository.deleteAllByMemberId(memberId);
        bookmarkRepository.deleteAllByMemberId(memberId);
+        entityManager.flush();


사실 저도 entityManager가 존재한다는게 거슬리긴 해요.

준팍이 말씀해주신

entityManager를 주입받을 필요없이 repository에서 flush하는 건 어떨까요??

이 부분은 별도의 설정하지 않아도 된다는 의미일까요 ?

아마, 테스트 코드 구성을 잘못해서 발생한 문제를 해결하기 위해 위와같은 구조를 갖게 된 걸로 알고있어요. (내가 짬 ㅋ)
테스트 코드에서 //given에서 데이터를 세팅해주는 작업과 같은 트랜잭션으로 묶이기 때문에 flush를 강제로 호출해주지 않으면 수정 관련 쿼리가 쓰기 지연 저장소에 저장되었다가, 트랜잭션이 커밋되기 직전에 데이터베이스로 쿼리를 날라가게 되요.
어떻게 해결 할 수 있다는 의미인지 궁금합니다 !

cpot5620 · 2023-10-05T15:14:26Z

backend/src/main/java/com/mapbefine/mapbefine/admin/application/AdminCommandService.java

+        Pin pin = pinRepository.findById(pinId)
+                .orElseThrow(() -> new PinNotFoundException(PIN_NOT_FOUND, pinId));
+
+        /// TODO: 2023/10/05 soft delete를 @Query로 구현하면서, pinCount 반영을 통합할 수 없음


지금 방법대로 하면 안 되는건가요 ?
상관 없어보이는데..

cpot5620 · 2023-10-05T15:15:34Z

backend/src/main/java/com/mapbefine/mapbefine/auth/application/AuthService.java

@@ -31,6 +30,7 @@ public void validateMember(Long memberId) {
        }
    }

+    // TODO 테스트가 필요하긴 한데, MemberQueryService와 너무 중복되는 내용이다. 엔티티 테스트로 중복을 없앨 수 있으면 좋겠다.


언젠가 MemberQueryService의 스펙이 바뀔수도 혹은 AuthService의 스펙이 바뀔 수도 있을 것 같은데요.
그런 의미에서, 중복되는 테스트라고 할지라도 수행하는 것이 좋지 않을까요 ?

cpot5620 · 2023-10-05T15:16:01Z

backend/src/main/java/com/mapbefine/mapbefine/bookmark/domain/BookmarkRepository.java

 import java.util.Optional;
+import org.springframework.data.jpa.repository.JpaRepository;

 public interface BookmarkRepository extends JpaRepository<Bookmark, Long> {

    boolean existsByMemberIdAndTopicId(Long memberId, Long topicId);



코멘트 위치가 잘못된듯요

cpot5620 · 2023-10-05T15:16:33Z

backend/src/main/java/com/mapbefine/mapbefine/bookmark/domain/BookmarkRepository.java


 public interface BookmarkRepository extends JpaRepository<Bookmark, Long> {

    boolean existsByMemberIdAndTopicId(Long memberId, Long topicId);

-    @Modifying(clearAutomatically = true)


이 부분 JPQL로 해결해야하는데... 도이꺼 PR 머지되면 브랜치 따서 해결할게요.

잊지말자 !!!!!!!!!!!!

여기다가 제가 단 거 였는데 ㅋㅋㅋㅋ

Service에서 EntityManager 불러오는 대신 JPQL로 바꾸고 @Modifying 다시 적용한다는거죵 ?
아주 좋습니다아

cpot5620 · 2023-10-05T15:17:39Z

backend/src/main/java/com/mapbefine/mapbefine/pin/domain/Pin.java

@@ -102,6 +104,10 @@ public void updatePinInfo(String name, String description) {
        pinInfo = PinInfo.of(name, description);
    }

+    public void decrementTopicPinCount() {


decrement는 명사 아닌가요 ?

엥 맞아요 왜 저렇게 썼을까요 ? 영어 천재시네요 . 고칠게용

천재아닌데요!

cpot5620 · 2023-10-05T15:18:54Z

backend/src/main/java/com/mapbefine/mapbefine/topic/application/TopicCommandService.java

+        /// TODO: 2023/10/05  topic의 pinCount는?
        topicRepository.deleteById(topicId);
    }


Pin 자체가 hard-delete 된게 아니다 보니, pinCount 값은 그대로 두어도 무방할 것 같은데요 ?

pin이 삭제될 때에만 topic의 pinCount 값을 신경쓰고요 !

topic 살면 pin 도 다 다시 살아나니까?

저도 똑같이 생각해서 적용 안했는데 주석이 남아있었네요! 지울게욥

cpot5620 · 2023-10-05T15:19:08Z

backend/src/main/java/com/mapbefine/mapbefine/topic/domain/Topic.java

@@ -147,6 +149,10 @@ public void removeImage() {
        this.topicInfo = topicInfo.removeImage();
    }

+    public void decrementPinCount() {


명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사명사

ㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇㅇ

cpot5620 · 2023-10-05T15:19:37Z

...end/src/test/java/com/mapbefine/mapbefine/location/application/LocationQueryServiceTest.java

+                createAndSaveTopic("준팍의 또간집", 2),
+                createAndSaveTopic("도이의 또간집", 3),
+                createAndSaveTopic("패트릭의 또간집", 4)
        );


아 ㅋㅋ 나도 없네
이 아래에 있는 코드 리뷰 안함 ㅂ2

yoondgu · 2023-10-05T18:00:43Z

말씀해주신 사항들 반영했습니다 !
delete flush문제는 일단 repository.flush를 호출하도록 변경했고,
@Modifying으로 일괄 적용은 쥬니가 추가 PR에서 하신다고 하셔서 따로 작업하지 않았습니다 ㅎㅎ
AuthServiceTest 작성하면서 차단,탈퇴 사용자 예외처리도 추가했습니다.
그리고 작업하면서 사용하지 않는 API인데 중복 코드여서 작업하는 게 무의미하게 느껴지는 부분은 @Deprecated 처리했어요.

kpeel5839

도이 넘흐넘흐 고생하셨어요!

Approve 드립니다!

kpeel5839 · 2023-10-06T01:20:11Z

backend/src/main/java/com/mapbefine/mapbefine/admin/application/AdminCommandService.java

+        Pin pin = pinRepository.findById(pinId)
+                .orElseThrow(() -> new PinNotFoundException(PIN_NOT_FOUND, pinId));
+
+        /// TODO: 2023/10/05 soft delete를 @Query로 구현하면서, pinCount 반영을 통합할 수 없음


kpeel5839 · 2023-10-06T01:53:35Z

backend/src/main/java/com/mapbefine/mapbefine/admin/application/AdminCommandService.java

+        permissionRepository.deleteAllByTopicId(topicId);
+        permissionRepository.flush();
+        atlasRepository.deleteAllByTopicId(topicId);
+        atlasRepository.flush();
+        bookmarkRepository.deleteAllByTopicId(topicId);
+        bookmarkRepository.flush();
+        pinImageRepository.deleteAllByPinIds(pinIds);
+        pinRepository.deleteAllByTopicId(topicId);


하나의 repository 가 flush 의무를 갖는 것이 애매하기 때문에 모든 repository 가 flush 를 호출할 수 있도록 하셨군요! 좋습니다.

그리고, 이 부분은 추후에 개선할 수 있으면 개선하도록 해보죠! (명시적으로 flush 호출 안해보도록)

앗 저 각각 해야 하는줄알았어요...ㅎㅎ 근데 매튜 말대로 곧 사라질 내용인걸로!

kpeel5839 · 2023-10-06T01:55:10Z

backend/src/test/java/com/mapbefine/mapbefine/auth/application/AuthServiceTest.java

+                .isInstanceOf(UnauthorizedException.class);
+    }
+
+    @Transactional(propagation = Propagation.REQUIRES_NEW)


야무지게 쓰셨네요

kpeel5839 · 2023-10-06T01:57:02Z

backend/src/main/java/com/mapbefine/mapbefine/permission/presentation/PermissionController.java

@@ -56,8 +56,7 @@ public ResponseEntity<TopicAccessDetailResponse> findTopicAccessDetailByTopicId(
        return ResponseEntity.ok(response);
    }

-    // TODO 이 API를 쓰는 곳이 있나? + 결국 특정 회원을 조회하는 건데 어떤 API인지 알기 어렵다..
-    //  회원 정보 조회는 /members 에서 하는 걸로 충분하지 않나? 재사용성이 떨어진다. 테스트의 DisplayName도 매칭이 안된다.
+    @Deprecated(since = "2023.10.06")


cpot5620

+1

yoondgu added 6 commits October 5, 2023 11:39

fix: Pin @where 적용으로 지연 로딩 시 soft delete 반영

8ef8e54

fix: Pin 삭제 시 pinCount 반영하도록 수정

9db5da3

- 검토할 TODO 다수 존재

fix: Pin @where 적용에 따라 JPQL 원복

ca7948b

refactor: Member update 메서드 형식 통일

1af7a11

fix: Topic @where 적용으로 지연 로딩 시 soft delete 반영

28e0e4e

fix: PinImage @where 적용으로 지연 로딩 시 soft delete 반영

83efb4b

yoondgu added bug Something isn't working BE 백엔드 관련 이슈 우선순위 : 상 refactor 리팩토링 관련 이슈 labels Oct 5, 2023

yoondgu added this to the 최종 데모데이 milestone Oct 5, 2023

yoondgu requested review from cpot5620, kpeel5839 and junpakPark October 5, 2023 08:21

yoondgu self-assigned this Oct 5, 2023

yoondgu changed the title ~~[BE] Refactor/#540 지연로딩으로 조회하는 경우, soft delete을 반영할 수 없음~~ [BE] Refactor/#540 지연로딩의 경우, soft delete을 반영할 수 없는 문제 해결 Oct 5, 2023

junpakPark approved these changes Oct 5, 2023

View reviewed changes

kpeel5839 requested changes Oct 5, 2023

View reviewed changes

cpot5620 requested changes Oct 5, 2023

View reviewed changes

yoondgu added 10 commits October 6, 2023 01:12

refactor: 메서드명 수정

c030c84

refactor: AdminCommandService 의 불필요한 EntityManager 의존성 제거

5f90719

refactor: 누락된 삭제 로직 TODO 작성

279566f

refactor: 개행 수정

d3c8c7e

refactor: 개행 수정

05c62bc

fix: 토픽 삭제 중복 로직이나 사용되지 않는 API 코드 @deprecated 처리

946c65a

fix: 토픽 삭제 시 연관 관계 지우지 않는 부분 수정

43fc9fc

fix: AuthMember 생성 시 정상 사용자만 조회하도록 filter 추가, 검증 추가

26d98b8

test: AuthServiceTest 작성

b75fdb6

chore: /permissions/id API @deprecated 추가

9df7456

kpeel5839 approved these changes Oct 6, 2023

View reviewed changes

cpot5620 approved these changes Oct 6, 2023

View reviewed changes

cpot5620 merged commit 9776f6e into develop-BE Oct 6, 2023
3 checks passed

yoondgu deleted the refactor/#540 branch October 6, 2023 15:14

[BE] Refactor/#540 지연로딩의 경우, soft delete을 반영할 수 없는 문제 해결 #553

[BE] Refactor/#540 지연로딩의 경우, soft delete을 반영할 수 없는 문제 해결 #553

Conversation

yoondgu commented Oct 5, 2023 • edited Loading

작업 대상

고려해야 할 연관관계 (테스트 작성)

작업 내용

📄 작업 내용

추가 작업

🙋🏻 주의 사항

스크린샷

📎 관련 이슈

레퍼런스

github-actions bot commented Oct 5, 2023 • edited Loading

Unit Test Results

junpakPark left a comment

Choose a reason for hiding this comment

junpakPark Oct 5, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

yoondgu Oct 5, 2023 • edited Loading

Choose a reason for hiding this comment

중간에 flush를 해줘야 하는 이유

Choose a reason for hiding this comment

junpakPark Oct 5, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kpeel5839 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

yoondgu Oct 5, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

yoondgu Oct 5, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

cpot5620 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

yoondgu Oct 5, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

yoondgu commented Oct 5, 2023 •

edited

Loading

github-actions bot commented Oct 5, 2023 •

edited

Loading

junpakPark Oct 5, 2023 •

edited

Loading

yoondgu Oct 5, 2023 •

edited

Loading

junpakPark Oct 5, 2023 •

edited

Loading

yoondgu Oct 5, 2023 •

edited

Loading

yoondgu Oct 5, 2023 •

edited

Loading

yoondgu Oct 5, 2023 •

edited

Loading

yoondgu commented Oct 5, 2023 •

edited

Loading