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-runcatching] suspend 함수에서 runCatching 사용하는 anti-pattern 수정 #405

Merged
merged 1 commit into from
Dec 25, 2024

Conversation

l2hyunwoo
Copy link
Collaborator

TL;DR

이슈 설명

  • 코루틴의 구조화 동시성의 구조상 한 Job이 캔슬되면 다른 잡이 캔슬되도록 만들어야 함. 그 구조가 Exception(CancellationException)을 다른 코루틴 잡에서 핸들링해서 터뜨리는 구조인데
  • 현재 구조를 보았을 때 이런 구조화 동시성이 제대로 동작하지 않도록 사용이 되고 있어, 원하지 않은 동작으로 돌아갈 수 있음 (runCatching은 모든 에러를 다 캐치하도록 되어있음)
  • 따라서 위의 제안해준 구조를 바탕으로 suspend 함수에서 runCatching 사용을 제거하고자 함

@l2hyunwoo l2hyunwoo added the bug Something isn't working label Dec 25, 2024
@l2hyunwoo l2hyunwoo self-assigned this Dec 25, 2024
Copy link
Member

@mwy3055 mwy3055 left a comment

Choose a reason for hiding this comment

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

함수 이름이 runCatching뭐시기였으면 좋을 거 같은데요 (자동완성에 보이니까)

근데 적당한 이름이... runCatchingCoroutines? 딱히 좋은 이름이 떠오르지 않아서 일단 approve 드립니다

@l2hyunwoo l2hyunwoo merged commit 0933f6a into develop Dec 25, 2024
2 checks passed
@l2hyunwoo l2hyunwoo deleted the feature/fix-runcatching branch December 25, 2024 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants