Redisson으로 분산락을 사용할 때, lockName 할당 관련하여 질문이 있습니다.

Question

안녕하세요.리뷰 기능을 만들다가 동시성 문제를 해결하기 위해 redisson으로 lock을 활용하여 개발을 진행 중에 lockName 할당을 어떻게 처리를 해야하는지 질문 드립니다.product 테이블에 total_review_count, total_review_score 컬럼을 관리하고 있습니다.리뷰가 등록/수정/삭제 된다면 해당되는 product_id 의 total_review_count, total_review_score 값을 변경해야 합니다.lockName을 product_id 를 활용해서 걸어야 한다고 생각을 했습니다.하지만 리뷰를 처리할 때, 클라이언트에서 product_id 를 받지 않고, order_id 또는 review_id 받아서 리뷰를 처리할 수 있는 상황입니다.로직은 Facade 또는 AOP를 활용해서 lock 획득 -> 비즈니스 로직 시작을 위해 Transaction 시작 순서로 진행됩니다.lockName은 product_id 를 활용해야 하는 상황이기 때문에.. 클라이언트에서 product_id 를 받아서 락을 걸고 Transaction 진입 후 product_id 를 검증을 진행해야 하는게 맞는건지 궁금합니다.질문 요약은 다음과 같습니다.product_id 를 활용해서 lock을 걸어야 한다.하지만 product_id 는 lock을 걸고, Transaction 시작 후 데이터를 조회하여 가져올 수 있다.lockName에 product_id 를 활용하기 위해서 어떻게 처리를 해야하는가?클라이언트에게 product_id를 받아서 lock을 걸고, Transaction 시작 후 데이터를 조회하여 product_id 를 검증한다?데이터베이스 설계를 바꾼다?굳이 동시성을 고려해서 Lock을 걸 필요 없는거 같다?모든 리뷰 요청(등록/수정/삭제)에 대해서 1개의 lockName으로 처리를 한다? (e.g. LOCK:REVIEW)lockName을 어떻게 처리하는게 적절한지 경험적 의견 부탁드리겠습니다~감사합니다.아래는 3-1 질문에 대한 예제 코드 입니다.아래 코드 작성을 위해 참고한 자료는 https://helloworld.kurly.com/blog/distributed-redisson-lock/ 입니다. &#64;DistributedLock(key &#61; &#34;#lockName&#34;) public void saveCustomerUnwrittenReview(String lockName, ReviewCreateReq request, List<MultipartFile> images) { Customer customer &#61; customerService.getCustomer(); OrderProduct orderProduct &#61; orderProductService.getUnwrittenReview(customer.getId(), request.getOrderProductId()); // 구매한 상품 아이디와 요청받은 상품 아이디의 일치 여부 검증 if (!orderProduct.getProduct().getId().equals(request.getProductId())) { return; } // ... 중략 }

wnsldjdnd · Answer

좋아요 수 같이 하나의 데이터에 접근하는게 아니라서 굳이 락을 걸 필요가 없어 보입니다

장독깨기 · Answer

첫째, 굳이 레디스를 활용해서 락을 걸겠다면,클라이어트에서 받은 주문 또는 리뷰 아이디로 프러덕트 아이디를 구하고, 이 후로 락을 걸어 처리하면 될 거 같습니다.둘째, 데이터베이스에서 직접 락을 걸면 되지 않을까요.select ... for update 하면 리드락이 걸리니 원하는걸 할 수 있습니다.셋째, 프러덕트에서 리뷰수를 조회해서 업데이트 하지 마시구요,이렇게 하려니 락을 걸어야 하잖아요. update set review_count = review_count+1 이런 식으로 하면 굳이 리뷰수를 읽을 필요 없겠죠.넷째, 리뷰 데이터가 얼마나 되는지 모르겠군요,웬만한건 굳이 카운트를 누적하지 않아도 됩니다.근데, 진짜 데이터가 많으면 반정규화를 고려하긴 해야 하는데요,이럴 경우, 첫번째나 두번째 처럼 락을 걸면 안 되겠죠. 락 거는 것 자체가 성능을 떨어뜨리는거잖아요.세번째 방식으로 하면 웬만큼 되겠지만, 그래도 데이터가 너무 많아 성능에 문제가 있다면,가령, 리뷰 작성과 카운트 업데이트를 하나의 트랜잭션으로 묶지 마시구요,카운트 누적하는 스레드 별도로 만들어서 처리하면 될 것 입니다.이렇게 보니, 레디스 활용하는 것이 제일 안 좋은 방식인거 같습니다. ㅎ한 번 참고해서 생각해보세요. :)

Clickin · Answer

https://okky.kr/questions/1496344비슷한 고민을 하신 분 질문 참고하시면 도움이 될 듯 합니다.

Redisson으로 분산락을 사용할 때, lockName 할당 관련하여 질문이 있습니다.

3개의 답변

Q&A 목록