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

포스트에 좋아요 컬럼추가 #99

Open
J-Jaeh opened this issue Apr 6, 2023 · 5 comments
Open

포스트에 좋아요 컬럼추가 #99

J-Jaeh opened this issue Apr 6, 2023 · 5 comments
Assignees
Labels
기능추가 새로운 기능 추가하고싶을때 성능개선 성능개선하고싶다이 이거야~!

Comments

@J-Jaeh
Copy link
Member

J-Jaeh commented Apr 6, 2023

게시글에 좋아요 갯수를 담는 컬럼을 추가하고

인덱스를 설정해서

좋아요 기준으로상위 3개만 가져오면 좀 더 효율적인 로직이 될 것 같다는 생각

@J-Jaeh J-Jaeh added 성능개선 성능개선하고싶다이 이거야~! 기능추가 새로운 기능 추가하고싶을때 labels Apr 6, 2023
@J-Jaeh J-Jaeh added this to the 좋아요 개선! milestone Apr 6, 2023
@J-Jaeh J-Jaeh self-assigned this Apr 6, 2023
@J-Jaeh J-Jaeh moved this to 해야될일 in 프로젝트관리 Apr 6, 2023
@J-Jaeh J-Jaeh changed the title 포스트에 인덱스 컬럼추가하기 포스트에 좋아요 컬럼추가 Apr 6, 2023
@J-Jaeh
Copy link
Member Author

J-Jaeh commented Apr 6, 2023

계획

  1. 현재 조회마다(GET) 게시글레포와 좋아요레포 둘 다 뒤적거리면서 조회

  2. 앞으로 개선방향은 좋아요 누를때 좋아요 레포 뒤적거려서 좋아요 갯수를 게시글에다가 업데이트 시켜줄 예정
    이러면 조회할때는 게시글레포만 뒤적거림..!

  3. 여기에 캐싱까지 적용되면 더 좋아질 듯..!

@allnight5
Copy link
Collaborator

allnight5 commented Apr 7, 2023

누를때마다 바뀌는걸 볼 수 있어서 좋겠지만 비효율 적일꺼에여
좋아요, 취소, 좋아요, 취소 하면서 계속 바꾼다면 서버에 무리가 갈꺼에여

스케줄러에 넣어서 일정 시간별로 업데이트 된다고
적어두고 자동으로 업데이트 해서 넣어줘여 누를때마다 그러면
업데이트 하면서 조회도 같이 해줘서 캐싱까지 맞춰둔다면
스케줄러 단위로 다 업데이트 될거에여
이제 문제는 좋아요 많아 졌을때 처리 속도가 늦어질 수 있다는 건데
이것을 최적화 하는 문제는 두개다 있겠지요

@wonizizi89
Copy link
Collaborator

두분 다 좋은 의견 갔습니다. 조회마다 두 레포지토리를 뒤적거리며 조회한다는 것을 문제 인식이 훌륭합니다. 효근님의 조언도 좋습니다. ! ㅎㅎ꼭 성공하시길

@J-Jaeh
Copy link
Member Author

J-Jaeh commented Apr 10, 2023

@allnight5 레디스 캐싱까지...해보는걸로....하겠습니다

@smalldevK
Copy link
Collaborator

너무 좋습니다!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능추가 새로운 기능 추가하고싶을때 성능개선 성능개선하고싶다이 이거야~!
Projects
Status: 해야될일
Development

No branches or pull requests

4 participants