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

[BE] 아티클 조회수 기능을 구현한다. #1070

Open
wants to merge 6 commits into
base: dev-be
Choose a base branch
from

Conversation

tsulocalize
Copy link
Contributor

@tsulocalize tsulocalize commented Dec 18, 2024

❗ Issue

✨ 구현한 기능

  • 아티클 조회 시 조회수 증가
  • 아티클 응답에 조회수 추가

📢 논의하고 싶은 내용

응답 필수!! ⭐ ⭐

조회수 기능을 위해 기존에 있던 Cache 기능을 잠시 지우게 됐는데,
캐싱이 AOP라서 클래스 내 자가 호출은 적용이 안된다고 하네요.
캐싱을 유지하고 싶다면 제가 생각하는 대안은 2가지 입니다.

  1. ArticleService 외 별도 서비스 layer를 두어 AOP가 정상 동작하도록 한다.
  2. 조회수 증가 로직을 비동기로 처리한다.

여러분의 생각은 어떤가요?

🎸 기타

@tsulocalize tsulocalize self-assigned this Dec 18, 2024
@tsulocalize tsulocalize changed the title Feat/980 article views [BE] 아티클 조회수 기능을 구현한다. Dec 18, 2024
# Conflicts:
#	backend/bang-ggood/src/main/java/com/bang_ggood/article/service/ArticleService.java
Copy link
Contributor

@tkdgur0906 tkdgur0906 left a comment

Choose a reason for hiding this comment

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

조회수의 정확도가 크게 중요하지 않다고 생각해서 비동기로 처리해도 괜찮을 것 같아요!

@shin-jisong
Copy link
Contributor

저 또한 조회수 기능은 정확도가 중요한 기능도 아니며, 이것을 보장하기 위해서 다른 성능을 할애할 부분도 아니라 생각합니다!
하지만 layer를 두는 것은 조회수 기능이 필요한 도메인마다 파생되는 layer가 너무 많아서 관리의 어려움이 발생할 것 같아요
비동기적으로 처리해도 된다는 생각입니다 👍

Copy link
Contributor

@JINU-CHANG JINU-CHANG left a comment

Choose a reason for hiding this comment

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

저도 비동기로 처리해도 괜찮을 것 같습니다!

@@ -159,6 +159,7 @@ CREATE TABLE article
keyword VARCHAR(255),
summary VARCHAR(255),
thumbnail VARCHAR(2500),
view_count BIGINT default 0,
Copy link
Contributor

Choose a reason for hiding this comment

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

머지되면 스키마 변경사항 prod db에도 반영해주세요~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants