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

[SIX-274] 채팅방이 중복되어서 생성되지 않도록 하고, 인덱스를 추가하였습니다. #127

Merged
merged 3 commits into from
Nov 30, 2023

Conversation

bombo-dev
Copy link
Member

🖊️ 1. Changes

  • 채팅방이 중복되어서 생기는 이슈가 있어서 중복 생성되지 않도록 수정하였습니다.
  • 인덱스를 3개를 걸었습니다.
    멀티 유니크를 걸어서 (mission_chat_room_id, user_id)에 대한 유니크 키 멀티 인덱스
    user_id로 조회하는 경우가 많아 user_id 인덱스
    mission_chat_room_id로도 조회하는 경우가 있어 mission_chat_room_id 에도 인덱스를 걸어두었습니다.

🖼️ 2. Screenshot

❗️ 3. Issues

😌 4. To Reviewer

✅ 5. Plans

🙌 6. Checklist

  • - 통합 테스트를 수행해보셨나요?
  • - 혹시 컨벤션에 맞지 않게 작성하지 않았나요?
  • - Change와 Issue, Reviewer를 알아보기 쉽게 작성하였나요?

@bombo-dev bombo-dev self-assigned this Nov 30, 2023
Copy link

github-actions bot commented Nov 30, 2023

Test Results

  66 files    66 suites   8s ⏱️
242 tests 240 ✔️ 2 💤 0
252 runs  250 ✔️ 2 💤 0

Results for commit 012dade.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@legowww legowww left a comment

Choose a reason for hiding this comment

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

👍

Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug B 6 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 71 Code Smells

65.0% 65.0% Coverage
0.0% 0.0% Duplication

@bombo-dev bombo-dev merged commit 4ae7b3f into develop Nov 30, 2023
5 checks passed
@bombo-dev bombo-dev deleted the SIX-274 branch November 30, 2023 05:09
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.

3 participants