🐛 bug: 비활성화 날짜 저장 시 시간도 함께 저장하도록 변경한다

[viiviii]

Fixed: #102

변경

• 시간을 함께 저장한다
  • DB: date → timestamp
  • 어플리케이션: LocalDate -> Instant
• 활성화 기준을 지금이 아닌 n분 뒤 시간으로 한다
  • 이유: 1분 뒤 멤버십이 종료될 경우 시청 불가능한데 짧은 시간 내에 컨텐츠를 고르긴 힘들다
• 활성화 기준 시간을 도메인이 아닌 서비스 레이어가 알게 한다

메모

ZonedDateTime 대신 Instant 사용 이유

처음엔 ZonedDateTime을 사용했으나 아래의 테스트가 하나가 실패했다

ZonedDateTime은 내부적으로 ZoneId.systemDefault()를 사용하여 zoneId를 파악한 뒤 toString()에서 포함하기 때문이다.

ZonedDateTime은 zone을 포함한 텍스트의 parsing도 가능하고, 로그 같은 걸 확인할 때도 해당 서버 설정에 맞춰 보여주는 것이 편할 수 있다곤 생각하는데, 나에겐 별로 필요하진 않아서 패스했다.

그 다음에 찾아본 Instant가 내가 원하던 거 였다.

처음엔 1970-01-01T00:00:00Z 이후의 초를 나타낸다고 해서 long 타입으로 사용할까봐 패스 했었는데 toString()이 내가 원하던 ISO-8601 방식으로 오버라이딩 되어있었다.

Instant와 UTC

• Instant는 Java Time-Scale를 정의한다
  • 이것은 하루를 86,400초로 나눈다
  • 현재 2개의 segment로 나눠져 있는데 1972년 11월 3일 이전은 UT1(그리니치 태양시) 기준이고, 그 이후는 UTC(윤초 포함)이다.
  • UTC 세그먼트는 윤초 포함 UTC지만 윤초를 고려할 필요가 없다. 이건 무슨 말일까?
  • 원래 윤초가 반영되면 +- 1초로 조정되는데(86,399 ~ 86,401 SI) 23:59:59 다음을 00:00:00이 아닌 23:59:60과 같이 만드는 것 같다.
  • 하지만 Java Time-Scale은 하루를 86,400초로 나누기 때문에 윤초가 있으나 없으나 항상 86,400초를 유지한다
  • 이게 가능한 이유는 UTC with Smoothed Leap Seconds((UTC-SLS) 표준을 사용하기 떄문이다.
  • 윤초 때문에 2012년도 적용 당시엔 많은 기업에서 문제가 발생했다고 하고 이를 해결하기 위한 방식이 윤초를 위와 같이 명시적으로 +-하는 것이 아닌 특정 시간대에 조금씩 증감하여 값을 보정하는 것이다.
  • UTC-SLS와 구글 leap smear가 이 아이디어인데 둘은 비슷하나 차이가 있다고 한다
• 윤초
  • 세슘 동위원소의 진동수(초당 91억9천263만1천770회)를 기준으로 삼는 원자시와 실제 지구 자전에 의한 천문시 사이의 오차를 맞추기 위한 것이다
  • 2035년까지 폐지될 예정이다
• 추가 링크
  • Java의 날짜와 시간 API(D2)
  • 2012년에 비트윈 서버를 마비시켰던 윤초 이야기(타다)

PostgreSQL과 timezone

처음엔 모르고 어플리케이션에서 ZonedTimeZone의 값을 UTC로 변경하여 DB에 저장했었다.

하지만, 테스트가 실패해서 디버깅 하다가 실행 애플리케이션에 따라 값이 다르다는 걸 알게됐다.

For timestamp with time zone, the internally stored value is always in UTC (Universal Coordinated Time, traditionally known as Greenwich Mean Time, GMT). An input value that has an explicit time zone specified is converted to UTC using the appropriate offset for that time zone. If no time zone is stated in the input string, then it is assumed to be in the time zone indicated by the system's TimeZone parameter, and is converted to UTC using the offset for the timezone zone.

When a timestamp with time zone value is output, it is always converted from UTC to the current timezone zone, and displayed as local time in that zone

문서를 찾아보니 내부적으로는 UTC로 저장하고 출력 시에 해당 zone의 현지 시각으로 변환된다고 한다.

Bean과 static

now 메서드의 테스트를 위해 now(Clock...) api를 사용하여 clock을 DI 받도록 변경했다. 아직은 따로 클래스를 만들 필요가 없을 것 같아 메서드로 주입받게 했는데 순환참조 에러가 발생한다. 생각해보면 해당 클래스를 bean으로 만들어야 하는데 그 인스턴스의 메서드를 참조하는 것이었다.

그래서 inner class를 만들까 하다가 Bean 문서에서 static 문구를 읽고 따라했음