Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX: Handling interrupt exception when blocking by future get invoked. #683

Closed
wants to merge 1 commit into from

Conversation

brido4125
Copy link
Collaborator

Motivation

기존 arcusClient의 경우 동기 api와 몇몇의 broadcast api는
내부에서 Future.get을 호출한 다음 해당 결과를 리턴한다.

이러한 api들의 interrupt Exception Handling 방식을
Thread.currentThread().interrupt();
호출하도록 통일한다.

@brido4125 brido4125 self-assigned this Oct 6, 2023
@brido4125 brido4125 changed the title FIX: Handling interrupt exception when blocking by future get. FIX: Handling interrupt exception when blocking by future get invoked. Oct 6, 2023
@brido4125 brido4125 requested a review from uhm0311 October 6, 2023 07:36
@uhm0311
Copy link
Collaborator

uhm0311 commented Oct 6, 2023

질문이 있습니다.

  1. MemcachedClient 클래스에서 일부 future.get()을 호출하면서도 Exception 발생 시 cancel 처리 하지 않는데, 올바른 것인지?
    • 고칠 필요가 있다면 별도의 PR로 처리합시다.
  2. MemcachedClient 클래스의 waitForQueues(), listSaslMechanisms(), mutateWithDefault(), mutate() 메소드에서는 interrupt 처리를 하지 않아도 되나요?

@brido4125
Copy link
Collaborator Author

MemcachedClient 클래스에서 일부 future.get()을 호출하면서도 Exception 발생 시 cancel 처리 하지 않는데, 올바른 것인지?

별도 PR로 처리할 예정입니다.

MemcachedClient 클래스의 waitForQueues(), listSaslMechanisms(), mutateWithDefault(), mutate() 메소드에서는 interrupt 처리를 하지 않아도 되나요?

놓친 부분이였습니다. 반영하겠습니다.

@jhpark816
Copy link
Collaborator

@brido4125 @uhm0311
InterruptedException 발생 시 해당 Exception을 그대로 던지는 것이 나은 방안이라, 본 PR은 close 합니다.

@jhpark816 jhpark816 closed this Oct 27, 2023
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