Skip to content

[조회 성능 개선하기 포츈(정윤성) 미션 제출합니다. #48

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

Merged
merged 1 commit into from
Oct 27, 2021

Conversation

unluckyjung
Copy link

안녕하세요 바다! 전에 협업 미션 이후로 오랜만에 뵙네요.

미션 제출이 많이 늦었습니다.
미션 진행 과정은 이곳에서 편하게 보실 수 있습니다.

@unluckyjung unluckyjung requested a review from xrabcde October 24, 2021 13:07
@unluckyjung unluckyjung self-assigned this Oct 24, 2021
@unluckyjung unluckyjung changed the base branch from main to unluckyjung October 24, 2021 13:07
Copy link
Member

@xrabcde xrabcde left a comment

Choose a reason for hiding this comment

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

안녕하세요 포츈! 바다입니다 🌊
오랜만이네요👏 미션 하시느라 고생 많으셨습니다 ㅎㅎ
미션을 진행하신 과정을 구체적으로 정리해주셔서 리뷰가 수월했던 것 같아요~
저와 비슷한 부분에서 고민을 하신 게 느껴지네요 ㅋㅋㅋ

쭉 읽어보니 미션을 진행하시면서 충분히 고민도 많이 하시고
다양한 시도를 하며 개선하신 게 느껴져서 바로머지하겠습니다~
이제 얼마 남지 않은 기간 지치지 말고 끝까지 화이팅해봅시다!!! 💪💪

Comment on lines +85 to +93
- 부서관리자 테이블의
- 사원번호를 통해
- 사원 테이블 -> 이름
- 급여 테이블 -> 연봉
- 직급 테이블 -> 직급명
- 부서번호를 통해
- 부서테이블 -> 부서번호

와 같이 JOIN 시키면 원하는 정보를 얻어올 수 있을것으로 파악되었습니다.
Copy link
Member

Choose a reason for hiding this comment

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

이야~ 요구사항을 파악하는 과정을 이렇게 자세히 적어주셔서 읽기 편하고 너무 좋네요 💯💯


![a](https://user-images.githubusercontent.com/43930419/138583974-f52fc4b9-8321-4711-8b0b-0a50a2f816ad.png)

- 현재 Full Table 스캔이 일어나는 `사원출입기록.사원번호` 를 인덱스를 걸어보기로 했습니다.
Copy link
Member

Choose a reason for hiding this comment

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

👍

![image](https://user-images.githubusercontent.com/43930419/138586617-00578af0-e469-448c-ada1-713565591e74.png)


- 큰차이가 보이진 않네요..
Copy link
Member

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋ covid.programmer_id 에 인덱스를 걸어봐도 괜찮겠네요
일단 요구사항은 만족하셨으니 패스하겠슴니다 👏

Comment on lines +278 to +279
- 기존 `프로그래밍이 취미인 학생 / 혹은 주니어(0-2)` 이었던것을
- `프로그래밍이 취미인 / 학생 혹은 주니어(0-2년) 으로 해석` 으로 재해석하면서 쿼리를 변경했습니다.
Copy link
Member

Choose a reason for hiding this comment

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

저도 이 부분 요구사항이 애매해서 헷갈렸는데
포츈이 이해한 방식을 구체적으로 적어주시니까 좋네요! 😊

@xrabcde xrabcde merged commit 78dedba into woowacourse:unluckyjung Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants