🚧 장애 로그. 매해 주 차(week) 계산의 오류

이슈 발생

2018년 12월 30일 일요일 새벽이었습니다.

당시 서비스 중이던 게임에는 매주 사용자들끼리 경쟁해서 월요일을 기점으로 보상을 주고 초기화하는 시스템이 있었습니다. 그런데 월요일도 아니고 일요일 새벽에 해당 시스템이 초기화 돼버리는 문제가 발생했습니다.

마침, 커뮤니티를 모니터링 중이던 운영팀이 문제를 실시간으로 확인해서 랭킹이 리셋됐다는 장애 보고를 전파했고, 당시 집이 가장 가까웠던 제가 먼저 회사로 달려가서 원인 분석을 시작했습니다. 문제는 생각보다 간단한 부분에서 발생하고 있었습니다.

사용자들이 경쟁한 기록을 DB에 남길 때 연도와 그해의 주 차를 조합한 값을 키값으로 사용해 기록하고 있었는데, 12월 30일로 넘어가면서 주 차가 리셋되고 새해 첫 주로 계산을 해서 201852가 아닌 201800을 키값으로 계산하고 있었던 것입니다.

벌써 출시한 지 4주년이 지난 게임이었는데, 매년 첫 주에 같은 장애가 발생했던 것인가? 하고 봤더니 비슷한 문제가 매년 있어서 운영팀에서 모니터링하고 있었던 것이었습니다. 장애 원인을 파악하지도 못하고 수정도 못한 채로 계속 서비스해 왔던 것이죠.

원인 파악

우선 이해를 돕기 위한 코드를 보여드리겠습니다.

Calendar calendar = Calendar.getInstance();
calendar.set(2018, 11, 30); // 2018-12-30 월은 0부터 시작하므로 12월은 11
int year = calendar.get(Calendar.YEAR);  // 년
int week = calendar.get(Calendar.DAY_OF_WEEK); // 요일 [1-7]
int weekOfYear = calendar.get(Calendar.WEEK_OF_YEAR); // 주 차
if (week == 1) {
  weekOfYear--; // 월요일에 리셋이므로 일요일이면 weekOfYear 값을 하나 감소 시킴
}
String yearWeekParam = String.format("%d%02d", year, weekOfYear);
System.out.println(yearWeekParam); // 201800

그 해 마지막 주가 새 해와 겹쳐있을 때는 새 해 첫 주에 편입이 되는데, 이 코드에서는 그 연도가 바뀌었을 때 처리가 없고 새해 첫 주로 판단하여 1이 된 weekOfYear 값을 감소시켜 버려서 처리가 잘못되고 있었습니다.

• 2018년 12월 29일 토요일(52주 차): 201852(의도) -> 201852(일치)
• 2018년 12월 30일 일요일(52주 차): 201852(의도) -> 201800(오류)

문제 수정

코드를 고치는 것은 간단했습니다. 일주일의 시작을 월요일로 변경하면 됐으니까요.

Calendar calendar = Calendar.getInstance();
calendar.setFirstDayOfWeek(Calendar.MONDAY); // 일주일의 시작을 월요일로 설정
calendar.set(2018, 11, 30);
int year = calendar.get(Calendar.YEAR);
// ... 삭제
int weekOfYear = calendar.get(Calendar.WEEK_OF_YEAR);
// ... 삭제
String yearWeekParam = String.format("%d%02d", year, weekOfYear);
System.out.println(yearWeekParam); // 201852

이것으로 2018년 12월 30일에 발생한 해당 문제는 해결됐습니다. 하지만 다른 문제가 더 있습니다.

• 2018년 12월 31일 월요일(1주 차): 201901(의도) -> 201801(오류)
• 2019년 1월 1일 일요일(1주 차): 201901(의도) -> 201901(일치)

2018년 12월 31일이 되었을 때 새해가 되어 주 차가 1이 되었지만, 연도는 바뀌지 않았기 때문에 키값이 201801이 되기 때문입니다. 이 문제는 아래 코드와 같이 주 차와 월을 검사해서 연도를 증가시켜 주는 것으로 해결했습니다.

Calendar calendar = Calendar.getInstance();
calendar.setFirstDayOfWeek(Calendar.MONDAY);
calendar.set(2018, 11, 31);
int year = calendar.get(Calendar.YEAR);
int month = calendar.get(Calendar.MONTH);
int weekOfYear = calendar.get(Calendar.WEEK_OF_YEAR);
if (weekOfYear == 1 && month != 0) { // 새해 첫 주인데 1월이 아닌 경우 연도 값을 증가
  year++;
}
String yearWeekParam = String.format("%d%02d", year, weekOfYear);
System.out.println(yearWeekParam); // 201901

그런데, 여기서 의문이 생겼습니다

2018년 12월 31일이 201901이 되는 것이 맞을까요? 2018년의 마지막 주 차니까 201853이 되는 것이 맞을까요?

실제 개발팀에서도 의견 충돌이 잠깐 발생했었는데, 사용 중인 Calendar 라이브러리에서 새해 첫 주는 1월 1일이 포함된 주로 하기 때문에 계산의 편의성을 고려해서 그해 마지막 주는 새해 첫 주에 편입되는 것으로 결론이 났습니다.

라이브러리마다 구현이 다르긴 하더군요. 나중에 찾아본 moment.js에서는 1월 1일이 지난해 마지막 주 차에 겹치는 경우 마지막 주 차에 편입되고 첫 주 차가 밀리도록 구현이 되어있었습니다.

결론

라이브러리에 종속된 방식으로 기능을 구현해야 하는 경우 그 라이브러리에 대한 충분한 파악이 필요하다는 점과 다양한 테스트 케이스가 필요하다는 점 꼭 기억해야겠습니다.

애초에 기준이 되는 일로부터 주 차를 카운트하는 방식이었다면 이런 문제도 없었을 것이고 더 구현하기 쉽고 좋았을 것 같습니다. 저는 실제로 이 경험 덕분에 이후 이와 비슷한 초기화 방식을 구현할 때는 서비스 시작일을 기준으로 키값을 카운트하는 것으로 구현하고 있습니다.

번외

글을 작성하면서 좀 더 조사해 보니 ISO에서 표준으로 지정한 주 차 계산 방법이 있었습니다. 자세한 것은 링크를 참고 바랍니다. https://en.wikipedia.org/wiki/ISO_week_date#First_week