인프런 커뮤니티 질문&답변

강의를 듣던중 궁금한 점이 생겨 질문드립니다.

아래의 코드는 MemberServiceV2의 두 함수 입니다.

public void accountTransfer(String fromId, String toId, int money) throws SQLException {

        Connection con = dataSource.getConnection();

        try {
            con.setAutoCommit(false); // ! 트랜잭션 시작

            // * 비즈니스 로직
            bizLogic(fromId, toId, money, con);

            con.commit(); // * 성공시 commit

        } catch (Exception e) {
            con.rollback(); // ! 실패시 rollback
            throw new IllegalStateException(e);
        } finally {
            release(con);
        }

    }

private void bizLogic(String fromId, String toId, int money, Connection con) throws SQLException {
        Member fromMember = memberRepository.findById(toId, con);
        Member toMember = memberRepository.findById(toId, con);

        ---<다른 db 커넥션에 의해 침범될 수 있는 구간>---

        memberRepository.update(fromId, fromMember.getMoney() - money, con);
        validation(toMember);
        memberRepository.update(toId, toMember.getMoney() + money, con);
    }

트랜잭션을 사용하기 위해서 con.setAutoCommit(false) 를 사용하고 있는데,

강의를 듣고 이해한 내용에 의하면, accountTransfer에 의해서 db에 요청되는 sql 쿼리는 총 4개입니다.

memberRepository.findById -> select 문 (2번)

memberRepository.update -> update 문 (2번)

그런데, select for update가 아닌 select문은 선택된 row에 대해 lock을 걸지 않기 때문에, 위의 코드에 적어놓은 것 처럼 update문이 수행되기 전에 다른 db 세션에 의해서 동일한 데이터가 수정될 여지가 있는 것 같아 보입니다.(update 문이 수행될 때 lock이 걸리는게 맞다면)

그럼, findById 함수 내의 sql문을 select for update로 변경하거나, 코드에 적어놓은 select문과 update 문 사이에 또다른 lock을 설정해주어야하는 건가요? + 만약, 또다른 lock을 설정해야한다면 어떻게 해야하나요?

P.S

https://www.inflearn.com/questions/653523

아래의 유사한 질문을 발견했는데, 해당 질문의 답변만으로 충분히 이해가 되지 않아서 추가적으로 질문드린 것입니다.