Remove Java9VersionSpecific clock implementation

Author: jack-berg

sdk/all/src/test/java/io/opentelemetry/sdk/common/SystemClockTest.java

@@ -8,26 +8,9 @@
 import static org.assertj.core.api.Assertions.assertThat;
 
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.condition.DisabledOnJre;
-import org.junit.jupiter.api.condition.EnabledOnJre;
-import org.junit.jupiter.api.condition.JRE;
 
-// This test is placed in the all artifact instead of the common one so it uses the dependency jar
-// instead of the classes directly, which allows verifying mrjar behavior.
 class SystemClockTest {
 
-  @EnabledOnJre(JRE.JAVA_8)
-  @Test
-  void now_millisPrecision() {
-    // If we test many times, we can be fairly sure we didn't just get lucky with having a rounded
-    // result on a higher than expected precision timestamp.
-    for (int i = 0; i < 100; i++) {
-      long now = SystemClock.getInstance().now();
-      assertThat(now % 1000000).isZero();
-    }
-  }
-
-  @DisabledOnJre(JRE.JAVA_8)
   @Test
   void now_microsPrecision() {
     // If we test many times, we can be fairly sure we get at least one timestamp that isn't
@@ -42,17 +25,6 @@ void now_microsPrecision() {
     assertThat(numHasMicros).isNotZero();
   }
 
-  @Test
-  void now_lowPrecision() {
-    // If we test many times, we can be fairly sure we didn't just get lucky with having a rounded
-    // result on a higher than expected precision timestamp.
-    for (int i = 0; i < 100; i++) {
-      long now = SystemClock.getInstance().now(false);
-      assertThat(now % 1000000).isZero();
-    }
-  }
-
-  @DisabledOnJre(JRE.JAVA_8)
   @Test
   void now_highPrecision() {
     // If we test many times, we can be fairly sure we get at least one timestamp that isn't

sdk/common/build.gradle.kts

@@ -10,8 +10,6 @@ apply<OtelVersionClassPlugin>()
 description = "OpenTelemetry SDK Common"
 otelJava.moduleName.set("io.opentelemetry.sdk.common")
 
-val mrJarVersions = listOf(9)
-
 dependencies {
   api(project(":api:all"))
 
@@ -23,61 +21,9 @@ dependencies {
   testImplementation("com.google.guava:guava-testlib")
 }
 
-for (version in mrJarVersions) {
-  sourceSets {
-    create("java$version") {
-      java {
-        setSrcDirs(listOf("src/main/java$version"))
-      }
-    }
-  }
-
-  tasks {
-    named<JavaCompile>("compileJava${version}Java") {
-      sourceCompatibility = "$version"
-      targetCompatibility = "$version"
-      options.release.set(version)
-    }
-  }
-
-  configurations {
-    named("java${version}Implementation") {
-      extendsFrom(configurations["implementation"])
-    }
-  }
-
-  dependencies {
-    // Common to reference classes in main sourceset from Java 9 one (e.g., to return a common interface)
-    add("java${version}Implementation", files(sourceSets.main.get().output.classesDirs))
-  }
-}
-
 tasks {
-  withType(Jar::class) {
-    val sourcePathProvider = if (name.equals("jar")) {
-      { ss: SourceSet? -> ss?.output }
-    } else if (name.equals("sourcesJar")) {
-      { ss: SourceSet? -> ss?.java }
-    } else {
-      { _: SourceSet -> project.objects.fileCollection() }
-    }
-
-    for (version in mrJarVersions) {
-      into("META-INF/versions/$version") {
-        from(sourcePathProvider(sourceSets["java$version"]))
-      }
-    }
-    manifest.attributes(
-      "Multi-Release" to "true",
-    )
-  }
-
   test {
     // For checking version number included in Resource.
     systemProperty("otel.test.project-version", project.version.toString())
   }
-
-  check {
-    dependsOn(testing.suites)
-  }
 }

sdk/common/src/main/java/io/opentelemetry/sdk/internal/JavaVersionSpecific.java

@@ -22,6 +22,8 @@
 
 package io.opentelemetry.sdk.internal;
 
+import java.time.Clock;
+import java.time.Instant;
 import java.util.concurrent.TimeUnit;
 import java.util.logging.Level;
 import java.util.logging.Logger;
@@ -59,6 +61,7 @@ String name() {
 
   /** Returns the number of nanoseconds since the epoch (00:00:00, 01-Jan-1970, GMT). */
   public long currentTimeNanos() {
-    return TimeUnit.MILLISECONDS.toNanos(System.currentTimeMillis());
+    Instant now = Clock.systemUTC().instant();
+    return TimeUnit.SECONDS.toNanos(now.getEpochSecond()) + now.getNano();
   }
 }