✨ [Feature/vote] implement save vote API #79

linirini · 2025-03-25T18:59:09Z

#️⃣ 연관된 이슈

[FEATURE] 투표하기 API #5

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

사용자가 찬성 투표한 후보지 목록을 requestBody로 받아서 저장합니다.
요청에 포함되지 않은 후보지는 반대로 판단합니다.
재투표 시 기존 투표 정보를 제거한 후, 새로운 투표 정보를 저장합니다.

설계

결과 조회/갱신(재투표) 시 연산 최적화를 위해 2가지 방식으로 투표 정보를 관리합니다.

후보지 기준 구조 (찬성/반대 모두 관리)

room:{roomId}:candidate:{stationId}:agree → Set[memberId]
room:{roomId}:candidate:{stationId}:disagree → Set[memberId]

사용자 기준 구조 (사용자별 후보지 투표 상태 관리)

room:{roomId}:member:{memberId} → Hash[stationId: "agree"/"disagree"]

이에 따른 투표 결과 저장 로직

사용자 투표 내역이 존재한다면
i) 사용자 기존 투표 내역 조회 -> HGETALL room:{roomId}:member:{memberId}
ii) 조회한 데이터 기반으로 후보지에서 사용자 ID 제거 -> SREM room:{roomId}:candidate:{stationId}:{agree/disagree} {memberId}
iii) 사용자 기존 투표 내역 정보 삭제 -> DEL room:{roomId}:member:{memberId}
사용자 투표 내역 저장
i) 사용자 투표 내역 저장 -> HSET room:{roomId}:member:{memberId} {stationId} {agree/disagree}
ii) 투표 결과에 따라 후보지에 사용자 ID 추가 -> SREM room:{roomId}:candidate:{stationId}:{agree/disagree} {memberId}

📸 스크린샷 (선택)

💬 리뷰 요구사항(선택)

이전 작업 내용이 필요하여 해당 pr의 작업 내용만 모아 보시려면 링크를 눌러주세요!

⚙️ [chore/develop] update .gitignore file

…ringBootTest for testing redis

…vote

…into feature/save-vote

…emberId

…omId

.../src/main/java/com/kok/kokapi/station/adapter/out/persistence/StationPersistenceAdapter.java

yunyoung1819 · 2025-03-26T14:47:53Z

kok-api/src/main/java/com/kok/kokapi/vote/adapter/in/web/VoteController.java

