Skip to content

Conversation

@yerim123456
Copy link
Contributor

@yerim123456 yerim123456 commented Mar 7, 2025

🎟️ 관련 Jira 이슈

Resolves #PRBE-35

👩‍💻 구현 내용

  • 게시물 및 분석 관련 엔티티 추가
  • 게시물 api 호출 및 저장
  • 분석 api 연결 및 저장
  • 로그인 api에 사용자 정보 응답 추가

💬 코멘트

  1. 재로그인 시, 현재는 추가된 게시물만 처리하고 있습니다. 하지만 게시물이 수정되거나 삭제되는 경우도 추후에 반영해야 합니다.
  2. dangerScore 부분 업로드하고 바로 저장하는 구조 > DB 조회가 너무 많음 > 수정 필요
  3. 가장 최근의 createTime 을 기준으로 새로운 게시물을 가져오는 게 최선일지 고민 필요
  4. 현재는 100개까지의 게시물만 가져오고 있습니다. paging 처리 가능하도록 진행할 예정입니다. 다만 paging 정보가 온전치 않게 응답하고 있는 것 같아 이 부분 추가 확인할 예정입니다.

💭 고려한 점

로그인 시 바로 게시물 분석을 하고 응답을 내보내면 사용자의 대기 시간이 너무 길어지므로 비동기로 로직을 처리해 응답을 먼저 내보낼 수 있도록 설정했습니다.
또한 계속 api 를 호출하여 가져오는 게 아니라 비용을 고려하여 로그인 시 대량의 데이터를 가져오고 재 로그인 시에는 그 후 추가된 데이터만을 가져오는 방식을 활용하여 보다 효율적인 외부 API 소통이 가능하도록 설정했습니다.
추후에는 스케쥴러 설정을 통해 사용자가 많이 사용하지 않는 시간대에 게시물에 대한 전반적인 업데이트를 통해 삭제, 수정 반영을 진행할 예정입니다.

yerim1ee added 20 commits March 7, 2025 15:13
@yerim123456 yerim123456 self-assigned this Mar 7, 2025
@github-actions
Copy link

github-actions bot commented Mar 7, 2025

📊 Jacoco Test Coverage

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

@yerim123456 yerim123456 merged commit 66ba180 into main Mar 7, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants