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

[BG-318]: RMQ notification 브로커 만들기 (2h / 2h) #62

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

Dltmd202
Copy link
Member

@Dltmd202 Dltmd202 commented Aug 2, 2024

Why

  • auto configuration 설정을 하려다 하지 않았다.
  • 일단 그 부분보다도 의존성이 받아지는 순간 rmq 클라이언트가 생성되었기 때문에 그 부분을 먼저 고민하는게 좋을 것 같았다.

How

config는 기본적으로 kafka로 유지

Result

  • 이메일 전송 성공
    스크린샷 2024-08-02 오후 1 19 08

Link

BG-318

@Dltmd202 Dltmd202 requested a review from GGHDMS August 2, 2024 04:19
@Dltmd202 Dltmd202 self-assigned this Aug 2, 2024
Copy link
Member

@GGHDMS GGHDMS left a comment

Choose a reason for hiding this comment

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

어떻게 해야하지..
client 때매 무조건 rabbitmq가 있어야 되넹

Comment on lines +11 to +14
kapt {
correctErrorTypes = true
}

Copy link
Member

Choose a reason for hiding this comment

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

뭐 때매 발생했어?

Copy link
Member Author

Choose a reason for hiding this comment

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

으음... 사실 잘 모르겠는데 그냥 .class 파일 번역한 부분에서 kapt 에서 코드를 찾을 수 없다 뭐 그런식으로 나왔는데 그냥 검색해서 대충 해결한거라ㅎㅎ

Copy link
Member

Choose a reason for hiding this comment

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

👍

@Dltmd202
Copy link
Member Author

Dltmd202 commented Aug 2, 2024

으음... 맞아 방법은 뭐 notification-broker-kafka, notification-broker-rmq 뭐 이런식으로 나누던가 해야할 것 같은데...

@Dltmd202 Dltmd202 added duplicate This issue or pull request already exists wontfix This will not be worked on and removed duplicate This issue or pull request already exists labels Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants