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

Week 3/donggeon #10

Merged
merged 2 commits into from
Nov 26, 2024
Merged

Week 3/donggeon #10

merged 2 commits into from
Nov 26, 2024

Conversation

DongGeon0908
Copy link
Member

πŸ“Œ Study

  • 코루틴 ν•™μŠ΅, μŠ€λ ˆλ“œμ™€μ˜ 차이

πŸ™ Focus on me

  • μ €λŠ” λΈ”λ‘œκ·Έμ— μ‹€ν—˜ λ‚΄μš©μ„ κΈ°λ‘ν–ˆμŠ΅λ‹ˆλ‹€!

Copy link
Collaborator

@jaeyeong951 jaeyeong951 left a comment

Choose a reason for hiding this comment

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

κ³ μƒν•˜μ…¨μŠ΅λ‹ˆλ‹€! 🫑

- λ””μŠ€νŒ¨μ²˜(Dispatcher)κ°€ 코루틴 싀행을 μ œμ–΄ν•˜λ©°, μ‹€ν–‰ κ°€λŠ₯ν•œ 코루틴을 μŠ€λ ˆλ“œ 풀에 배치.
- μŠ€μΌ€μ€„λ§μ΄ ν˜‘λ ₯적(cooperative) λ°©μ‹μœΌλ‘œ μ΄λ£¨μ–΄μ§€λ―€λ‘œ, μ˜€λ²„ν—€λ“œκ°€ 적음.

4. **κ΅¬μ‘°ν™”λœ λ™μ‹œμ„±**:
Copy link
Collaborator

Choose a reason for hiding this comment

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

κ΅¬μ‘°ν™”λœ λ™μ‹œμ„± λ„ˆλ¬΄ μ‘°ν•˜μš”

Copy link
Member

@char-yb char-yb left a comment

Choose a reason for hiding this comment

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

LGTM

Comment on lines +18 to +21
- λΉ„μš© μ΅œμ ν™”
- Webfluxμ™€λŠ” λ‹€λ₯Έ 가독성
- μ½”λ“œ μœ μ§€λ³΄μˆ˜μ— 맀우 탁월

Copy link
Member

Choose a reason for hiding this comment

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

ν™•μ‹€νžˆ Webflux보단 가독성이 λ„ˆλ¬΄ μ‘°ν•˜μš”!

Comment on lines +87 to +89
**정리**
- **μŠ€λ ˆλ“œ**: μš΄μ˜μ²΄μ œκ°€ κ΄€λ¦¬ν•˜λŠ” 무거운 μ‹€ν–‰ λ‹¨μœ„λ‘œ, 높은 λΉ„μš©μ„ κ°μˆ˜ν•΄μ•Ό 함.
- **코루틴**: μ‚¬μš©μž μˆ˜μ€€μ—μ„œ κ΄€λ¦¬λ˜λŠ” κ²½λŸ‰ μ‹€ν–‰ λ‹¨μœ„λ‘œ, **ν˜‘λ ₯적 μŠ€μΌ€μ€„λ§**κ³Ό **μƒνƒœ λ¨Έμ‹  기반의 λ™μž‘**을 톡해 μžμ›μ„ μ΅œμ ν™”.
Copy link
Member

Choose a reason for hiding this comment

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

μ‹€μŠ΅ν•΄λ΄€λŠ”λ° μ§„μ§œ μŠ€λ ˆλ“œλŠ” λΉ„μš©μ΄ λ†’λ”λΌκ³ μš”γ… 

@DongGeon0908 DongGeon0908 merged commit d741149 into main Nov 26, 2024
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