-
Notifications
You must be signed in to change notification settings - Fork 3
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
Querydsl 설정 #110
Open
HanKwanJin
wants to merge
1
commit into
main
Choose a base branch
from
feature/#109/querydsl
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Querydsl 설정 #110
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Querydsl에서 생성된 Q 클래스들을 왜 .gitignore에 등록했는지에 대한 내용도 추가하겠습니다.
Querydsl을 사용하면, 컴파일 시점에 Q 클래스를 생성합니다. 이 Q 클래스는 엔티티 클래스의 필드와 연관 관계를 모두 포함하고 있으며, JPA 쿼리를 작성할 때 사용됩니다.
그러나, Q 클래스는 컴파일 시점에 생성되기 때문에, 소스 코드에 직접 포함되지 않습니다. 따라서, 별도의 파일로 생성되어야 합니다.
Q 클래스는 자주 변경되지 않으며, 소스 코드와는 별개의 파일로 생성됩니다. 따라서, 이 파일들은 버전 관리 시스템에서 제외해야 합니다. 그렇지 않으면, 굳이 변경되지 않는 파일들까지 커밋되어, 저장소 용량이 증가하고, 이를 동기화하는 데 시간이 오래 걸릴 수 있습니다. 따라서, Querydsl에서 생성되는 Q 클래스들은 .gitignore 파일에 추가하여, Git 저장소에서 제외하는 것이 좋습니다.
작성된 경로는 Querydsl 설정을
src/main/generated
에 생성되도록 했기 때문입니다.