[Refector] event 관리 코드 수정

[username1103]

체크 리스트

• 적절한 제목으로 수정했나요?
• 관련된 이슈와 연결 시켰나요?
• Target Branch를 올바르게 설정했나요?
• Label을 알맞게 설정했나요?

작업 내역

• NestJS에서 제공하는@nestjs/event-emitter 는 내부적으로 event emitter2를 사용하고 있다. 기존에도 정상적으로 동작하긴 했으나 사용법이 적절하지 못해 수정했다.

기존에는 아래와 같이 비동기함수인 event listenr를 단순히 emit해 우회적으로 동작시켰다.

web13-moyeomoyeo/backend/src/app/group-article/group-article.service.ts

Lines 95 to 98 in 399ce79

	this.eventEmitter.emit(
	'group.succeed',
	new GroupSucceedEvent(groupArticle),
	);

web13-moyeomoyeo/backend/src/app/notification/notification.listener.ts

Lines 27 to 30 in 399ce79

	@OnEvent('group.succeed')
	async handleGroupSucceedEvent(event: GroupSucceedEvent) {
	const { groupArticle } = event;

async옵션을 통해 setImmediate를 사용해 비동기적으로 호출될 수 있도록 아래와 같이 변경했다.

web13-moyeomoyeo/backend/src/app/group-article/group-article.service.ts

Lines 95 to 98 in 6f41f44

	await this.eventEmitter.emitAsync(
	GroupSucceedEvent.event,
	new GroupSucceedEvent(groupArticle),
	);

web13-moyeomoyeo/backend/src/app/notification/notification.listener.ts

Lines 27 to 30 in 6f41f44

	@OnEvent(GroupSucceedEvent.event, { async: true })
	async handleGroupSucceedEvent(event: GroupSucceedEvent) {
	const { groupArticle } = event;

문제 상황과 해결

• 현재 async는 true로 두어 비동기적으로 setImmediate을 사용하도록 되어있다. { nextTick: true } 옵션을 주어 next tick에 바로 실행되도록 할수도 있다. setImmediate는 이벤트 루프에 check phase에 실행되고 nextTick은 nextTickQueue에 담겨 다음 페이즈로 넘어가기전에 실행된다.

두 옵션 모두 비동기적으로 동작하게 할 수 있는데 어떤 옵션을 주는게 좋을까?

비고

• 참고했던 링크 등 참고 사항을 적어주세요. 코드 리뷰하는 사람이 참고해야 하는 내용을 자유로운 형식으로 적을 수 있습니다.

[kong430]

👏🏻