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

public void buy(String ticketName) throws InterruptedException {
    RLock lock = redissonClient.getLock(ticketName);

    try {
        boolean available = lock.tryLock(5, 1, TimeUnit.SECONDS);
        if (!available) {
            return;
        }

        ticketServiceWithRedisRedissonClientLock.buy(ticketName);
    } finally {
        if (lock.isLocked() && lock.isHeldByCurrentThread()) {
            lock.unlock();
        }
    }
}

이 구조에서 tryLock에 대해서 5초동안 Lock 획득을 위해서 대기하고 Lock을 1초동안 점유하고 release하는 형식으로 알고있는데 만약에 쓰레드가 5초 동안 대기를 하더라도 최종적으로 Lock을 얻지 못한다면 return이 됨으로써 buy로직으로 못들어가는거 아닌가요??

만약에 반드시 Lock을 얻고 buy로 들어가야만 하는 경우 Lock 재획득에 대한 로직을 따로 구현해야 하는건가요. 아니면 tryLock의 재시도에 대해서 제가 모르는 부분이 있는건가요??

public void buy(String ticketName) throws InterruptedException {
    final RLock lock = redissonClient.getLock(ticketName);
    final int maxRetryCount = 10;
    final int retryIntervalMillis = 1000;

    try {
        int retryCount = 0;

        boolean lockAcquired = lock.tryLock(5, 1, TimeUnit.SECONDS);
        while (!lockAcquired && retryCount < maxRetryCount) {
            log.info(
                    "--> Thread [{}] Redis Distributed Lock (Redisson Client) 획득 대기",
                    Thread.currentThread().getName()
            );
            Thread.sleep(retryIntervalMillis);
            lockAcquired = lock.tryLock(5, 1, TimeUnit.SECONDS);
            retryCount++;
        }

        if (!lockAcquired) {
            log.info(
                    "--> Thread [{}] Redis Distributed Lock (Redisson Client) 획득 실패",
                    Thread.currentThread().getName()
            );
            return;
        }

        ticketServiceWithRedisRedissonClientLock.buy(ticketName);
    } finally {
        if (lock.isLocked() && lock.isHeldByCurrentThread()) {
            lock.unlock();
        }
    }
}

이런 방식으로 최대 재시도 횟수와 재시도 간 간격을 통해서 waitTime동안 Lock을 못얻을 경우 Lock자체를 다시 얻는 방식을 구현해보았고 waitTime=1, unit=ms로 변경하고 테스트했을 경우 정상적으로 retry가 됨을 확인했습니다.

이런 구조가 최선일까요??

아니면 더 나은 방식이 있을까요?