feat(server) : 정산금액을 계산해서 갱신하는 기능을 스케줄러에 등록한다. (#439) #440
Conversation
| .calculateExchangeAmountFromDonation(exchange.getMember(), exchange.getExchangeOn()); | ||
| exchange.registerExchangeAmount(exchangeAmount); | ||
| }); | ||
| } |
There was a problem hiding this comment.
@Be-poz 그런데 회원마다 업데이트 되는 금액이 다른데 벌크 업데이트가 가능한거야?
전에 근로 때도 내가 정책 정할 일이 있었는데, 잘 몰라서 이거 bulk update 하는게 어떻냐구 했다가.... 잘 안돼서 작업자가 고생했거든
|
벌크성 쿼리로 업데이트 하는 것 찾아봤는데 exchange id별로 exchange amount를 다르게 업데이트 할 경우에는 적용이 불가능 한 것 같아! 그래서 우선 계산 쿼리를 한번 날리고, 이후에 각각 exchange를 update 하는 식으로 변경했어! JPA가 익숙치 않아서 이게 맞는지 더 좋은 방법이 있는지 잘 모르겠어 ㅜ |
There was a problem hiding this comment.
크게 코멘트 달 부분은 없는 것 같아
스케쥴러 공부 간단하게 했는데 충분히 어떤 작업을 했는지 알 수 있는 것 같고,
벌크 업데이트 부분은 가능한건지 잘 모르겠어, 일괄적으로 동일한 금액을 업데이트해주는거면 모르겠는데, 각 exchange 마다 다른 금액이 update 되는 거잖아 ? 이런 경우도 벌크 업데이트가 가능한건가? (대상 지정은 Where in 으로 두면 될 것 같은데....)
이건 @Be-poz 파즈한테 질문하고싶은건데!!
혹시 QueryDsl로 동적으로(?) 벌크 업데이트가 가능한 방법이 있을까?
코멘트에 달아둔 수리 의견에 대해서는
MemberService의 exchange()를 requestExchange로 변경하면 어떤지,,
이 부분은 사실 난 exchange()도 충분하다고 생각해 exchange를 동사로 보냐 명사로 보냐의 차이라고 생각하는데, 우리 서비스에서 exchange를 하나의 객체이자 Entity로 가지고 있어서 수리의 말도 일리가 있다고 생각하는데, 정산한다는 의미로도 충분히 의미가 전달된다고 생각해
| import java.util.List; | ||
|
|
||
| public interface ExchangeQueryRepository { | ||
| List<ExchangeAmountDto> calculateExchangeAmountFromDonation(YearMonth exchangeOn); |
There was a problem hiding this comment.
| List<ExchangeAmountDto> calculateExchangeAmountFromDonation(YearMonth exchangeOn); | |
| List<ExchangeAmountDto> calculateExchangeAmountFromDonation(YearMonth exchangeOn); |
다른 클래스들처럼 첫 줄은 띄우면 좋을 것 같아! 😃
| .calculateExchangeAmountFromDonation(exchange.getMember(), exchange.getExchangeOn()); | ||
| exchange.registerExchangeAmount(exchangeAmount); | ||
| }); | ||
| } |
There was a problem hiding this comment.
@Be-poz 그런데 회원마다 업데이트 되는 금액이 다른데 벌크 업데이트가 가능한거야?
전에 근로 때도 내가 정책 정할 일이 있었는데, 잘 몰라서 이거 bulk update 하는게 어떻냐구 했다가.... 잘 안돼서 작업자가 고생했거든
이게 지금 문제! 수리 말대로 update를 한 번에 동일한 값을 조정하는데 써야될 것 같아서... 지금의 경우는 회원마다 각기 다른 값 들을 넣어야 해서 문제가 되는 것 같아! insert table(...) values( ...) ; 이렇게 update도 대량으로 할 수 있는 방법이 있나 한 번 보구있엇어 일단 못찾앗으 ㅜ |
Joyykim
changed the title
|
제안
close #440