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

[20기_김연수] Docker & API 리팩토링 미션 제출합니다. #45

Open
wants to merge 2 commits into
base: juanxiu
Choose a base branch
from

Conversation

juanxiu
Copy link
Contributor

@juanxiu juanxiu commented Nov 16, 2024

No description provided.

Copy link

@seona-moon seona-moon left a comment

Choose a reason for hiding this comment

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

연수! 날이 많이 추워지는데 감기조심하고, 슬슬 세션 마지막이 다가오는데 잘 마무리해봅시다 🫶 이번주도 수고 많았어요!


## WEEK 6
### 도커

Choose a reason for hiding this comment

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

정리 꼼꼼하게 잘 해주신 것 같아요!

@@ -454,12 +454,13 @@ URI: `api/posts` 게시물 생성 API
성공 응답을 나눌 필요는? 200과 201

### @Transactional(readOnly = true)
- 트랜잭션 전역처리
- Service 단에서 @Transactional(readOnly=True)를 클래스 레벨에 선언하면 자동으로 모든 서비스 내 메소드에 읽기전용 트랜잭션이 적용되기에, 삭제와 추가같이 디비를 조작할 일이 있는 메소드에만 @Transactional을 쓰는 방식으로 구현
- 트랜잭션 전역처리
Copy link
Contributor

Choose a reason for hiding this comment

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

이전 리드미까지 업데이트 해주셔서 복습하기에 좋았습니다 :)

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