Skip to content

fix: redis scheduler stability/#698#699

Merged
chaeyn merged 1 commit into
developfrom
fix/redis-scheduler-stability/#698
Apr 23, 2026
Merged

fix: redis scheduler stability/#698#699
chaeyn merged 1 commit into
developfrom
fix/redis-scheduler-stability/#698

Conversation

@chaeyn

@chaeyn chaeyn commented Apr 23, 2026

Copy link
Copy Markdown
Member

변경사항

  • redis 타임아웃 시간 축소, 스케줄러 풀 확장

관련 이슈

Closes #698

추가 컨텍스트

@chaeyn chaeyn merged commit 2f7ace6 into develop Apr 23, 2026
1 check passed
@chaeyn chaeyn deleted the fix/redis-scheduler-stability/#698 branch April 23, 2026 00:05

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request configures a scheduling thread pool size and reduces Redis connection and operation timeouts to 3 seconds. The review feedback suggests adding a maximum wait time for the Redis connection pool to prevent infinite thread waiting, ensuring manual Redis client configurations in the Java code also reflect these timeout changes, and adding a thread name prefix to the task scheduler to improve log readability.

Comment on lines +64 to +65
connect-timeout: 3s
timeout: 3s

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

Redis 안정성을 위해 타임아웃을 단축한 것은 좋은 방향입니다. 다만, 시스템 안정성을 위해 다음 두 가지 사항을 추가로 검토해 주세요:

  1. Lettuce 커넥션 풀 대기 시간 설정: 현재 max-wait 설정이 없어 커넥션 풀 고갈 시 스레드가 무한정 대기할 수 있습니다. spring.data.redis.lettuce.pool.max-wait: 3s 설정을 추가하는 것을 권장합니다.
  2. 수동 설정 빈 확인: RedisConfig.java에서 직접 생성하는 bucket4jRedisClient 빈은 이 설정을 상속받지 않으므로, 해당 빈 설정에도 명시적인 타임아웃 적용이 필요합니다.

Comment on lines +5 to +8
task:
scheduling:
pool:
size: 10 # 각 스케줄러가 독립 스레드 사용

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

스케줄러 풀 크기를 확장하여 여러 작업이 병렬로 실행될 수 있도록 개선되었습니다. 이때 thread-name-prefix 설정을 함께 추가하면, 로그에서 스케줄러 스레드를 명확히 구분할 수 있어 운영 및 디버깅 시 유리합니다.

  task:
    scheduling:
      pool:
        size: 10 # 각 스케줄러가 독립 스레드 사용
      thread-name-prefix: scheduling-

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant