Feature/#261 feature/fix_sort->dev: 주유기록 정렬 수정 #263

gyuturn · 2023-02-21T09:00:43Z

🌿 이슈 번호 : #261

📝 PR 유형

새로운 feature 추가
feature 수정 (기능 업데이트, 버그 수정 등)
관련 문서 추가
관련 문서 수정

🧑‍💻 작업 내용

주유소 정렬시 같은날짜인경우 최신정렬이 되지 않는 로직을 수정했습니다.

❗️ 리뷰어는 여기에 집중해주세요!

Localdate기준으로 정렬하면 위의 오류가 발생합니다.
그렇기에 auto increment인 Pk를 사용해서 정렬을 하였습니다.
오히려 더 좋은 성능을 추구할 수 있어 좋은 방법이 된것 같습니다.

🏃‍♂️ 다음 작업 계획

노션 정리 및 개발 작업 문서화

기타 사항

언제든 의논 후 PR template 양식을 수정하고 싶어요.

ajongs · 2023-02-21T09:02:54Z

localdatetime으로 바꾸는 방법보다 효과적으로 보이네용~

ajongs

굳

ajongs · 2023-02-21T09:04:47Z

backend/src/main/java/com/kaspi/backend/dao/UserGasRecordDao.java

+            "where user_no = :userNo \n" +
+            "order by ugr.user_gas_record_no DESC")
    List<UserGasRecord> findGasRecordListByUserId(@Param("userNo") Long userNo);


같이 해결했으니 가볍게 리뷰합니다~

gyuturn added 2 commits February 21, 2023 15:29

feat: sql trace 추적

24cef93

#261 fix: 유저 주유기록 조회 정렬 수정

1891c91

gyuturn added the backend 백엔드 관련 이슈 입니다. label Feb 21, 2023

gyuturn requested a review from ajongs February 21, 2023 09:00

Merge branch 'dev' into feature/#261_performance_improvement

c12641f

gyuturn mentioned this pull request Feb 21, 2023

오류 수정 및 성능개선 #261

Closed

ajongs merged commit 3e6f77f into softeerbootcamp:dev Feb 21, 2023

ajongs reviewed Feb 21, 2023

View reviewed changes

gyuturn self-assigned this Feb 24, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/#261 feature/fix_sort->dev: 주유기록 정렬 수정 #263

Feature/#261 feature/fix_sort->dev: 주유기록 정렬 수정 #263

gyuturn commented Feb 21, 2023 •

edited

Loading

ajongs commented Feb 21, 2023

ajongs left a comment

ajongs Feb 21, 2023

Feature/#261 feature/fix_sort->dev: 주유기록 정렬 수정 #263

Feature/#261 feature/fix_sort->dev: 주유기록 정렬 수정 #263

Conversation

gyuturn commented Feb 21, 2023 • edited Loading

🌿 이슈 번호 : #261

📝 PR 유형

🧑‍💻 작업 내용

❗️ 리뷰어는 여기에 집중해주세요!

🏃‍♂️ 다음 작업 계획

기타 사항

ajongs commented Feb 21, 2023

ajongs left a comment

Choose a reason for hiding this comment

ajongs Feb 21, 2023

Choose a reason for hiding this comment

gyuturn commented Feb 21, 2023 •

edited

Loading