Skip to content

Conversation

@blu3fishez
Copy link
Collaborator

  • 작성자: 김찬우
  • 작성 날짜: 2024.11.27

✅ 체크리스트

  • 코드가 정상적으로 작동하는지 확인했습니다.
  • 주요 변경사항에 대한 설명을 작성했습니다.
  • 코드 스타일 가이드에 따라 코드를 작성했습니다.

🧩 작업 내용

  • nestjs-redis-om 모듈을 수정해서 오류를 해결했습니다.

📝 작업 상세 내역

패키지 출시

저 패키지 출시했어요. 축하해주세요.

들어가보기

MIT 라이선스에 따라 원저자 출처를 해서 출시를 했습니다.

패키지 출시를 해야했던 이유

아래 nestjs-redis-om 코드를 수정할 필요가 있었습니다.

const schema = new Schema(
  entityMetadata.schemaName,
  entityMetadata.fields,
  entityMetadata.schemaOptions,
);
const repository = new Repository(schema, dataSource);
await repository.createIndex();
return repository;

이부분에서 인덱스 생성 시, 이미 존재하거나 상태가 동일하지 않을 경우 에러처리를 하지 않던 것이 문제였습니다.

실제로 nestjs 를 위한 redis-om 래퍼가 없었고, 다른 사람들도 불편함을 느꼈을 것으로 판단했습니다.

그래서 이왕 하는 김에 실제로 배포까지 진행했습니다. 추후에 Schema 기능 중에 TTL 설정하는 것까지 지원해볼 생각입니다.

💬 다음 작업 또는 논의 사항

  • 스터디 기능을 위한 웹소켓 핸들러를 만들 생각입니다.

🐥 리뷰 받고 싶은 포인트

  • 저 패키지 출시했어요 축하해줘요 🐥🐥

- 외부라이브러리를 고쳐서 해결했습니다.
@blu3fishez blu3fishez added the 🐛 Bug Fix 해충 제거 label Nov 27, 2024
@blu3fishez blu3fishez requested a review from twalla26 November 27, 2024 11:46
@blu3fishez blu3fishez self-assigned this Nov 27, 2024
Copy link
Collaborator

@twalla26 twalla26 left a comment

Choose a reason for hiding this comment

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

와우.... 정말 제대로된 문제 해결 경험을 쌓으셨군요.... 역시 고수.... 👍👍
정말 고생하셨습니다..!!!

@blu3fishez blu3fishez merged commit 7ecc44c into boostcampwm-2024:dev Nov 27, 2024
@blu3fishez blu3fishez changed the title [Fix] index already exists 문제 해결 (패키지 출시함) [Fix] index already exists 문제 해결 Nov 27, 2024
@ShipFriend0516
Copy link
Member

패키지 배포.. 신기하네요 고수👏

@blu3fishez blu3fishez deleted the fix/nestjs-redis-om branch September 1, 2025 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Bug Fix 해충 제거

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants