이미지url에 id 대신 파일 이름을 사용하도록 변경 #717

kwonyj1022 · 2023-10-18T18:07:56Z

📄 작업 내용 요약

이미지 url에 id 대신 파일 이름을 사용하도록 변경
이미지 이름 사용으로 인한 n+1 개선
이미지 파일 조회 시 이미지 이름 기반의 url로 통신하도록 api 수정

🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드

file changes가 많지만, ImageCalculator.calculateBy()의 파라미터와 반환값 타입이 달라져서 그렇습니다.
n+1 개선 부분은 도메인별로 커밋을 나눠두었습니다.
cloudfront 활용해서 s3에서 조회하는 것은 캠퍼스 가서 aws 접속해서 좀 더 알아보고 추가 또는 별도의 pr로 만들겠습니다.

📎 Issue 번호

closed 이미지url에 id 대신 파일 이름을 사용하도록 변경 #715

JJ503

엔초 자잘한 수정 사항이 많았을텐데 정말 고생 많으셨습니다!
제 눈에 별다른 수정 사항은 보이지 않아 approve 합니다.
다만, 궁금한 점이 있어 질문 남겨두었으니 해당 부분만 확이해주시면 감사하겠습니다.

JJ503 · 2023-11-01T15:32:11Z

...java/com/ddang/ddang/chat/infrastructure/persistence/QuerydslChatRoomAndImageRepository.java