+        @RequestBody VoteRequest voteRequest
+    ) {
+        voteFacadeService.createVote(roomId, memberId, voteRequest);
+        return ResponseEntity.ok(ApiResponseDto.success(null));


응답값이 200인 경우만 처리되고 있는데 404, 500 등에 대한 반환값 처리도 구현되면 좀더 안전한 코드가 될 것 같습니다.

kok-api/src/main/java/com/kok/kokapi/vote/adapter/out/persistence/VoteQueryRedisAdapter.java

yunyoung1819 · 2025-03-26T14:56:59Z

kok-api/src/main/java/com/kok/kokapi/vote/adapter/out/persistence/VoteCommandRedisAdapter.java

+    @Override
+    public void deleteAllByRoomIdAndMemberId(String roomId, String memberId) {
+        String key = getMemberVoteKey(roomId, memberId);
+        redisTemplate.delete(key);


Redis에 save, delete 등으로 데이터 변경 작업을 처리하고 있는데 DB에서처럼
trasaction이 걸려있고 실패 시에 Rollback 처리가 되지 않기 때문에 예외 처리 로직이 추가된다면
좀더 안전한 코드가 될 것 같아요. 삭제할때 redis 장애가 발생한 상태 또는 key가 만료된 상태 등. 여러가지 이유로 인해 삭제가 실패하는 경우에 대해 retry 또는 로그를 출력한다던가 등의 로직이 추가되면 좋을 것 같아요 😄

Redis 오류에 의해 발생할 수 있는 에러를 파악하고, 각 에러에 따라 처리 방법에 대해 다음과 같이 정리해보았어요!

예외 종류 로그 레벨 클라이언트에 throw throw 시 예외 유형 클라이언트에 알리는 이유 retry 여부 retry 근거

RedisConnectionFailureException error Yes InternalServerError Redis 자체가 죽은 상태로, 정상 처리가 불가능 Yes 일시적 네트워크 장애, Redis 재시작 등의 가능성 있음

RedisSystemException error No (조회) / Yes (저장/삭제/수정) InternalServerError 저장 실패 시 데이터 불일치 우려. 수정/갱신 실패 시 데이터 오염 우려. 조회라면 fallback 가능하므로 무시 가능 No 직렬화 오류는 코드나 데이터 구조의 문제(개발자 잘못)이므로 반복해도 해결되지 않음

DataAccessException warn No - 대부분 명령어 수준의 오류로 fallback 처리 가능 No 일시적 오류거나 무시 가능 수준. 반복 시 큰 효과 없음

Exception (그 외 모든 예외) error Yes InternalServerError 알 수 없는 예외는 반드시 추적 및 알림 필요 No 원인 불명으로 재시도 전략 수립이 어려움

반복적인 try-catch에 대하여 탬플릿 메서드 패턴을 적용하여 RedisExecutor라는 util 클래스를 분리했어요!

예외 처리 커밋: 88dfe47

retry 커밋: 8ca1527

yunyoung1819

리뷰가 늦어진 것 같아서 빠른 리뷰를 위해 새벽이지만 리뷰 남겨놓았습니다. 고생하셨어요 👍

kok-api/src/main/java/com/kok/kokapi/vote/application/service/CandidateService.java

…o feature/save-vote

yunyoung1819

고생하셨습니다! 작업한 부분이 많으셔서 고생 많이 하셨을 것 같아요 👍
우선 이렇게 머지하고, 다른 redis를 쓰는 곳에도 공통으로 적용되면 좋은 부분이라서 나중에 redisTemplate을 한번 감싸서 예외 처리 로직을 만들어두는 방식 등도 논의해보아도 좋을 것 같아요 😄

yunyoung1819 and others added 28 commits February 11, 2025 23:42

Merge pull request #24 from depromeet/develop

7f4dfe3

⚙️ [chore/develop] update .gitignore file

✨ feat: create domain for voting

96ca485

✅ test: create RedisDatabaseCleaner And change RepositoryTest into Sp…

e59015f

…ringBootTest for testing redis

🔧 config: fix ddl type error

9614e56

✨ feat: implement port for saving candidate

567cff1

✨ feat: implement port for saving votes

72276ea

✨ feat: implement port for loading candidates

b59d4cd

✨ feat: implement checking if candidates exists

df0ba6a

✨ feat: implement getting recommended stations from cache

5c44a26

Merge branch 'develop', remote-tracking branch 'origin' into feature/…

19511ac

…vote

♻️ refactor: move getting recommended stations from adapter to service

dc41579

❇️ feat: implement get candidate usecase

dee7f45

❇️ feat: implement get candidate Facade service and Controller

9da74b5

❇️ feat: add updating route names into "1호선"

d8b37e6

❇️ feat: save votes by member

ffe356d

✅ test: test for saving votes by candidate

dd7c989

✨ feat: implement saving vote usecase

1e6455f

❇️ feat: implement deleting cache of recommended Stations

15af6ff

❇️ feat: delete cached recommended stations after saving candidates

b1444c6

Merge branch 'feature/vote' of https://github.com/depromeet/kok-server …

034c8da

…into feature/save-vote

❇️ feat: implement creating Vote Controller and facade service

bb4f1fa

❇️ feat: implement loadVotePort checking if votes exists by member

b3be90b

♻️ refactor: refactor key-format for candidate

bad648c

✨ feat: implement delete vote port deleting votes by candidate

7d74084

✨ feat: implement delete vote port deleting all votes by roomId and m…

6dbe01a

…emberId

♻️ refactor: rename key constant

11db9bd

✨ feat: implement load vote port finding all votes by memberId And ro…

945f42c

…omId

✨ feat: add logic deleting previous vote data on saveVoteUsecase

0aab750

linirini added ✨ Feature new feature 🥇 Urgency 1 긴급도 높음 labels Mar 25, 2025

linirini added this to the 2차 MVP milestone Mar 25, 2025

linirini requested review from yunyoung1819 and minseokey March 25, 2025 18:59

linirini self-assigned this Mar 25, 2025

linirini linked an issue Mar 25, 2025 that may be closed by this pull request

[FEATURE] 투표하기 API #5

Closed

2 tasks

linirini changed the title ~~✨ [Feature/Vote] implement save vote API~~ ✨ [Feature/vote] implement save vote API Mar 25, 2025

linirini added 6 commits March 26, 2025 04:00

Merge branch 'feature/vote' into feature/save-vote

2c5f1b8

Merge branch 'develop' into feature/vote

e70c739

Merge branch 'develop' into feature/vote

b659c98

♻️ refactor: merge conflict

d412b39

Merge branch 'feature/vote' into feature/save-vote

52a715c

♻️ refactor: move package for port

1f95472

yunyoung1819 reviewed Mar 26, 2025

View reviewed changes

.../src/main/java/com/kok/kokapi/station/adapter/out/persistence/StationPersistenceAdapter.java Outdated Show resolved Hide resolved

yunyoung1819 reviewed Mar 26, 2025

View reviewed changes

kok-api/src/main/java/com/kok/kokapi/vote/adapter/out/persistence/VoteQueryRedisAdapter.java Outdated Show resolved Hide resolved

yunyoung1819 reviewed Mar 26, 2025

View reviewed changes

minseokey approved these changes Mar 27, 2025

View reviewed changes

kok-api/src/main/java/com/kok/kokapi/vote/application/service/CandidateService.java Show resolved Hide resolved

linirini added 6 commits March 29, 2025 00:09

✨ feat: throw exception if cannot parse to Long

57ecdc7

✨ feat: handle redis error

88dfe47

✨ feat: implement retry for redis connection failure error

8ca1527

Merge branch 'develop' into feature/save-vote

11f0c3b

♻️ refactor: implement getting saved candidates by roomID

2bd5c74

Merge branch 'develop' of https://github.com/depromeet/kok-server int…

806f997

…o feature/save-vote

linirini requested a review from yunyoung1819 March 29, 2025 07:06

yunyoung1819 approved these changes Mar 29, 2025

View reviewed changes

linirini merged commit 9c44744 into develop Mar 29, 2025
2 checks passed

linirini deleted the feature/save-vote branch March 29, 2025 07:18

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

✨ [Feature/vote] implement save vote API #79

✨ [Feature/vote] implement save vote API #79

linirini commented Mar 25, 2025 •

edited

Loading

yunyoung1819 Mar 26, 2025

yunyoung1819 Mar 26, 2025 •

edited

Loading

linirini Mar 28, 2025 •

edited

Loading

yunyoung1819 left a comment •

edited

Loading

yunyoung1819 left a comment

예외 종류	로그 레벨	클라이언트에 throw	throw 시 예외 유형	클라이언트에 알리는 이유	retry 여부	retry 근거
RedisConnectionFailureException	error	Yes	InternalServerError	Redis 자체가 죽은 상태로, 정상 처리가 불가능	Yes	일시적 네트워크 장애, Redis 재시작 등의 가능성 있음
RedisSystemException	error	No (조회) / Yes (저장/삭제/수정)	InternalServerError	저장 실패 시 데이터 불일치 우려. 수정/갱신 실패 시 데이터 오염 우려. 조회라면 fallback 가능하므로 무시 가능	No	직렬화 오류는 코드나 데이터 구조의 문제(개발자 잘못)이므로 반복해도 해결되지 않음
DataAccessException	warn	No	-	대부분 명령어 수준의 오류로 fallback 처리 가능	No	일시적 오류거나 무시 가능 수준. 반복 시 큰 효과 없음
Exception (그 외 모든 예외)	error	Yes	InternalServerError	알 수 없는 예외는 반드시 추적 및 알림 필요	No	원인 불명으로 재시도 전략 수립이 어려움

✨ [Feature/vote] implement save vote API #79

✨ [Feature/vote] implement save vote API #79

Conversation

linirini commented Mar 25, 2025 • edited Loading

#️⃣ 연관된 이슈

📝 작업 내용

설계

후보지 기준 구조 (찬성/반대 모두 관리)

사용자 기준 구조 (사용자별 후보지 투표 상태 관리)

이에 따른 투표 결과 저장 로직

📸 스크린샷 (선택)

💬 리뷰 요구사항(선택)

yunyoung1819 Mar 26, 2025

Choose a reason for hiding this comment

yunyoung1819 Mar 26, 2025 • edited Loading

Choose a reason for hiding this comment

linirini Mar 28, 2025 • edited Loading

Choose a reason for hiding this comment

yunyoung1819 left a comment • edited Loading

Choose a reason for hiding this comment

yunyoung1819 left a comment

Choose a reason for hiding this comment

linirini commented Mar 25, 2025 •

edited

Loading

yunyoung1819 Mar 26, 2025 •

edited

Loading

linirini Mar 28, 2025 •

edited

Loading

yunyoung1819 left a comment •

edited

Loading