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

Fix: searchQuestions 쿼리문 distinct 제거 #269

Merged
merged 1 commit into from
Nov 3, 2024
Merged

Conversation

adorableco
Copy link
Contributor

PR

✨ 작업 내용


✨ 참고 사항

  • 커밋에 코멘트를 달아서 알림이 잘 전달이 안됐을 수도 있겠다는 생각이 들어서.. 일단 제가 distinct 없애고 커밋 푸시했습니다이! 내가 구현한 부분이 아니라 잘못됐을 수도 있으니 컨펌받은 후에 롤백하거나 develop 으로 합치거나 할게용

⏰ 현재 버그

x


✏ Git Close

@adorableco adorableco added 🛠️ Fix Something isn't working 🫧 seyeon seyeon labels Nov 3, 2024
@adorableco adorableco requested a review from RinRinPARK November 3, 2024 06:06
@adorableco adorableco self-assigned this Nov 3, 2024
@adorableco adorableco linked an issue Nov 3, 2024 that may be closed by this pull request
Copy link

github-actions bot commented Nov 3, 2024

테스트 커버리지 측정

There is no coverage information present for the Files changed

Total Project Coverage 79.5% 🍏

Copy link
Contributor

@RinRinPARK RinRinPARK left a comment

Choose a reason for hiding this comment

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

내 기억에 여기서 fetch join 쓰면 tag수만큼 question 똑같은게 중복되어 결과로 나와서 distinct 붙여줬던거 같아 아마두??
내가 지금 노트북을 놓고 본가에 와가지구 혹시 테스트 한번 돌렸을때 distinct 빼고도 중복 안나오면 바로 머지해도 될 듯!!!

@adorableco
Copy link
Contributor Author

내 기억에 여기서 fetch join 쓰면 tag수만큼 question 똑같은게 중복되어 결과로 나와서 distinct 붙여줬던거 같아 아마두?? 내가 지금 노트북을 놓고 본가에 와가지구 혹시 테스트 한번 돌렸을때 distinct 빼고도 중복 안나오면 바로 머지해도 될 듯!!!

그 내가 실행했을 땐 distinct()를 붙였을 때랑 뺐을 때랑 둘 다 중복은 나오고 있거던..!
근데 이게 칼럼 몇개 뽑아오는게 아니라 전체 칼럼 조회니까 distinct 를 해도 그대로 나오는게 당연한 것 같더라구 그런 의미에스 비용드는 distinct를 뺐수다!!
급한건 아니니까 나중에 확인 한번 플리주~~!!

@adorableco adorableco merged commit 8215701 into develop Nov 3, 2024
1 check passed
@adorableco adorableco deleted the seyeon/swm-203 branch November 3, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️ Fix Something isn't working 🫧 seyeon seyeon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix: searchQuestions distinct 제거
2 participants