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

#330 [feat] 메트릭 api 구현 #331

Merged
merged 5 commits into from
Aug 31, 2024
Merged

#330 [feat] 메트릭 api 구현 #331

merged 5 commits into from
Aug 31, 2024

Conversation

KWY0218
Copy link
Member

@KWY0218 KWY0218 commented Aug 30, 2024

✒️ 관련 이슈번호

Key Changes 🔑

  1. 내용
    • 명세대로 api 구현했습니다.
    • api 응답 속도 감소와 서비스와 slack 서비스의 의존성을 줄이고자 스프링 이벤트 방식을 사용했습니다.
    • 스프링 이벤트는 동기 방식으로 동작하기 때문에, 비동기 방식으로 동작하기 위해 @Async 어노테이션을 붙였고,
      @Async 어노테이션은 호출할 때마다 스레드를 생성하기 때문에, 별도의 스레드 풀을 만들었습니다.
      스레드 풀 같은 경우에는 이벤트 호출이 적을 것을 예상하여 스레드 풀을 1로 설정했습니다.

개설된 총 회의 수, 확정된 총 회의 수, 사용한 총 사용자 수를 반환하는 쿼리
slack 알림을 전송하는 기능을 아래 장점 때문에 이벤트로 등록
1. 서비스와 인프라 간 의존성 제거
2. api 응답 속도 향상

스프링 이벤트는 동기 방식으로 진행하기 때문에 비동기 처리를 위해
@async 및 async config 등록
@KWY0218 KWY0218 requested a review from sohyundoh August 30, 2024 13:42
@KWY0218 KWY0218 self-assigned this Aug 30, 2024
Copy link
Member

@sohyundoh sohyundoh left a comment

Choose a reason for hiding this comment

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

LGTM👍👍

@KWY0218 KWY0218 merged commit 8bcbff8 into develop Aug 31, 2024
4 checks passed
@KWY0218 KWY0218 deleted the feat/#330 branch August 31, 2024 03:01
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.

[feat] 메트릭 api 구현
2 participants