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

nplusone 적용 및 코드 리팩토링 #112

Closed
wants to merge 4 commits into from
Closed

Conversation

shinkeonkim
Copy link
Member

작업 내용

기타 사항

  • 현재 nplusone 경고를 로그에만 띄우려고 했는데, 아예 서버 오류로 올려버리고 있어요.. ㅠ.
    방법을 찾긴 해야 하는데, 일단 눈에 띄는 것들은 처리했습니다. 그리고 다 whitelist로 올려놨어요.. ㅋㅋ

@shinkeonkim shinkeonkim added 개선 기존의 기능을 개선합니다. API 제작 안정화 and removed API 제작 labels Mar 19, 2022
@shinkeonkim shinkeonkim self-assigned this Mar 19, 2022
@shinkeonkim shinkeonkim linked an issue Mar 19, 2022 that may be closed by this pull request
Copy link
Member

@Jin409 Jin409 left a comment

Choose a reason for hiding this comment

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

현재 nplusone 경고를 로그에만 띄우려고 했는데, 아예 서버 오류로 올려버리고 있다는 말을 이해하지 못했는데 ,, ,,, 이에 대해서 알 수 있을까요? + 이전에 n+1 이슈에 대해서 지현님이 처리하셨던걸로 아는데 아예 그 코드를 없애고 패키지(?)로 처리하신건지도 궁금합니다!

@shinkeonkim
Copy link
Member Author

nplusone은 n+1이슈를 해결해주는 아이가 아니라, detect 해주는 아이입니다 ㅎㅎ
그래서 이제부터 발생하는 n+1이 발생하지 않게 코드를 작성해야 합니다.

그래서 현재 저희의 개발에 일종의 족쇄를 채운 느낌이죠

Copy link
Member

@Jihyun-Choi Jihyun-Choi left a comment

Choose a reason for hiding this comment

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

확인했습니다! 👍

@Jihyun-Choi
Copy link
Member

지난번에 n+1과 관련해 개발했을때 실제로 쿼리가 어떻게 찍히는지 궁금했는데, 혹시 쿼리를 어떻게 확인하는지 알 수 있을까요? Django Debug Toolbar 등 여러 방법이 있어서 어떻게 해야될지 궁금했습니다.

@shinkeonkim
Copy link
Member Author

지금 당장 쉬운 방법은 없을것같아요
디비 쿼리가 어떻게 나가고있는지에대한 로그를 볼 수 있게 추가해놔야할것같아요

@shinkeonkim shinkeonkim closed this Nov 3, 2022
@shinkeonkim shinkeonkim deleted the nplusone_library branch November 3, 2022 03:10
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.

nplusone 추가하기
3 participants