Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor/#233, #235 장터게시판 fetch join 코드 추가 및 게시글 생성 제한 기능 추가 #236

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

JeongeunChoi
Copy link
Contributor

@JeongeunChoi JeongeunChoi commented Oct 11, 2023

🤨 Motivation

🔑 Key Changes

  • 장터게시판 검색 기능에 fetch join 적용하는 코드 추가(기존 코드는 그대로 있고, 추가만 해뒀습니다)
  • fetch join을 적용한 경우와 그렇지 않은 경우, 테스트 코드 작성을 통해 시간 비교
    • fetch join 적용한 경우
      image
    • fetch join 적용하지 않은 경우
      image
  • 게시글 생성 후 5분 후 재 생성 가능하도록 수정
    image

🙏 To Reviewers

  • 게시글의 좋아요 수와 댓글 수를 가져오는 코드를 fetch join으로 바꾸니, 레코드 자체를 가져와서 카운트하는 개념이라 쿼리가 더 많이 나가서 시간이 오래 걸리는 것 같습니다! 우선, 시간적인 측면에서 이득이 없어서 코드를 수정하지 않고 추가만 해두었습니다 :)

@JeongeunChoi JeongeunChoi self-assigned this Oct 11, 2023
@JeongeunChoi JeongeunChoi added 🧪test 테스팅 관련 이슈 ⚙️refactor 리팩토링 관련 이슈 labels Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️refactor 리팩토링 관련 이슈 🧪test 테스팅 관련 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant