DOC: Update ConnectionFactoryBuilder docs#1053
Merged
jhpark816 merged 1 commit intonaver:developfrom Mar 6, 2026
Merged
Conversation
oliviarla
reviewed
Mar 6, 2026
| Operation Queue 내 연속된 GET 요청을 최대 100개 단위로 조합하여 하나의 요청으로 처리하는 최적화 로직 사용 여부를 설정한다. | ||
|
|
||
| 이 기능은 기존 spymemcached 호환을 위해 존재하며, cancel, replication 등의 시나리오에서 검증이 충분히 이루어지지 않았다. | ||
| 따라서, 비활성화(false)를 권장한다. |
Collaborator
There was a problem hiding this comment.
기본값인 비활성화(false) 사용을 권장한다. 이런 표현이면 좋겠습니다.
| - 기본값은 true로 응용 종료 시 I/O 스레드가 함께 종료된다. | ||
| - 기본값을 유지하며, Graceful Shutdown이 필요한 경우 `shutdown(long timeout, TimeUnit unit)`을 명시적으로 호출하는 것을 권장한다. | ||
|
|
||
| `setDaemon(false)` 설정 후 명시적으로 `shutdown()`이 호출되지 않은 상태에서 프로세스 종료가 시도ㅗ디면, 정상적으로 종료되지 않을 수 있다. |
|
|
||
| `setInitialObservers`는 노드 연결 상태를 모니터링하기 위한 Observer를 등록하는 메서드이다. | ||
|
|
||
| ArcusClient 또는 ArcusClientPool 생성 시 내부 초기화 과정에서 사용되어 등록한 Observer가 정상적으로 동작하지 않을 수 있다. |
Collaborator
Author
리뷰해주신 내용 반영하였습니다. |
Collaborator
|
@oliviarla 최종 리뷰 바랍니다 |
oliviarla
approved these changes
Mar 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 Related Issue
⌨️ What I did
setShouldOptimize개선setDaemon개선useNagleAlgorithm추가setInitialObservers추가setReadPriority / setAPIReadPriority추가