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.

Sign up for GitHub

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

Jump to bottom

[#90] mysql2 PreparedStatment 적용 #91

Open
wants to merge 1 commit into
base: feature/86-test-api-with-code-refectoring
Choose a base branch
from
Open

[#90] mysql2 PreparedStatment 적용 #91

wants to merge 1 commit into from

Conversation

yanggwangseong
Copy link
Collaborator

✨ 구현 기능 명세

  • 참여자 리스트 API를 RPS 1070 → 1300으로 개선 되었습니다!
  • 멤버 조회 API를 RPS 1600 → 2200으로 개선 되었습니다!

🎁 주목할 점

  • nestjs/typeorm 부터 TypeORM, mysql2를 분석하여 어떻게 동작 하는지와 mysql2 핵심 기능들을 어떻게 적용하고 사용 할 수 있는지에 대해서 코드를 통해서 공부 했습니다.
  • 해당 라이브러리들의 호출 순서와 호출되는 클래스 메서드 옵션들과 관계되는 외부 라이브러리들을 이해하고 파악했습니다.

😭 어려웠던 점

🌄 스크린샷

@yanggwangseong yanggwangseong added Assignee 양광성 feat 구현·개선 사항에 관련된 내용입니다. labels Feb 5, 2025
@yanggwangseong yanggwangseong self-assigned this Feb 5, 2025
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Feb 5, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
3.8% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@f-lab-jayce f-lab-jayce Awaiting requested review from f-lab-jayce

Assignees

@yanggwangseong yanggwangseong

Labels
Assignee 양광성 feat 구현·개선 사항에 관련된 내용입니다.
Projects
None yet
Milestone
추가개선사항과 2차목표 설정
Development

Successfully merging this pull request may close these issues.
1 participant
@yanggwangseong