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.

Sign up for GitHub

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

Jump to bottom

[박승훈] 6장: 동시성 #55

Merged
merged 1 commit into from
Sep 16, 2024
Merged

[박승훈] 6장: 동시성 #55

merged 1 commit into from
Sep 16, 2024
+194 −0

Conversation

Orchemi
Copy link
Collaborator

@Orchemi Orchemi commented Aug 24, 2024 •
edited
Loading

🙇🙇

@github-actions github-actions bot added the 6장 label Aug 24, 2024
hyesungoh
hyesungoh approved these changes Aug 27, 2024
View reviewed changes
Copy link
Member

@hyesungoh hyesungoh left a comment

Choose a reason for hiding this comment

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

👍 👍 👍 👍 👍 👍 👍 👍 👍

sangbooom
sangbooom approved these changes Aug 28, 2024
View reviewed changes
Copy link
Collaborator

@sangbooom sangbooom left a comment

Choose a reason for hiding this comment

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

👍🏻👍🏻👍🏻

6장/박승훈.md
Comment on lines +28 to +29
- 프로그래밍을 할 때에는 보통 직선적 사고를 하기 마련이다. '이것을 하고, 그런 다음에 저것을 하고.' 하지만 이런 사고방식은 시간적 결합을 만들어낸다.
- 시간이나 순서에 의존하는 시간적 결합을 끊는 방법을 생각해야 한다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

제가 요즘 이러는데.. 뜨끔하네요

6장/박승훈.md
Comment on lines +114 to +116
- 하지만 그 뒤부터는 난해했다. 앞에서는 '비원자적'인 큰 프로세스에서부터 공유 상태의 갱신이 늦어지며 문제가 발생했는데, 그래서 점유 권한을 위해 '세마포어'라는 개념이 등장했다가, 이건 모두가 합의해야 되고, 동기적인 문제가 있다며 '트랜잭션'으로 관리하자고 했다. 그런데 '트랜잭션'도 결국 큰 하나의 일련의 작업이므로 여러 스레드에서 동시에 호출될 수 있다는 '비원자적'인 성격을 가지기 때문에 문제가 있어 '세마포어'로 관리해야 한다고 했다. 그리고 정상 동작을 위해 여러 실패 방어처리를 하는데, 이것도 별로라고 했다.
- 결론은 리소스를 공유하면서 동시성을 지키는 건 어렵다는 교훈만 남겼다.
- 다음 토픽에서 명쾌한 해답을 제안하며 극적인 반전을 주기 위한, 답답함을 쌓는 빌드업이었다면 훌륭했다. 충분히 난해했고 답답했다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

저도 읽으면서 같은 생각을 했습니다..

@Orchemi Orchemi merged commit e11c7e0 into main Sep 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gutenLEE gutenLEE gutenLEE approved these changes

@sangbooom sangbooom sangbooom approved these changes

@hyesungoh hyesungoh hyesungoh approved these changes

@100Gyeon 100Gyeon Awaiting requested review from 100Gyeon 100Gyeon is a code owner

@Dongkyuuuu Dongkyuuuu Awaiting requested review from Dongkyuuuu Dongkyuuuu is a code owner

@metacode22 metacode22 Awaiting requested review from metacode22 metacode22 is a code owner

@Seojunhwan Seojunhwan Awaiting requested review from Seojunhwan Seojunhwan is a code owner

@sjoleee sjoleee Awaiting requested review from sjoleee sjoleee is a code owner

@sumi-0011 sumi-0011 Awaiting requested review from sumi-0011 sumi-0011 is a code owner

Assignees

@Orchemi Orchemi

Labels
6장
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Orchemi @hyesungoh @sangbooom @gutenLEE