Skip to content

CLEANUP: Reuse callback instance for repeated Op creation#1051

Merged
jhpark816 merged 1 commit intonaver:developfrom
f1v3-dev:fix/op-callback
Feb 26, 2026
Merged

CLEANUP: Reuse callback instance for repeated Op creation#1051
jhpark816 merged 1 commit intonaver:developfrom
f1v3-dev:fix/op-callback

Conversation

@f1v3-dev
Copy link
Collaborator

@f1v3-dev f1v3-dev commented Feb 26, 2026

🔗 Related Issue

⌨️ What I did

  • AS-IS: 복수 개의 Operaiton 을 생성하는 연산에서 반복문 내에 동일한 로직의 Callback을 생성하고 있었습니다.
  • TO-BE: 반복문 외부에서 Callback 을 단일 인스턴스로 생성하고 공유하는 형태로 변경했습니다.

변경사항

  • ArcusClient: flush, asyncCollectionInsertBulk2, btreeGetBulk, btreeGetBulkByteArrayBKey
  • MemcachedClient: flush, listSaslMechanisms, waitForQueues

@jhpark816 jhpark816 requested a review from oliviarla February 26, 2026 02:24
@jhpark816
Copy link
Collaborator

@f1v3-dev CI 확인 바랍니다.

oliviarla
oliviarla previously approved these changes Feb 26, 2026
@oliviarla oliviarla requested a review from jhpark816 February 26, 2026 06:30
Copy link
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.

리뷰 완료

public void complete() {
rv.complete();
}
};
Copy link
Collaborator

Choose a reason for hiding this comment

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

위의 코드는 checkState(); 호출하기 전으로 옮기면 좋겠습니다.

PR 전체 코드에서 함께 수정 바랍니다.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

반영했습니다.

@jhpark816 jhpark816 merged commit 621751b into naver:develop Feb 26, 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