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

20250214/JpaRepository-Transactional - 김대현 기술블로그 제출 #129

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

kdh10806
Copy link
Contributor

@kdh10806 kdh10806 commented Feb 13, 2025

20250214/JpaRepository-Transactional - 김대현 기술블로그 제출

@kdh10806 kdh10806 self-assigned this Feb 13, 2025
@kdh10806 kdh10806 changed the title 20250207/JpaRepository-Transactional - 김대현 기술블로그 제출 20250214/JpaRepository-Transactional - 김대현 기술블로그 제출 Feb 13, 2025
Copy link

@baki-719 baki-719 left a comment

Choose a reason for hiding this comment

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

안녕하세요 대현님

우선 제 눈에는 몇가지 피드백을 드리자면 틀린 내용은 없어보입니다.

다만, 좀 더 내용이 추가되면 좋을꺼 같은데요

우선 특정 프레임워크나 라이브러리에 관한 설명 글에는 실제 코드나 사용예시를 보여주면 좋습니다.
예시 코드를 좀 더 추가해보는 건 어떨까요?

그리고 저희가 JpaRepository를 사용할때 쿼리메서드를 추가해서 사용하자나요?
그떄는 단순 구체 클래스인 SimpleJpaRepository에 없는 내용일 껀데 어떻게 동작할지 추가해주면 좋을 것으로 보입니다. (이정도로 파악하셨으면 이미 알고 계실꺼니까요)

추가적으로 프락시 객체가 무엇 인지를 좀 더 적고 이를 Spring data jpa에서 어떻게 사용하는 가로 보여주는게 좋을 것으로 보여요
아무래도 프락시객체에 대한 개념이 없는상태에서 읽으면 이해가 어려울꺼니까요

그래도 좋은 주제로 좋은 글을 작성해주셔서 재밋게 읽었습니다
감사합니다.

@kdh10806
Copy link
Contributor Author

피드백 부분 수정하였습니다.

@kdh10806 kdh10806 requested a review from baki-719 February 20, 2025 00:22
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.

2 participants