fix(sql): Adjust comment and change to MICROS for tests

spinnaker · Nov 15, 2024 · f379420 · f379420
1 parent dbe8e0b
commit f379420
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...re-test/src/main/kotlin/com/netflix/spinnaker/keel/persistence/ArtifactRepositoryTests.kt b/...re-test/src/main/kotlin/com/netflix/spinnaker/keel/persistence/ArtifactRepositoryTests.kt
@@ -653,7 +653,7 @@ abstract class ArtifactRepositoryTests<T : ArtifactRepository> : JUnit5Minutests
       // We truncate this since we're using a serialization to java that reduces the level of precision
       // and later comparisons break otherwise.  This is needed to work with generated columns in
       // certain databases.  See the PrecisionSqlSerializer class for more info
-      val createdAt = Instant.now().truncatedTo(ChronoUnit.MILLIS)
+      val createdAt = Instant.now().truncatedTo(ChronoUnit.MICROS)
 
       before {
         subject.register(versionedSnapshotDebian)

diff --git a/keel-core/src/main/java/com/netflix/spinnaker/keel/serialization/PrecisionSqlSerializer.java b/keel-core/src/main/java/com/netflix/spinnaker/keel/serialization/PrecisionSqlSerializer.java
@@ -22,7 +22,7 @@
  *   <li>https://stackoverflow.com/a/38042457 for more information.
  * </ul>
  *
- * <p>NOTE we're going to 3 digits, but could go to 6 if but NO larger than 6 due to <a
+ * <p>NOTE we're going to 6 digits to match <a
  * href="https://www.w3schools.com/sql/func_mysql_str_to_date.asp">Mysql str_to_date function
  * limits</a> on microseconds parsing. This is used in the SQL definition: <code>
  *  add column triggered_at datetime(3) generated always as (str_to_date(json->>'$.triggeredAt', '%Y-%m-%dT%T.%fZ'))