-                            .leftJoin(chatRoom.buyer).fetchJoin()
-                            .leftJoin(chatRoom.auction, auction).fetchJoin()
-                            .leftJoin(auction.seller).fetchJoin()
+                            .join(chatRoom.buyer).fetchJoin()
+                            .join(chatRoom.auction, auction).fetchJoin()
+                            .join(auction.seller).fetchJoin()
+                            .leftJoin(auction.seller.profileImage).fetchJoin()
                            .leftJoin(auctionImage).on(auctionImage.id.eq(


칭찬

👍👍👍

JJ503 · 2023-11-01T15:33:08Z

backend/ddang/src/main/java/com/ddang/ddang/image/domain/ProfileImage.java

-    // TODO: 10/13/23 앞으로 id가 아닌 store name으로 진행하기로 했는데, 임시로 해둡니다. 추후 삭제해주시면 감사하겠습니다.
-    public static final String DEFAULT_PROFILE_IMAGE_ID = "1";


칭찬

잊지않고 해주셨군요 👍

칭찬

훌륭합니다 👍

JJ503 · 2023-11-01T15:51:40Z

backend/ddang/src/main/java/com/ddang/ddang/qna/infrastructure/JpaQuestionRepository.java

-        JOIN FETCH q.writer
+        JOIN FETCH q.writer w
+        LEFT JOIN FETCH w.profileImage
        LEFT JOIN FETCH q.answer
        JOIN FETCH q.auction a
-        JOIN FETCH a.seller
+        JOIN FETCH a.seller s
+        JOIN FETCH s.profileImage


칭찬

storename으로 변경되면서 fetch join을 해줘야 하게 됐군요..!
수정하시느라 고생 많으셨습니다 👍👍👍

질문

제이미가 남겨주신 칭찬에 대한 질문입니다!!
이해한 게 맞는지 궁금해서.. 남겨요
storename은 id가 아니고, id가 아닌 다른 필드를 조회하기 위해 추가쿼리가 발생하기 때문에 fetch join을 해야한다고 이해했는데 맞나요?!

근데 엔초 진짜 꼼꼼하고 똑똑하네요

넵 맞습니다!!

JJ503 · 2023-11-01T15:54:44Z

...rc/main/java/com/ddang/ddang/image/infrastructure/persistence/JpaAuctionImageRepository.java

+    @Query("""
+        SELECT COUNT(auction_image) > 0
+        FROM AuctionImage auction_image
+        WHERE auction_image.image.storeName = :storeName
+    """)
+    boolean existsByStoreName(final String storeName);


질문

네이밍만으로 충분해보이는데 @Query로 해줘야 하나요?

넵 AuctionImage 안에 Image가 임베디드로 있어서 JPQL 없이 메서드 네이밍만으로는 불가능합니다.

apptie

문제가 없어 보이므로 approve 하겠습니다
제이미의 문서 최신화랑 충돌날 것 같은데 문서 충돌이야 뭐...괜찮겠죠

apptie · 2023-11-01T16:04:31Z

backend/ddang/src/main/java/com/ddang/ddang/image/domain/ProfileImage.java

-    // TODO: 10/13/23 앞으로 id가 아닌 store name으로 진행하기로 했는데, 임시로 해둡니다. 추후 삭제해주시면 감사하겠습니다.
-    public static final String DEFAULT_PROFILE_IMAGE_ID = "1";


칭찬

훌륭합니다 👍

swonny

엔초 진짜 자잘한 수정사항이 많았는데 역시 꼼꼼하게 잘 해주셨군요
감사합니다
고생하셨습니다!!

swonny · 2023-11-03T01:50:26Z

backend/ddang/src/main/java/com/ddang/ddang/user/domain/repository/UserRepository.java

@@ -9,6 +9,7 @@ public interface UserRepository {
    User save(final User user);

    Optional<User> findById(final Long id);
+    Optional<User> findByIdWithProfileImage(final Long id);


필수

개행이 한 줄 필요해보입니당

swonny · 2023-11-03T01:51:52Z

backend/ddang/src/main/java/com/ddang/ddang/image/presentation/ImageController.java

-    public ResponseEntity<Resource> downloadProfileImage(@PathVariable Long id) throws MalformedURLException {
-        final Resource resource = imageService.readProfileImage(id);
+    @GetMapping("/users/images/{storeName}")
+    public ResponseEntity<Resource> downloadProfileImage(@PathVariable String storeName) throws MalformedURLException {


필수

final이 누락되어버렸어요..!

swonny · 2023-11-03T01:51:59Z

backend/ddang/src/main/java/com/ddang/ddang/image/presentation/ImageController.java

-    public ResponseEntity<Resource> downloadAuctionImage(@PathVariable Long id) throws MalformedURLException {
-        final Resource resource = imageService.readAuctionImage(id);
+    @GetMapping("/auctions/images/{storeName}")
+    public ResponseEntity<Resource> downloadAuctionImage(@PathVariable String storeName) throws MalformedURLException {


필수

final이 누락되어버렸어요..! 22

swonny · 2023-11-03T01:52:53Z

...est/java/com/ddang/ddang/image/infrastructure/persistence/JpaAuctionImageRepositoryTest.java

        assertThat(actual.getId()).isPositive();
    }

+


필수

이번엔 개행이 두 번 추가되었어요!

# Conflicts: # backend/ddang/src/main/java/com/ddang/ddang/user/presentation/dto/ReadUserResponse.java # backend/ddang/src/main/java/com/ddang/ddang/user/presentation/dto/response/ReadAuctionDetailResponse.java # backend/ddang/src/main/java/com/ddang/ddang/user/presentation/dto/response/ReadUserResponse.java # backend/ddang/src/main/resources/static/docs/docs.html

swonny

확인 완료했어요~~ 고생하셨습니다 엔초

# Conflicts: # backend/ddang/src/main/java/com/ddang/ddang/chat/infrastructure/persistence/QuerydslChatRoomAndMessageAndImageRepository.java # backend/ddang/src/main/java/com/ddang/ddang/user/domain/repository/UserRepository.java # backend/ddang/src/main/java/com/ddang/ddang/user/infrastructure/persistence/UserRepositoryImpl.java # backend/ddang/src/test/java/com/ddang/ddang/chat/presentation/fixture/ChatRoomControllerFixture.java

kwonyj1022 added 12 commits October 19, 2023 00:43

refactor: 이미지 상대 url 기반으로 url을 계산하는 메서드 파라미터로 이미지 id 대신 이미지 이름을 받도록 변경

f1fff59

refactor: 이미지 절대 url 기반으로 url을 계산하는 메서드 파라미터로 이미지 id 대신 이미지 이름을 받도록 변경

0c7b648

refactor: 경매 조회 시 판매자 프로필 이미지 조회로 인한 n+1 개선

1375972

refactor: 입찰 목록 조회 시 입찰자 정보 조회로 인한 n+1 개선

fd9e440

refactor: 채팅방 관련 조회 시 참여자 프로필 이미지 조회로 인한 n+1 개선

05b09e2

refactor: qna 관련 조회 시 사용자 프로필 이미지 조회로 인한 n+1 개선

69a134f

refactor: 신고 관련 조회 시 사용자 프로필 이미지 조회로 인한 n+1 개선

a02c1d6

refactor: 평가 관련 조회 시 사용자 프로필 이미지 조회로 인한 n+1 개선

c98c3a7

refactor: 사용자 정보 조회 시 사용자 프로필 이미지 조회로 인한 n+1 개선

7ad27b7

refactor: 메시지 알림 시 작성자 프로필 이미지 조회로 인한 n+1 개선

f6c2d4e

feat: 이미지 이름이 포함된 url로 조회 시 이미지 파일을 반환하는 기능 추가

5aa95a9

docs: 문서 최신화

f66f23e

kwonyj1022 added refactor 기존 기능에 변경이 없는 구현 변경 시 backend 백엔드와 관련된 이슈나 PR에 사용 labels Oct 18, 2023

kwonyj1022 added this to the 최종 데모데이 milestone Oct 18, 2023

kwonyj1022 requested review from apptie, JJ503 and swonny October 18, 2023 18:07

kwonyj1022 self-assigned this Oct 18, 2023

JJ503 approved these changes Nov 1, 2023

View reviewed changes

apptie approved these changes Nov 1, 2023

View reviewed changes

swonny approved these changes Nov 3, 2023

View reviewed changes

kwonyj1022 added 3 commits November 3, 2023 11:13

style: 개행 수정 및 누락된 final 추가

d56bda4

rename: test에 사용되는 fixture 클래스 이름 변경

e5e8cad

swonny approved these changes Nov 3, 2023

View reviewed changes

kwonyj1022 added 2 commits November 17, 2023 16:14

ci: 충돌 해결

beebe84

kwonyj1022 merged commit 07bad46 into develop-be Nov 17, 2023
1 check passed

kwonyj1022 deleted the refactor/715 branch November 17, 2023 07:31

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

이미지url에 id 대신 파일 이름을 사용하도록 변경 #717

이미지url에 id 대신 파일 이름을 사용하도록 변경 #717

kwonyj1022 commented Oct 18, 2023 •

edited

Loading

JJ503 left a comment

JJ503 Nov 1, 2023

JJ503 Nov 1, 2023

apptie Nov 1, 2023

JJ503 Nov 1, 2023

swonny Nov 3, 2023 •

edited

Loading

kwonyj1022 Nov 3, 2023

JJ503 Nov 1, 2023

kwonyj1022 Nov 3, 2023 •

edited

Loading

apptie left a comment

apptie Nov 1, 2023

swonny left a comment

swonny Nov 3, 2023

swonny Nov 3, 2023

swonny Nov 3, 2023

swonny Nov 3, 2023

swonny left a comment

		// TODO: 10/13/23 앞으로 id가 아닌 store name으로 진행하기로 했는데, 임시로 해둡니다. 추후 삭제해주시면 감사하겠습니다.
		public static final String DEFAULT_PROFILE_IMAGE_ID = "1";

이미지url에 id 대신 파일 이름을 사용하도록 변경 #717

이미지url에 id 대신 파일 이름을 사용하도록 변경 #717

Conversation

kwonyj1022 commented Oct 18, 2023 • edited Loading

📄 작업 내용 요약

🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드

📎 Issue 번호

JJ503 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

칭찬

swonny Nov 3, 2023 • edited Loading

Choose a reason for hiding this comment

질문

Choose a reason for hiding this comment

Choose a reason for hiding this comment

질문

kwonyj1022 Nov 3, 2023 • edited Loading

Choose a reason for hiding this comment

apptie left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

칭찬

swonny 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

필수

swonny left a comment

Choose a reason for hiding this comment

kwonyj1022 commented Oct 18, 2023 •

edited

Loading

swonny Nov 3, 2023 •

edited

Loading

kwonyj1022 Nov 3, 2023 •

edited

Loading