[🌎 Feature] 타임 라인용 목표 조회 API 개발 - Pagination 방식 offset 기반으로 변경 #243

egg528 · 2024-03-27T03:28:29Z

📒 Issue

[🌎 Feature] 타임 라인용 목표 조회 API 개발 #235

🎯 어떤 작업을 했는지

타임 라인용 목표 조회 API Pagination 방식 offset 기반으로 변경

🌍 영향범위 (모듈)

application/api

Resolves: # See also: #

…Map)

#235

binary-ho · 2024-03-27T13:35:53Z

Pagination 방식 Offset 기반으로 바뀌게 된 이유가 궁금합니다! 혹시 설명해주실 수 있을까요?

제가 페이지네이션 방식에 대해 잘 모르지만, 오늘 아침 보내주셨던 자료에는 무한 스크롤 상황에선 커서 기반 구현이 좋다는 이야기를 봤던 것 같아 질문 드립니다 ㅎㅎ

egg528 · 2024-03-27T13:39:01Z

Pagination 방식 Offset 기반으로 바뀌게 된 이유가 궁금합니다! 혹시 설명해주실 수 있을까요?

제가 페이지네이션 방식에 대해 잘 모르지만, 오늘 아침 보내주셨던 자료에는 무한 스크롤 상황에선 커서 기반 구현이 좋다는 이야기를 봤던 것 같아 질문 드립니다 ㅎㅎ

@binary-ho
CursorId가 중복이 있을 경우 Cursor 방식은 사용할 수가 없어서 부득이하게 OffSet 방식으로 바꾸게 되었습니다! (deadline값 중복이 많을 수 있습니다)
또한 목표의 개수는 life_map_id 당 50개로 제한되어 있어 OffSet 방식도 크게 부담이 없지 않을까 하는 생각도 있었습니다 ㅎㅎ

ManHyuk

👍

ManHyuk · 2024-03-27T14:48:25Z

backend/application/api/src/main/kotlin/io/raemian/api/goal/service/GoalQueryService.kt

