Taewan Kim

Results 10 comments of Taewan Kim

![Screenshot from 2021-09-27 14-22-39](https://user-images.githubusercontent.com/49307266/134849651-d8266937-7572-442d-8241-2005d031140c.png)

DB replication으로 구조를 수정하게 되었을 때, Master -> Slave 사이에 data gap 이 발생한다. 기존에도 검증+CUD작업이 별도의 Transaction으로 이루어졌었는데, 더욱 더 수정이 필요하다.

오호, 기존의 구현을 유지한다는 생각을 했더니, 지워버린다는 생각은 못했네요! 그렇다면 아래의 부분에 어떤 정보들이 들어가는 게 좋을까요? 남겨진 Job의 갯수 정도? ![Screenshot from 2021-09-10 15-28-57](https://user-images.githubusercontent.com/49307266/132809829-eac6041e-2b37-4209-ac47-738c7a191551.png) Job의 상태마다 보여지는 정보가 달라지겠네요!...

요 부분 작업 진행할 때, Job 등록 시 server가 켜져 있는지 검증하는 로직을 구현하고 진행했으면 좋겠음! 밀접하게 연관되어 있다 생각하고, 이 기능을 테스트하기 위해서는 위 검증 로직이 필요하다는 생각입니다!

@sjpark-dev 제가 이해하기로는 Worker에서 `@PutMapping("gpus/{serverId}/status")` 요청이 왔을 때, 둘다 변경되는 걸로 알고 있어요! user가 등록하면 isOn 은 False로 등록되고요! https://github.com/woowacourse-teams/2021-gpu-is-mine/blob/db1f6f5d8e5cc64accdbcc3b910f9caf1da8441d/back-end/src/main/java/mine/is/gpu/worker/application/WorkerService.java#L90-L95

아아 지금의 구조가 아니라 앞으로의 구조가 어떻게 바뀔 것인지에 대한 이야기 였군요! isOn이라는 column을 삭제하고, 사용할 때 lastResponse 값을 비교하는 로직으로 수정될 것이라 생각해요!

```java @Transactional public Long save(Long memberId, JobRequest jobRequest) { //패키지간의 의존 관계를 끊어야 한다. // A라는 패키지의 로직을 수정했을 때, B를 수정해야됨, A를 수정해야 됨, B를 수정해야 됨. // job을...

`GET /api/labs/1/jobs/1 HTTP/1.1` ``` Hibernate: select administra0_.id as col_0_0_ from administrator administra0_ where administra0_.email=? limit ? Hibernate: select member0_.id as id1_5_, member0_.created_at as created_2_5_, member0_.updated_at as updated_3_5_, member0_.email as email4_5_,...

field가 많아짐에 따라, getter/setter, null 체크 로직이 많아지기 때문에, 도메인 로직을 확인하기 어려움 존재 고민하는 문제 1. Controller에서 Member객체를 굳이 받아야 할까?, memberId만 받아도 충분하지 않을까? -> 컨트롤러에서 member의 종류에...

모든 Job관련 서비스가 `GpuServer.isOn=false` 인 경우 정상적으로 등록할 수 있게끔 테스트 코드가 짜여 있었음. RestAssured에서는 우선 Worker측에서 Gpu상대 변경을 진행한 후 테스트를 진행하도록 수정 JobStatus.Waiting이 기본 상태가 아니라, 예약을 하는...