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

✨ 전체 메모 조회 및 검색 API #117

Merged
merged 2 commits into from
Feb 20, 2024
Merged

✨ 전체 메모 조회 및 검색 API #117

merged 2 commits into from
Feb 20, 2024

Conversation

psychology50
Copy link
Member

작업 이유

허허, 실수로 브랜치를 새로 안 파고 전에 작업하던 브랜치에서 해버렸네요.

  • UI 변경으로 인한 기능 스펙 변경 대응
  • 관리 중인 전체 반려동물에게 등록된 메모를 대상으로 조회 및 검색하는 API 추가

작업 사항

image


⚠️ text 검색 시, match percent가 낮아서 검색이 안 되는 이슈

image

  • 현재 MySQL의 Full Text Index를 사용하여 빠른 검색을 보장함.
  • 다만, 특정 단어가 분명히 존재함에도 찾지 못하는 경우가 존재.
  • 해당 부분은 추후 속도를 포기해서 Like를 사용하거나, 메모리를 포기해서 Full Text Index with PARSER를 적용해야 할 것 같다.

이슈 연결

close #116

Copy link
Contributor

@heejinnn heejinnn left a comment

Choose a reason for hiding this comment

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

전체 메모 조회로 바꿔주신 거 확인했습니다~ 검색은,,, 잘 해결되길 바랄게요 ㅎㅎ

@heejinnn heejinnn merged commit 7ac08ef into develop Feb 20, 2024
0 of 3 checks passed
@heejinnn heejinnn deleted the hotfix/111 branch February 20, 2024 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ 반려동물 전체 기록 및 검색 API
2 participants