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

[chore] : JDK version migration 11 to 17 #372

Merged
merged 7 commits into from
Feb 20, 2024
Merged

[chore] : JDK version migration 11 to 17 #372

merged 7 commits into from
Feb 20, 2024

Conversation

devxb
Copy link
Member

@devxb devxb commented Feb 20, 2024

어떤 기능을 개발했나요?

JDK version을 11 에서 17로 올렸습니다.

어떻게 해결했나요?

  • Dockerfile에 jre-slim-11jdk-slim-17 로 변경
  • CI 의 jdk 11을 17로 변경
  • gradle.properties의 11을 17로 변경
  • RequestTimeUtil이 UTC Zone의 6-digit 시간을 반환하도록 수정

이슈 넘버

참고자료

Comment on lines 25 to 28
var current = Instant.now();
var formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd'T'HH:mm:ss.SSSSSSX")
.withZone(ZoneId.of("UTC"));
this.instant = Instant.parse(formatter.format(current));
Copy link
Member Author

Choose a reason for hiding this comment

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

변경이유

DB에서 TIMESTAMP(6) 로 저장하고있는데, Instant.now() 가 9-digit 날짜를 생성해서 DB데이터에 들어갔다가 나온 데이터 <-> 검증데이터 간의 테스트가 깨지는 이슈가 있었고, RequestTimeUtil이 6-digit 시간을 반환하도록 수정했습니다.

Copy link
Member

Choose a reason for hiding this comment

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

DB쪽 timestamp precision 올리는 건 어렵나?

Copy link
Member Author

Choose a reason for hiding this comment

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

될거같은데, 한번 flyway랑 @Entity 쪽 수정해서 CI 돌려보겠슴니다

@devxb devxb marked this pull request as ready for review February 20, 2024 05:16
Comment on lines +23 to +28
- name: Cache Gradle packages
uses: actions/cache@v3
with:
path: ~/.gradle/caches
key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }}
restore-keys: ${{ runner.os }}-gradle
Copy link
Member

Choose a reason for hiding this comment

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

setup-java에서 gradle cache 있었던 거 같은데 다른가 이거랑?

Copy link
Member Author

Choose a reason for hiding this comment

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

이게 다른 .yml에서 한번이상 cache하면 재사용되는데, 우리처럼 시간 오래지나서 cache한게 없으면 실패되더라고
이거 ciaks 독립적으로 재사용하려면 넣어줘야할거같아

Comment on lines 25 to 28
var current = Instant.now();
var formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd'T'HH:mm:ss.SSSSSSX")
.withZone(ZoneId.of("UTC"));
this.instant = Instant.parse(formatter.format(current));
Copy link
Member

Choose a reason for hiding this comment

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

DB쪽 timestamp precision 올리는 건 어렵나?

Copy link

@devxb devxb merged commit aac0bc9 into main Feb 20, 2024
4 checks passed
@devxb devxb deleted the devxb/iss-#371 branch February 20, 2024 07:02
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.

[chore] : JDK version 11 to 17
2 participants