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

Bump up spring boot 3 #124

Merged
merged 20 commits into from
Jun 18, 2024
Merged

Bump up spring boot 3 #124

merged 20 commits into from
Jun 18, 2024

Conversation

sohyun-ku
Copy link
Contributor

related to #122

@sohyun-ku sohyun-ku self-assigned this May 28, 2024
@sohyun-ku sohyun-ku requested review from junoyoon and a team as code owners May 28, 2024 02:04
Copy link

github-actions bot commented May 28, 2024

Scavenger Test Results

161 files  161 suites   1m 28s ⏱️
267 tests 255 ✅ 12 💤 0 ❌
285 runs  273 ✅ 12 💤 0 ❌

Results for commit 8e9bc8e.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@taeyeon-Kim taeyeon-Kim left a comment

Choose a reason for hiding this comment

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

Replace io.github.microutils:kotlin-logging-jvm with https://github.com/oshai/kotlin-logging

Copy link
Contributor

@taeyeon-Kim taeyeon-Kim left a comment

Choose a reason for hiding this comment

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

Delete spring-cloud if not needed

Copy link
Contributor

@taeyeon-Kim taeyeon-Kim left a comment

Choose a reason for hiding this comment

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

bump up spring-boot-starter-data-jdbc of scavenger-entity to 3.2.4

scavenger-collector/build.gradle.kts Outdated Show resolved Hide resolved
Comment on lines +72 to +75
fun toInstantMicro(instant: Instant): Instant {
val epochMicro = (instant.epochSecond * 1_000_000) + (instant.nano / 1000)
return Instant.ofEpochSecond(epochMicro / 1_000_000, (epochMicro % 1_000_000) * 1000)
}
Copy link
Contributor Author

@sohyun-ku sohyun-ku Jun 3, 2024

Choose a reason for hiding this comment

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

After switching to jdk17, when using Instant.now(), I modified it to use micro because it tends to use different time precision depending on the OS.
That's why the test was previously crashed.

https://stackoverflow.com/questions/74781495/changes-in-instant-now-between-java-11-and-java-17-in-aws-ubuntu-standard-6-0

MySQL permits fractional seconds for TIME, DATETIME, and TIMESTAMP values, with up to microseconds (6 digits) precision.

https://dev.mysql.com/doc/refman/8.4/en/date-and-time-type-syntax.html

Copy link
Contributor

Choose a reason for hiding this comment

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

that's new to me, thanks 😄

@@ -50,12 +50,11 @@ dependencies {
configure<DependencyManagementExtension> {
imports {
mavenBom(org.springframework.boot.gradle.plugin.SpringBootPlugin.BOM_COORDINATES)
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe this is also a configuration we need??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think yes.
It allows us to leverage the Spring Boot BOM to simplify dependency management, ensure version consistency, and reduce the risk of version conflicts in our Spring Boot project.

The SpringBootPlugin class provides a BOM_COORDINATES constant that can be used to import the bom without having to know its group ID, artifact ID, or version.
https://docs.spring.io/spring-boot/docs/3.2.4/gradle-plugin/reference/htmlsingle/#managing-dependencies.dependency-management-plugin.using-in-isolation

Copy link
Contributor

@taeyeon-Kim taeyeon-Kim left a comment

Choose a reason for hiding this comment

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

👍 👍

@sohyun-ku sohyun-ku force-pushed the feature/bump-up-spring-boot-3 branch from 4078520 to 8e9bc8e Compare June 18, 2024 01:37
@sohyun-ku sohyun-ku merged commit aada473 into develop Jun 18, 2024
6 checks passed
@sohyun-ku sohyun-ku deleted the feature/bump-up-spring-boot-3 branch June 18, 2024 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants