Skip to content

[Dk-459] 매치 조회 경기일자별 페이징 추가 - #85

Open
pjh612 wants to merge 5 commits into
developfrom
DK-459
Open

[Dk-459] 매치 조회 경기일자별 페이징 추가#85
pjh612 wants to merge 5 commits into
developfrom
DK-459

Conversation

@pjh612

@pjh612 pjh612 commented Aug 12, 2022

Copy link
Copy Markdown
Collaborator

✅  작업 단위

  • [DK-459 feat: 경기일자별 매치 리스트 조회 기능 추가](322059b)
    • 지정한 매치 날짜 타입별로 페이징 하고 싶을때 searchDateType을 CREATED_AT 또는 MATCH_DATE로 지정을 해 페이징할 수 있도록 변경했습니다.
    • 경기일자 기준 조회는 가장 빠른 경기 순으로 보여집니다.
      • matchDate를 입력해 오늘 기준으로 가장 빠른 경기일자의 매칭을 조회할 수 있습니다.
      • id는 입력하지 않을 시 기본값으로 0입니다.
      • id를 사용한 matchDate 중복 회피는 여전히 하셔야합니다.
    • 작성일자 기준 조회는 가장 최근 글을 보여줍니다.
      • createdAt을 입력해 특정시간부터 가장 최근 글을 조회할 수 있습니다.
      • id는 입력하지 않을 시 기본값으로 Long MAX입니다.
      • id를 사용한 createdAt 중복 회피는 여전히 하셔야합니다.
        이전에는 id와 createdAt 둘 중 하나가 빠지면 validation 에러가 떴는데 현재는 id의 기본값설정이 되어 조금 더 유연하게 사용할 수 있습니다.(id는 모르겠고 createdAt, matchDate를 입력해 특정 시작지점부터 페이징 하겠음. -> 오늘을 기준으로, 또는 지금을 기준으로)

요청/응답 예시

🤔 고민 했던 부분

페이징 ID 기본값을 Long MAX로 했는데 이거 DB 조회 성능상 문제 없나요?.. 아시는분?

@midasWorld midasWorld left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다! 👍
Long.MAX_VALUE 는 조건절에서 비교연산자로 쓰는거면 성능엔 상관 없을것 같습니다.

@HyoungUkJJang HyoungUkJJang left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다 !

@HYEBPARK HYEBPARK left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

진형님 수고하셨습니당 :)

@github-actions github-actions Bot closed this Aug 13, 2022

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

테스트코드를 다시 확인해주세요.

@midasWorld midasWorld reopened this Aug 13, 2022

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

테스트코드를 다시 확인해주세요.

@github-actions github-actions Bot closed this Aug 13, 2022
@midasWorld midasWorld reopened this Aug 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants