기능 개발 공유 사항

[jongmee]

Description ✏️

기능을 개발하며 공유할 사항, 변경할 사항, 부탁할 사항이 생기면 코멘트로 기록한다.

To-Do🐢

• swagger 설명 추가하기
• '@transactional' 추가하기
• api 별 권한 설정 논의 #16
• mongo DB로 리팩토링
• spring security 권한 설정 해주기

Need to improve 🥕

[nzeong]

swagger 에서는
@RestController("/api/applicants") 가 인식이 안되는 것 같습니다!
url 작성 시 @RequestMapping(value = "/api/applicants") 이런 식으로 작성해주어야 합니당

[nzeong]

[논의사항]

지원자 정보 입력 시 면접관 정보를 받는데, 예전에 기획&디자인에 문의해본 결과 이게 실제 목록에서 면접관 선택하는게 아니라 단순 이름 text만을 받는 거여서 우리 db에서 Interviewer 테이블을 생성할 수 없음.

아마 이런 식으로 기존 Interviewer 테이블&연관관계 수정 및 삭제하고 새로 테이블 생성해야 할 것 같음.

일단 아직까지 내가 구현한 기능 내에서는 상관없긴한데.. 혹시 많이 수정을 해야 하는지..? 의견부탁드립니당
그리고 이대로 한다면 면접 준비완료 버튼 누르는 것도 한 명만 누르면 바로 넘어간다든가(면접관이 누구이고 몇명인지 알 수 없으니).. 수정요청해야될 것 같긴 하다

[nzeong]

#1 (comment)

마찬가지로 Evaluation도 fk 연관관계 Member로 바꾸는게 어떤지?! 작성자 이름 조회시에 불필요한 join만 늘어나고 굳이 Interviewer와 연결할 필요 없어보인다

[jongmee]

근데 만약에 subcollabrator가 아닌 사람 이름 입력했을 때 검증할 방법이 없으니까 그냥 이름 입력했을 때 해당 이름의 subcollaborator가 있으면 interviewer 생성하면 되는 것 아녀?? 왜 interviewer에서 member가 필요없게되는거지??

[jongmee]

기존 interviewer는 유지하고 evaluation이나 one-liner 같은 것들 fk를 member로 바꾸는 건 어때?? interviewer에 연결되어 있던 것들!

[nzeong]

기존 interviewer는 유지하고 evaluation이나 one-liner 같은 것들 fk를 member로 바꾸는 건 어때?? interviewer에 연결되어 있던 것들!

일단 이것들은 그렇게 하고 있음!

[nzeong]

근데 만약에 subcollabrator가 아닌 사람 이름 입력했을 때 검증할 방법이 없으니까 그냥 이름 입력했을 때 해당 이름의 subcollaborator가 있으면 interviewer 생성하면 되는 것 아녀?? 왜 interviewer에서 member가 필요없게되는거지??

그 면접관 이름을 입력하는게 subcollaborator 인지 아닌지 검증할 필요 없고 그냥 누구누구 들어가는지 알려주는 용도로 쓰는거여서 기존 evaluation이나 one-liner 같은 것들 fk를 member로 바꾸게 되면 굳이 member fk가 쓰임이 없을거라 생각함..! 일단 나는 안쓸 것 같아서 너는 어떤지 궁금했음! 만약 member fk가 너 로직에서 필요할 것 같다면 기존 interviewer 테이블에 name만 추가하는 식으로 해도 되긴 함!! 근데 이 경우 member fk가 null인 경우(갓챠를 사용하지 않을 면접관 이름을 입력하는 경우, 귀찮아서 이름을 입력 안하거나 별명 등으로 입력할 경우..등?) 도 좀 있을 것이라 예상됨.

[jongmee]

1. 아마 interviewer fk 를 다 member로 바꾸면 내 코드 거의 안 돌아갈텐데 담번 너꺼 머지하고 나서 내가 고치면 될듯??

2. 면접관 이름을 검증해야 될 거 같은게 페이지별로 권한이 있어야 하는데 "갓챠를 사용하지 않을 면접관 이름을 입력하는 경우, 귀찮아서 이름을 입력 안하거나 별명 등으로 입력할 경우" 이거는 권한 처리를 못하지 않을까?? 그러면 "준비 완료된 면접관" 같은것도 못 세서 기능 구현이 안 됨.. 로그인한 유저 계정으로 판단할텐데

[nzeong]

1. 아마 interviewer fk 를 다 member로 바꾸면 내 코드 거의 안 돌아갈텐데 담번 너꺼 머지하고 나서 내가 고치면 될듯??

굳굳

2. 면접관 이름을 검증해야 될 거 같은게 페이지별로 권한이 있어야 하는데 "갓챠를 사용하지 않을 면접관 이름을 입력하는 경우, 귀찮아서 이름을 입력 안하거나 별명 등으로 입력할 경우" 이거는 권한 처리를 못하지 않을까?? 그러면 "준비 완료된 면접관" 같은것도 못 세서 기능 구현이 안 됨.. 로그인한 유저 계정으로 판단할텐데

마자마자.. 그래서 기획&디자인 분들께 함 물어보는게 좋을듯..!

[nzeong]

[몽고 DB 관련]
https://www.erdcloud.com/d/ya8AMkc65FJrFyGHe
일단 몽고 DB 공부하면서 우리 ERD mongoDB 구조로 대충 짜봤습니다! 확실히 간단해지긴 하고 불필요한 조인 사라지고 테이블이 단순화되기는 하는데.. 문제는 이미 쿼리 dsl 쓰면서 코드 자체를 mySql 기준으로 짜두어서 리팩토링에 시간이 걸릴 것으로 보임. 이미 열심히 queryDSL까지 써서 코드 작성해두기도 했구 🤣🤣 그치만 할 의향은 없는 것은 아님! 다만 domain, repository, service까지 바꿔주어야 해서(mongoDB 코드는 간단) 배포 포함 시간 고려했을 때 좀 빠듯하지 않을까.. 싶은...

// applicant_id가 3인 모든 interviewer 수 조회
public long countInterviewersByApplicantId(String applicantId) {
    Query query = new Query(Criteria.where("applicant_id").is(applicantId));
    return mongoTemplate.count(query, Interviewer.class);
}

// applicant_id가 3이고 prepared tag가 Y인 interviewer 수 조회
public long countPreparedInterviewersByApplicantId(String applicantId) {
    Query query = new Query(
        Criteria.where("applicant_id").is(applicantId)
            .and("prepared_tag").is("Y")
    );
    return mongoTemplate.count(query, Interviewer.class);
}

잦은 변경이 있고 속도가 중요한 데이터의 경우(ex. 실시간) 몽고 db를 쓰는게 좋을 듯. 그래서 테이블 분리해서 실시간으로 데이터의 내용을 변경해주어야하는 들어갈 것 같은 테이블(interviewer)은 몽고 db에 따로 빼서 사용하는게 어떨까.. 싶습니다.

그리고 지원자 포폴/지원서 파일을 몽고 db에 저장한다하면 url이 아니라(url 저장할거면 기존의 mySql 대신 굳이 쓸 이유가 없음) 파일 자체를 저장하게 되는데 파일 내용을 검색해서 쓸 것도 아니고.. 몽고 db를 쓸 이유가 굳이 없는 것 같아서 그냥 mySql + s3 로 대체 하기로 했슴니다.

[nzeong]

[배포 일정 관련]
월요일이나 화요일까지 대략적으로 개발 마무리하고 (mongoDB포함 DB 구조 확정하구)
배포하는거 어떤가요?(-> 그렇다면 Script 작성 필요) 이후에 실시간 기능 구현?? 이게 얼마나 걸릴지 모르겠네요 🤣

실시간 기능까지 구현하고 배포하면 좋긴한데 프론트 api 연동 시간상 안될 것 같기두..?

[nzeong]

[erd 관련]

Prepared~테이블 제외하고 이렇게 Interviewer 테이블랑 합쳐서 mongoDB로 빼는거 어떤가요?

공통질문도 순서랑 중요도 계산같은거 해야되는데 IndividualQuestion 테이블이랑 CommonQuestion이랑 합쳐버리는 거 어떤가요? 만약 중요도도 실시간으로 구현할 경우 얘도 mongoDB로 빼기..?