Skip to content

FEATURE: Add CompletableFuture incr, decr API#1058

Merged
jhpark816 merged 1 commit intonaver:developfrom
f1v3-dev:feat/v2-incr-decr
Mar 23, 2026
Merged

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

Conversation

@f1v3-dev
Copy link
Copy Markdown
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
Copy Markdown
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
@f1v3-dev f1v3-dev force-pushed the feat/v2-incr-decr branch from 2a66bf6 to bfac666 Compare March 23, 2026 02:12
oliviarla
oliviarla previously approved these changes Mar 23, 2026
Copy link
Copy Markdown
Collaborator

@jhpark816 jhpark816 left a comment

Choose a reason for hiding this comment

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

리뷰 완료

Copy link
Copy Markdown
Collaborator

@jhpark816 jhpark816 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 d7ad4a0 to 919a27e Compare March 23, 2026 09:25
@f1v3-dev
Copy link
Copy Markdown
Collaborator Author

리뷰 완료

반영하였습니다.

@jhpark816 jhpark816 merged commit 9c86d30 into naver:develop Mar 23, 2026
2 checks passed
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.

3 participants