Skip to content

FEATURE: Add CompletableFuture incr, decr API#1058

Open
f1v3-dev wants to merge 1 commit intonaver:developfrom
f1v3-dev:feat/v2-incr-decr
Open

FEATURE: Add CompletableFuture incr, decr API#1058
f1v3-dev wants to merge 1 commit intonaver:developfrom
f1v3-dev:feat/v2-incr-decr

Conversation

@f1v3-dev
Copy link
Collaborator

🔗 Related Issue

⌨️ What I did

  • 값 증감 (incr, decr) API 구현을 진행했습니다.
    • 기존 사용자의 혼란을 최소화하고자 API명을 유지하였습니다.

두 메서드 동작 차이

메서드 키 없을 때 동작
incr(key, delta) -1 반환
incr(key, delta, initial, exp) initial로 초기화 (delta 미적용)

decr()도 동일한 동작 차이를 가집니다.

@f1v3-dev f1v3-dev requested a review from oliviarla March 12, 2026 10:37
@f1v3-dev f1v3-dev self-assigned this Mar 12, 2026
@f1v3-dev f1v3-dev force-pushed the feat/v2-incr-decr branch from 80e42eb to 24bdf53 Compare March 13, 2026 01:30
Copy link
Collaborator

@oliviarla oliviarla left a comment

Choose a reason for hiding this comment

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

리뷰 완료입니다.

@f1v3-dev f1v3-dev force-pushed the feat/v2-incr-decr branch from 24bdf53 to 2a66bf6 Compare March 13, 2026 10:17
}

@Test
void decrTypeMismatchException() throws ExecutionException, InterruptedException,
Copy link
Collaborator

Choose a reason for hiding this comment

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

decrTypeMismatchException -> decrTypeNotNumberException
로 변경하는게 좋겠습니다.

collection 타입 키를 생성해서 TypeMismatch 발생시키는 테스트도 추가되면 좋겠습니다.

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.

2 participants