2주차 리뷰#173
Conversation
…168-redis ♻️ refactor : 수정 사항 리팩토링
…168-redis ✨ feat : 결제 취소 반환값 추가 및 예약 수정사항 적용
# Conflicts: # src/main/java/roomit/main/domain/chat/chatmessage/service/ChatService.java # src/main/java/roomit/main/domain/chat/chatroom/controller/ChatRoomController.java # src/main/java/roomit/main/domain/chat/chatroom/dto/ChatRoomBusinessResponse.java # src/main/java/roomit/main/domain/chat/chatroom/dto/ChatRoomMemberResponse.java # src/main/java/roomit/main/domain/chat/chatroom/dto/response/ChatRoomResponse.java # src/main/java/roomit/main/domain/chat/chatroom/entity/ChatRoom.java # src/main/java/roomit/main/domain/chat/chatroom/repositoroy/ChatRoomRepository.java # src/main/java/roomit/main/domain/chat/chatroom/service/ChatRoomService.java # src/test/java/roomit/main/domain/chat/service/ChatServiceIntegrationTest.java
Account-4-Review
left a comment
Account-4-Review
left a comment
There was a problem hiding this comment.
우선 러프하게 작성해봤습니다.
스타일이 너무 어긋난게 아니면 대부분 패스했고요, (시간 문제 때문에 ㅠㅠ) 러프하게 본지라 아직 로직을 구체적으로 살펴보진 않았습니다.
내일 조금 더 살펴보고 추가 리뷰 남길게요.
| @PutMapping("api/v1/business") | ||
| public void businessModify(@RequestBody @Valid BusinessUpdateRequest businessUpdateRequest | ||
| ,@AuthenticationPrincipal CustomBusinessDetails customBusinessDetails){ | ||
| @PutMapping("") |
There was a problem hiding this comment.
사소한거긴 한데, 사실 이런 경우엔 그냥 @PutMapping 을 써도 됩니다.
| @Query(value = "SELECT CASE WHEN COUNT(b) > 0 THEN true ELSE false END FROM Business b WHERE b.businessNum.value=:businessNum") | ||
| Boolean existsByBusinessNum(String businessNum); | ||
|
|
||
| @Query(""" |
There was a problem hiding this comment.
존재 여부를 확인하는 건가요?
이 경우, EXIST 을 걸면 좋습니다.
한 개 검색이 되는 순간 쿼리를 종료시켜 버리거든요.
@Query("""
SELECT CASE WHEN EXISTS (
SELECT 1
FROM Business b
WHERE b.businessId = :senderId AND b.businessName = :senderName
) THEN TRUE ELSE FALSE END
""")
boolean existsByIdAndBusinessName(Long senderId, String senderName);| Optional<Business> findByBusinessEmail(String businessEmail); | ||
|
|
||
| @Query(value = "SELECT CASE WHEN COUNT(b) > 0 THEN true ELSE false END FROM Business b WHERE b.businessName.value=:businessName") | ||
| boolean existsByBusinessName(String businessName); |
|
|
||
| import java.time.LocalDateTime; | ||
|
|
||
| @Builder |
There was a problem hiding this comment.
직접 Builder 를 쓸 일은 없겠죠?
(타 record 포함)
| return tokenService.memberLogin(loginRequest,response); | ||
| } | ||
|
|
||
| @PostMapping("/login/business") |
|
|
||
| @Scheduled(fixedRate = 60000) // 1분마다 실행 | ||
| public void flushMessages() { | ||
| List<Long> roomIds = chatRoomRepository.findAllRoomIds(); // Room ID 동적 조회 |
There was a problem hiding this comment.
roomId 의 수가 끝도 없이 많을 수 있지 않을까요?
| redisPublisher.publish(topic, request); | ||
|
|
||
| // Redis에 저장 | ||
| saveMessageToRedis(request); |
There was a problem hiding this comment.
해당 로직은, "Redis에 1분간 발생한 모든 채팅의 내용이 전부 적재될 수 있다" 라는 전제하에 수행되는 로직으로 보여요.
그렇지만... 여러분들 서버의 Redis 메모리는 안녕하실까요?
There was a problem hiding this comment.
ttl에 따라 일정 시간이 지나면 삭제되도록 했는데 그래도 문제가 될까요?
|
|
||
| ChatMessage message = new ChatMessage(room, request); | ||
|
|
||
| messageRepository.save(message); |
There was a problem hiding this comment.
결국 이것도 모든 데이터에 대해 일일히 INSERT를 날리는거 아닌가요?
묶어서 처리한다고 해도, 결국 성능을 별 차이 없을 것으로 보여요.
| END | ||
| ) | ||
| FROM ChatRoom c | ||
| LEFT JOIN c.messages m |
There was a problem hiding this comment.
쿼리의 실행계획을 확인해 주세요. 쿼리 성능 병목으로 인해 발목잡힐 수 있을 것 같아보여요.
There was a problem hiding this comment.
혹시 이렇게 쿼리를 바꿨는데 이렇게 하면 될까요?
| "AND w.workplace_longitude BETWEEN :left AND :right " + | ||
| "ORDER BY distance ASC", nativeQuery = true) | ||
| @Query(value = """ | ||
| SELECT w.workplace_id, w.workplace_name, |
There was a problem hiding this comment.
장소라면, 공간인덱스를 설정하고 관련 함수를 쓰는게 훨씬 성능이 좋고, 정확하지 않을까요?
…168-redis feat : 결제 취소 반환값 추가 및 예약 수정사항 적용
develop 브랜치 clone
# Conflicts: # src/main/java/roomit/main/domain/reservation/service/ReservationService.java
|
6 participants
🛠️ 과제 요약
📝 요구 사항과 구현 내용
✅ 피드백 반영사항
💬 PR 포인트 & 궁금한 점
🔗 연관된 이슈