-        return PaginationResult.from(
-            lifeMap.goals.size,
-            goals.transform {
+        goals.map {


여기 복붙하다가 아래 반환쪽이랑 중복코드 된듯~~

앗!!! ㅋㅋㅋㅋㅋㅋ
감사합니다 👍

ManHyuk · 2024-03-27T14:50:36Z

...age/db-core/src/main/kotlin/io/raemian/storage/db/core/common/pagination/PaginationResult.kt

    val contents: List<T>,
-    val nextCursor: CursorType?,
+    val nextCursor: Long?,


next가 없는 경우는 첫페이지를 뜻하는건가?

만약 첫페이지라면 LONG MAX(MIN) VALUE 을 할당하는건 어떨까?

최대한 null 을 허용하지 않는 방향으로가는게 좋을것같아~~

null 이 들어가는 경우라면 어쩔수없고 ㅎㅎ

현재 pagination 요청의 결과가 마지막 페이지일 경우(isLast = true) nextCursor를 null로 보내고 있는 상황입니다!
만약 null이 아닌 값을 넣는다면 -1? 과 같은 값으로 nextCursor가 없다는 걸 나타낼 수 있을 것 같습니다.
null이나 -1 혹은 다른 값 중에 하나 사용하면 좋을 것 같은데 의견 부탁드립니다~!

binary-ho

LGTM!

@FunctionalInterface

* feat: remove forward header optiion * refactor : birth 관련 유저 정보 업데이트 API에 birth를 nullable로 변경 (#206) (#207) * feat(#209): spring security authorization_request_not_found 수정 (#210) * [🌎 Feature] Goal 댓글 API 개발 (#208) * feat : 댓글 엔티티와 repository 생성 (#196) * feat : Goal 댓글 오름차순 전체 조회 메서드 구현 (#196) * feat : Goal 댓글 작성 삭제 메서드 구현 (#196) * feat : Goal의 마지막 댓글 Read 시간을 저장하는 GoalCommentReadTime 구현 (#196) * feat : Goal의 생성자가 읽지 않은 댓글이 있는지 확인하는 메서드 구현 (#196) * feat : 비동기적으로 마지막 댓글 확인 시간을 update하는 메서드 구현 (#196) * ktlint formatting * refactor : 마지막 댓글 Read 시간을 분리된 테이블이 아닌 Goal이 갖도록 변경 (#196) * feat : 댓글 조회시 요청자가 Goal의 주인이라면 comment read time을 update하는 이벤트 발행 (#196) * test : Goal 필드 추가에 따른 테스트 코드 변경 (#196) * refactor : CoreApiException이 직접 errorMessage를 입력 받을 수 있도록 변경 (#196) * refactor : Comment가 스스로 제한 조건을 검증하도록 변경 (#196) * refactor : CommentService의 필드들에 private 접근 제한자 추가 (#196) * refactor : comment require 조건 정정 (#196) * refactor : 불필요한 트랜잭션 전파 및 Transactional 이벤트 리스너 제거 (#196) * refactor : 불필요한 Goal 조회 쿼리 제거 (#196) * test : Comment 관련 기능 통합 테스트 코드 작성 (#196) * ktlint formatting * refactor : getAllByGoalId 메서드의 이름을 역할에 맞게 findAllByGoalId로 변경 (#196) * feat : 댓글 API 구현 (#196) * chore : 잘못 작성된 메서드 이름 변경 (#196) * refactor : DTO 패키지 변경 (#196) * refactor : comments에서 goal을 꺼낼 때, first 메서드 활용 (#196) * refactor : 불필요한 @async 어노테이션 제거 (#196) * refactor : findAllByGoalId 가독성 개선 (#196) * refactor : Emoji API Restful하게 개선 (#196) (#215) * [🚌 Issue ] Emoji Count 작업 및 Count 관련 로직 개선 (#213) * refactor(#211): count관련 eventListener 단일 component화 * feat(#211): emoji count 테이블 추가 및 count관련 event application lock 적용 * feat(#211): add goal count event에 exclusive runner 적용 * refactor: event 같은 패키지로 이동 * feat: lock 적용 시점 비동기 이벤트 실행 Thread로 변경 * refactor: event 네이밍 컨벤션 적용 * fix: key에 매핑된 lock 대기 thread가 없다면 map에서 제거 * [🚌 Issue] 리팩토링 이슈 템플릿 세팅 (#220) * chore : 리팩토링 이슈 템플릿 세팅 * chore : 기타-이슈 기본 생성 제목에 불필요한 띄어쓰기 제거 * [🛠️ Refactor] Goal Emoji API 응답에 "이모지를 누른 사람 수" 추가 (#221) * refactor : Goal에 반응 조회 API에 반응한 사람 수 값 추가 (#218) * test : Goal 반응 조회 API 반응한 사람 수 값이 정확한지 확인하는 테스트 작성 (#218) * [🛠️ Refactor] IsMyReaction을 확인할 때 Username이 아닌 id로 확인하기 (#219) * refactor : IsMyReaction을 확인할 때 Username이 아닌 id로 확인하도록 변경 (#217) * merge with develop * [🆘 Bug] authorization_request_not_found 오류 (#216) * feat(#209): spring security authorization_request_not_found 수정 * fix: AuthorizationRequestRepository cookie 기반에서 state parameter 기반으로 변경 * fix: AuthorizationRequestRepository remove 메서드에 명시적인 삭제 추가 * fix: AuthorizationRequestRepository oauthRequestStorage 데이터 명시적 삭제 로직 제거 * refact(#171): add emoji count * feat(#171): 피드에 이모지및 내가 누른 이모지 추가 * chore(#171): ktlint * chore(#171): 사용하지 않는 코드 삭제 * chore(#171): ktlint * [🛠️ Refactor] 댓글 응답에 댓글 갯수 추가 (#225) * refactor : Goal에 달린 댓글 응답에 댓글 갯수 추가 (#224) * test : Goal에 달린 댓글 응답이 댓글 갯수를 반환하는 것을 확인하는 테스트 코드 작성 (#224) * ktlint formatting * refact(#171): edit field name * [🛠️ Refactor] RunnerLock 필드 변수명 변경 �+ Security Uri 상수화 + H2 웹 콘솔용 WebSecurityCustomizer 제거 (#226) * refactor: RunnerLock 내부 필드 변수명 변경 * refactor: security permitall uri 상수화 * [🛠️ Refactor] 댓글 조회 API 응답에 "내 지도 확인" key-value 추가 (#230) * refactor : CommentsResponse에 자신의 목표인지 확인하는 isMyGoal 필드 추가 (#229) * test : CommentsResponse를 통해 자신의 목표인지 확인할 수 있는지 검증하는 테스트 코드 작성 (#229) * feat(#232): add title field * [🛠️ Refactor] 프로젝트 구조 변경 (1번, 3번 제안만 변경) (#234) * refactor(#235): pagination 공통화 + cheering squad 공통 pagination 적용 * [🛠️ Refactor] 잘못 작성된 읽지 않은 댓글 조회 쿼리 올바르게 정정 (#237) * refactor : CommentsResponse에 자신의 목표인지 확인하는 isMyGoal 필드 추가 (#229) * test : CommentsResponse를 통해 자신의 목표인지 확인할 수 있는지 검증하는 테스트 코드 작성 (#229) * refactor : exist new comment 쿼리를 id와 시간으로 검색하도록 변경 (#236) * refactor : 쿼리가 Goal Id를 사용하도록 변경 (#236) * test : 본인의 Goal만 조회하는지 검증하는 테스트 추가 작성 (#236) * feat: pagination 공통 response 추가 + cheering squad 조회를 위한 queryRepository 추가 * feat(#235): comment count 엔티티 추가 + comment create/delete event 생성 * feat(#235): 목표 타임라인 페이징 조회 API 작업 * fix: dev 환경 로그인 callback 시 redirect uri 변경 * chore: apply lint * refactor: CheererRepository 미사용 메서드 제거 * refactor: TriFunction @FunctionalInterface 적용 * refactor(#235): PR 코멘트 반영 * [🌎 Feature] 타임 라인용 목표 조회 API 개발 (#240) * refactor(#235): pagination 공통화 + cheering squad 공통 pagination 적용 * feat: pagination 공통 response 추가 + cheering squad 조회를 위한 queryRepository 추가 * feat(#235): comment count 엔티티 추가 + comment create/delete event 생성 * feat(#235): 목표 타임라인 페이징 조회 API 작업 * fix: dev 환경 로그인 callback 시 redirect uri 변경 * chore: apply lint * refactor: CheererRepository 미사용 메서드 제거 * refactor: TriFunction @FunctionalInterface 적용 * refactor(#235): PR 코멘트 반영 * hotfix(#235): 타인 타임라인 조회 API 추가 * chore: apply klint * [🛠️ Refactor] 타임 라인 API 작업 리팩토링 (#241) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * hotfix: dev login redirect url 수정 * hotfix: login redirect url referer 기준 분기 처리 * hotfix: login redirect url referer 기준 분기 처리 조건 추가 * hotfix: login redirect url referer 기준 분기 처리 제거 * [🌎 Feature] 타임 라인용 목표 조회 API 개발 - Cursor Deadline으로 변경 (#242) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * fix: 목표 피드 조회 API deadline 역순으로 정렬 조회 * chore: apply lint * fix: goals 테이블의 index 수정 * refactor: pagination cusorId type Any -> Generic으로 변경 * [🌎 Feature] 타임 라인용 목표 조회 API 개발 - Pagination 방식 offset 기반으로 변경 (#243) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * fix: 목표 피드 조회 API deadline 역순으로 정렬 조회 * chore: apply lint * fix: goals 테이블의 index 수정 * refactor: pagination cusorId type Any -> Generic으로 변경 * fix: 타임 라인 목표 조회 API paging 처리 방식 offset으로 변경 * refactor: cursor 기반 pagination 공통 모듈 cursor type long으로 통일 * refactor: pagination 공통 모듈 cursor와 offset 구분 * refactor: 미사용 코드 및 의미 없는 중복 코드 삭제 * hotfix: global exception config 내 NoSuchElementException 설정 수정 * [🚌 Issue] dev api 로그인 요청 위치에 따라 redirect url 변환 (#245) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * fix: 목표 피드 조회 API deadline 역순으로 정렬 조회 * chore: apply lint * fix: goals 테이블의 index 수정 * refactor: pagination cusorId type Any -> Generic으로 변경 * fix: 타임 라인 목표 조회 API paging 처리 방식 offset으로 변경 * refactor: cursor 기반 pagination 공통 모듈 cursor type long으로 통일 * refactor: pagination 공통 모듈 cursor와 offset 구분 * feat(#244): dev api 로그인 요청 위치에 따라 redirect url 변환 * chore: apply lint * fix: @value(spring.profiles.active) default값 추가 * refactor: 코드 리뷰 반영(#245) * chore: encrypt하던 데이터 삭제 * chore: encrypted test properties 삭제 * chore: referer 저장/조회 logging 추가 * chore: logging 제거 * fix: getReferer 메서드 referer값 null일 경우 Empty String 반환 * [🆘 Bug] 타임라인 관련 버그 수정 (#248) * fix: timeline 조회 시 목표 없으면 early return * feat: 유저정보 기반 타임 라인 조회 API 작업 * fix: 목표 삭제 시 LifeMapCount goalCount 필드 -1 처리 * refactor: timeline 조회 api 내 목표 개수 조회 쿼리 변경 * feat(#171): edit comment count * chore: ktlint * refactor : Comment와 Emoji에 Goal Delete Casacade 속성 추가 (#246) (#249) * refactor : 자신의 Goal에 달린 Comment 삭제를 허용하도록 변경 (#251) (#252) * bug(#254): 타임라인 조회 API 내 코멘트 개수 조회 쿼리 오류 수정 * feat : 이모지 추가 전에 같은 이모지가 존재하는지 확인하는 메서드 작성 (#253) (#256) * refactor : 모든 Emoji 조회 API에서 일관되게 Emoji id 순서로 정렬해 응답하도록 변경 (#257) * refactor : 불필요한 정렬 제거 - MySQL Clustered Index로 인해 이미 pk 정렬 되어 있다 (#257) * fix : 피드 조회 (#259) * feat(#171): 목표 생성시 기본 댓글 개수 추가 * chore(#171): ktlint * feat(#171): 기본 카운트 추가 * chore: ktlint * refact(#171): 변수 삭제 --------- Co-authored-by: ManHyuk <[email protected]> Co-authored-by: binary_ho <[email protected]>

egg528 added 17 commits March 24, 2024 11:34

delete: 유저 정보를 활용한 타임라인 조회 API 삭제

b99367c

refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일

4249f81

refactor: 메서드 로직에 따라 get, find 명칭 구분

854f4a2

refactor: lamda body 내 생략 가능한 parameter 생략

23aa5f9

refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCount…

1c7d46f

…Map)

refactor: associateWith 메서드 활용

74b372b

refactor: PaginationResult total값 Long으로 다시 변경

54c8b81

Merge branch 'develop' of github.com:depromeet/amazing3-be into feature/

ef9f69d

#235

fix: 목표 피드 조회 API deadline 역순으로 정렬 조회

30b557b

Merge branch 'develop' of github.com:depromeet/amazing3-be into feature/

89264c7

#235

chore: apply lint

5425558

fix: goals 테이블의 index 수정

48e2709

refactor: pagination cusorId type Any -> Generic으로 변경

a9135a1

Merge branch 'develop' of github.com:depromeet/amazing3-be into feature/

63bb7e9

#235

fix: 타임 라인 목표 조회 API paging 처리 방식 offset으로 변경

4c110bc

refactor: cursor 기반 pagination 공통 모듈 cursor type long으로 통일

bb151c4

refactor: pagination 공통 모듈 cursor와 offset 구분

0a00a11

egg528 self-assigned this Mar 27, 2024

egg528 requested review from ManHyuk and binary-ho as code owners March 27, 2024 03:28

ManHyuk approved these changes Mar 27, 2024

View reviewed changes

binary-ho approved these changes Mar 27, 2024

View reviewed changes

refactor: 미사용 코드 및 의미 없는 중복 코드 삭제

4f8d56b

egg528 merged commit ca23a69 into develop Mar 28, 2024
1 check passed

egg528 deleted the feature/#235 branch March 28, 2024 13:08

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[🌎 Feature] 타임 라인용 목표 조회 API 개발 - Pagination 방식 offset 기반으로 변경 #243

[🌎 Feature] 타임 라인용 목표 조회 API 개발 - Pagination 방식 offset 기반으로 변경 #243

egg528 commented Mar 27, 2024

binary-ho commented Mar 27, 2024

egg528 commented Mar 27, 2024 •

edited

Loading

ManHyuk left a comment

ManHyuk Mar 27, 2024

egg528 Mar 27, 2024

ManHyuk Mar 27, 2024

egg528 Mar 27, 2024

binary-ho left a comment

[🌎 Feature] 타임 라인용 목표 조회 API 개발 - Pagination 방식 offset 기반으로 변경 #243

[🌎 Feature] 타임 라인용 목표 조회 API 개발 - Pagination 방식 offset 기반으로 변경 #243

Conversation

egg528 commented Mar 27, 2024

📒 Issue

🎯 어떤 작업을 했는지

🌍 영향범위 (모듈)

binary-ho commented Mar 27, 2024

egg528 commented Mar 27, 2024 • edited Loading

ManHyuk left a comment

Choose a reason for hiding this comment

ManHyuk Mar 27, 2024

Choose a reason for hiding this comment

egg528 Mar 27, 2024

Choose a reason for hiding this comment

ManHyuk Mar 27, 2024

Choose a reason for hiding this comment

egg528 Mar 27, 2024

Choose a reason for hiding this comment

binary-ho left a comment

Choose a reason for hiding this comment

egg528 commented Mar 27, 2024 •

edited

Loading