Skip to content

Conversation

@ayoung-dev
Copy link
Collaborator

📌 과제 설명

👩‍💻 요구 사항과 구현 내용

  • 커뮤니티 게시글 키워드 검색 조회 (제목, 내용)
  • 봉사활동 모집글 키워드 검색 조회 (제목, 내용)

✅ PR 포인트 & 궁금한 점

  • 봉사활동의 경우 기관명도 같이 조회하고 싶었는데 테이블이 한 번 조인되어있어 머리가 안돌아가서 추후에 리팩토링 하겠습니다..
  • 테스트는 elastic search에서 검색 결과 조회 해오는 부분이 개별로 실행해야만 성공해서 따로 두었습니다.
  • 코드가 좀 더러운 거 같은데 더 못보겠어서 pr 올립니다 아무거나 다 코드 리뷰로 말해주세요..

@ayoung-dev ayoung-dev self-assigned this Dec 8, 2024
@ayoung-dev ayoung-dev changed the title Feature/137 add search Feature/137 Feature/137 봉사활동 모집글, 커뮤니티 게시글 검색 조회 (elastic search) Dec 8, 2024
@github-actions github-actions bot changed the title Feature/137 Feature/137 봉사활동 모집글, 커뮤니티 게시글 검색 조회 (elastic search) [BUILD FAIL] Feature/137 add search Dec 8, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

빌드에 실패했습니다.

@github-actions github-actions bot closed this Dec 8, 2024
@ayoung-dev ayoung-dev changed the title [BUILD FAIL] Feature/137 add search [BUILD FAIL] Feature/137 봉사활동 모집글, 커뮤니티 게시글 검색 조회 (elastic search) Dec 8, 2024
@ayoung-dev ayoung-dev reopened this Dec 8, 2024
@ayoung-dev ayoung-dev closed this Dec 8, 2024
@github-actions github-actions bot changed the title [BUILD FAIL] Feature/137 봉사활동 모집글, 커뮤니티 게시글 검색 조회 (elastic search) [BUILD FAIL] [BUILD FAIL] Feature/137 봉사활동 모집글, 커뮤니티 게시글 검색 조회 (elastic search) Dec 8, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

빌드에 실패했습니다.



@Scheduled(cron = "10 * * * * *")
public void updateCommunityBoardDocuments() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

간격이 타이트 한 것 같습니다 제 생각엔 데이터 최신화를 위한 스케줄러 같은데 맞나요?
맞다면 getAll Community Boards라 부하가 클 거 같다는 생각이 듭니다
새벽에 한번 최신화를 시키고 사이에 벌어지는 업데이트들은
logstash로 저희 RDB를 체크하는게 어떨까요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵! 자정으로 수정하겠습니다

private final RecruitBoardDocumentUseCase recruitBoardDocumentUseCase;

@Scheduled(cron = "10 * * * * *")
public void updateRecruitBoardDocuments() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기도 간격이 타이트 한 것 같습니다!
스케줄러는 일 단위로 작동시키고 그 사이의 데이터 업데이트는 LogStash로 체크하는게 좋을것 같아요

Copy link
Collaborator

@leebs0521 leebs0521 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다,...

@Configuration
public class ElasticsearchConfig extends ElasticsearchConfiguration {
@Value("${elastic.search.uris}")
private String uri;
Copy link
Collaborator

Choose a reason for hiding this comment

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

uris 리스트인가요? 아니라면 url가 더 나은것 같습니다.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

수정했습니다!

bucket: somemore
base-url: https://somemore-image.s3.ap-northeast-2.amazonaws.com
base-url: https://somemore-images.s3.ap-northeast-2.amazonaws.com/
stack:
Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 바뀐 이유가 있나요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

엇 그러게요...?
env 수정하다가 잘못 수정했나봐요
다시 수정하겠습니다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] 커뮤니티 게시글 / 봉사 활동 모집글 elasticsearch 검색

4 participants