동시성 이슈

[jeongmallro]

💬 Describe

• 동시성 문제가 발생할 수 있는 메서드에 대한 조치
• 동시성 문제 발생 가능 기능
  • BM 스크랩
  • SP 좋아요

📌 Tasks

현재 단일 서버이기 때문에 자바 코드 단에서 synchronized로 간단히 해결할 수 있을 것 같아요.
충돌이 빈번하게 일어나지 않는다고 가정하면 MySQL의 낙관적 락을 구현해도 괜찮다고 생각합니다.

분산 락을 적용할 거라면 MySQL, Redis 둘 다 사용하고 있기 때문에 둘 다 무리는 없어보이고 비용적인 측면에서 고려하면 될 것 같아요!

어떻게 생각하시나요?

ETC

No response

[lsh2613]

이왕 동시성 문제를 해결한다면, 멀티 서버 환경을 고려해서 작업하는 게 좋을 것 같아요

좋아요의 경우 soft-real time으로 처리해도 괜찮기 때문에, redis를 활용한 대기열큐에 적재하고 스케쥴러를 통해 반영해주는 방법도 고려해보면 좋겠네요

[jeongmallro]

//SpService
Long likeCnt = spLikeService.countBySpId(sp.getId());

현재 sp 좋아요 수를 조회해올 때 sp_like 테이블을 count하기 때문에 말씀하신 방법은 sp 테이블에 like_count 필드를 추가하는 반정규화를 할 때 고려해볼 수 있을 것 같아요!

일단 반정규화를 할지 말지 고민인데... 반정규화 하기 전에 조회 쿼리랑 인덱스 튜닝을 먼저해야 하지 않나 싶긴 해요

[lsh2613]

제가 말한 방법은 정규화 반정규화랑은 관련 없이 해당 데이터를 어떻게 처리 하느냐의 관점 아닌가요??

어떤 의미에서 반정규화 시 고려할 만한 내용인지 잘 와닿지 않네요. 추가 설명 해주실 수 있을까요??

[jeongmallro]

redis를 활용한 대기열큐에 적재하는 게 업데이트될 좋아요 수 아닌가요?

현재 사용자가 sp 조회할 때 사용자의 좋아요 여부도 같이 조회되기 때문에 sp_like 테이블에는 바로바로 반영해줘야 한다고 생각했어요

그래서 반정규화해서 sp 테이블에 like_count 필드가 추가된다면 redis 큐에 like_count에 업데이트될 변동량이 적재될 거라고 생각했습니다!

[lsh2613]

아 저는 계속 조회수를 생각하고 있었네요

스크랩과 좋아요에 대해 동시성 이슈를 다룰 필요가 있을까요??

위 상황에서 동시성 이슈가 발생하려면 SpLike(spId, memberId) 또는 BmScrap(bmId, memberId)에 여러 스레드가 접근해야 되는데, 이 경우는 특정 한 유저가 같은 Api를 동시에 호출하는 경우밖에 없지 않나요?

[jeongmallro]

맞아요 그래서 오늘 다시 생각해보니까 Lock을 굳이 걸어야 할까 생각 중이긴 했는데요... ㅋㅋㅋㅋㅋ

찾아보니까 제가 해결하려고 했던 동시성 이슈에 대해서 그냥 unique 제약조건을 걸고 동시성 이슈(unique 제약 조건 위반) 발생 시 그냥 공통 예외 처리 정도로 하더라고요

그런데 생각해보면 이런 동시성 문제는 진짜 거의 터질일이 없습니다. 따라서 DB에서 동시성 문제가 발생하면(Unique 제약조건 위배) 따로 catch로 잡지말고, 그냥 공통 예외로 컨트롤러 끝까지 보내서 공통 예외로 처리되도록 만듭니다. 그리고 공통 예외 처리에서 고객에게는 시스템에 문제가 있습니다. 정도로 뿌리고 대신 디버깅을위해 시스템에 로그로 자세히 남기는 정도로 마무리하면 됩니다^^
안녕하세요. Unique Index 데이터 저장에 대해서 질문드립니다.

현재 발생하는 동시성 에러

• sp_like 테이블에 member_id, sp_id 조합에 대한 unique 제약 조건 없음
• 에러 메시지: Row was updated or deleted by another transaction (or unsaved-value mapping was incorrect)

unique 제약 조건 추가 시 발생하는 동시성 에러

• sp_like 테이블에 member_id, sp_id 조합에 대한 unique 제약 조건 아래와 같이 설정

//SpLike
@Table(uniqueConstraints = @UniqueConstraint(columnNames = {"member_id", "sp_id"}))

• 에러 메시지: Unique index or primary key violation: "PUBLIC.CONSTRAINT_INDEX_B ON PUBLIC.SP_LIKE(MEMBER_ID NULLS FIRST, SP_ID NULLS FIRST) VALUES ( /* key:2 */ CAST(1 AS BIGINT), CAST(1 AS BIGINT))";
• 해당 에러를 컨트롤러 단까지 보내서 예외로 처리 "이미 좋아요를 누르셨습니다" 정도

반정규화 계획이 없다면, unique 제약 조건 걸고 예외를 그대로 터뜨리는 정도로 하면 될 것 같아요
사실 이 unique 제약 조건은 진즉 걸었어야 했는데 이제 발견했네요ㅋㅋ큐ㅠㅠ

[lsh2613]

네 오히려 조회수 로직에서 동시성 이슈를 잡으면 모를까 스크랩 좋아요에서는 굳이 필요 없을 것 같아요

[jeongmallro]

아 저는 오히려 조회수를 생각 못 하고 있었네요
우선 스크랩, Sp는 저렇게 처리하고, 조회수는 말씀하신 방법으로 한번 적용해보겠습니다