WIP: Move to Java 17

.github/workflows/nebula-branch.yml

@@ -11,8 +11,8 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        # test against JDK 8
-        java: [ 8 ]
+        # test against JDK 17
+        java: [ 17 ]
     name: CI with Java ${{ matrix.java }}
     steps:
       - uses: actions/checkout@v1

.github/workflows/nebula-pr-functional-tests.yml

@@ -7,8 +7,8 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        # test against JDK 8
-        java: [ 8 ]
+        # test against JDK 17
+        java: [ 17 ]
     name: Functional tests with Java ${{ matrix.java }}
     steps:
       - uses: actions/checkout@v1

.github/workflows/nebula-publish.yml

@@ -13,10 +13,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v1
-      - name: Setup jdk 8
+      - name: Setup jdk 17
         uses: actions/setup-java@v1
         with:
-          java-version: 1.8
+          java-version: 17
       - name: Login to Docker Hub
         uses: docker/login-action@v1
         with:

.github/workflows/nebula-pull-request.yml

@@ -7,8 +7,8 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        # test against JDK 8
-        java: [ 8 ]
+        # test against JDK 17
+        java: [ 17 ]
     name: CI with Java ${{ matrix.java }}
     steps:
       - uses: actions/checkout@v1

.github/workflows/nebula-snapshot.yml

@@ -15,7 +15,7 @@ jobs:
       - name: Set up JDK
         uses: actions/setup-java@v1
         with:
-          java-version: 8
+          java-version: 17
       - name: Login to Docker Hub
         uses: docker/login-action@v1
         with:

build.gradle

@@ -81,8 +81,8 @@ allprojects {
 
 idea {
     project {
-        jdkName = "8"
-        languageLevel = "8"
+        jdkName = "17"
+        languageLevel = "17"
         vcs = "Git"
     }
 }
@@ -98,8 +98,8 @@ configure(javaProjects) {
 
     group = "com.netflix.${githubProjectName}"
 
-    sourceCompatibility = JavaVersion.VERSION_1_8
-    targetCompatibility = JavaVersion.VERSION_1_8
+    sourceCompatibility = JavaVersion.VERSION_17
+    targetCompatibility = JavaVersion.VERSION_17
 
     dependencyManagement {
         imports {
@@ -251,6 +251,7 @@ configure(javaProjects) {
             exceptionFormat = "full"
             events "PASSED", "FAILED", "SKIPPED"
         }
+        jvmArgs('--add-opens=java.base/java.lang.invoke=ALL-UNNAMED')
     }
 
     javadoc {

gradle.properties

@@ -21,7 +21,7 @@
 # Used in documentation and for including the Gradle plugin
 spring_boot_version=2.7.11
 spring_cloud_version=Hoxton.SR8
-google_guice_version=4.1.0
+google_guice_version=5.1.0
 spock_version=2.1-groovy-3.0
 
 ## Override Spring Boot versions

metacat-common/src/main/java/com/netflix/metacat/common/type/RowType.java

@@ -24,9 +24,9 @@
 import lombok.EqualsAndHashCode;
 import lombok.Getter;
 import lombok.NonNull;
+import org.checkerframework.checker.nullness.qual.Nullable;
 
 import javax.annotation.Nonnull;
-import javax.annotation.Nullable;
 import java.util.Collections;
 import java.util.List;
 

metacat-connector-hive/src/main/java/com/netflix/metacat/connector/hive/client/thrift/HiveMetastoreClientFactory.java

@@ -52,7 +52,7 @@ public HiveMetastoreClientFactory(@Nullable final HostAndPort socksProxy,
     }
 
     private static Socket createSocksSocket(final HostAndPort proxy) {
-        final SocketAddress address = InetSocketAddress.createUnresolved(proxy.getHostText(), proxy.getPort());
+        final SocketAddress address = InetSocketAddress.createUnresolved(proxy.getHost(), proxy.getPort());
         return new Socket(new Proxy(Proxy.Type.SOCKS, address));
     }
 

metacat-connector-polaris/src/test/java/com/netflix/metacat/connector/polaris/store/PolarisStoreConnectorTest.java

@@ -10,19 +10,28 @@
 import lombok.Getter;
 import org.junit.Assert;
 import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.orm.jpa.AutoConfigureDataJpa;
 import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.boot.test.mock.mockito.MockBean;
+import org.springframework.boot.test.mock.mockito.SpyBean;
 import org.springframework.dao.DataAccessException;
 import org.springframework.dao.OptimisticLockingFailureException;
+import org.springframework.data.auditing.AuditingHandler;
+import org.springframework.data.auditing.DateTimeProvider;
 import org.springframework.test.context.ActiveProfiles;
 import org.springframework.test.context.junit.jupiter.SpringExtension;
 
+import java.time.Instant;
+import java.time.temporal.ChronoUnit;
 import java.util.Optional;
 import java.util.Random;
 
+import static org.mockito.Mockito.when;
+
 /**
  * Test persistence operations on Database objects.
  */
@@ -46,6 +55,20 @@ public class PolarisStoreConnectorTest {
     @Autowired
     private PolarisStoreConnector polarisConnector;
 
+    @MockBean
+    private DateTimeProvider dateTimeProvider;
+
+    @SpyBean
+    private AuditingHandler auditingHandler;
+
+    @BeforeEach
+    void beforeEach() {
+        // truncate audit time to micros to match DB so assertions will match
+        when(dateTimeProvider.getNow()).thenReturn(
+            Optional.of(Instant.now().truncatedTo(ChronoUnit.MILLIS)));
+        auditingHandler.setDateTimeProvider(dateTimeProvider);
+    }
+
     public static String generateDatabaseName() {
         return DB_NAME_FOO + "_" + random.nextLong();
     }

metacat-connector-s3/build.gradle

@@ -27,7 +27,6 @@ dependencies {
     api("com.google.guava:guava")
     api("com.google.inject:guice")
     api("com.google.inject.extensions:guice-persist")
-    api("com.google.inject.extensions:guice-multibindings")
     api("com.google.inject.extensions:guice-servlet")
     api("joda-time:joda-time:2.10.10")
     api("org.hibernate:hibernate-entitymanager")

metacat-functional-tests/metacat-test-cluster/docker-compose.yml

@@ -15,11 +15,10 @@
 #
 #
 
-version: "2"
-
 services:
     metacat:
-        image: tomcat:8.0-jre8
+        image: tomcat:9.0-jre17
+        platform: linux/x86_64
         depends_on:
             - hive-metastore
             - hive-metastore-db
@@ -85,6 +84,7 @@ services:
     ##TODO: We need to add them back with light images
     hive-metastore-db:
         image: mysql:5.7
+        platform: linux/x86_64
         volumes:
             - ./datastores/mysql/docker-entrypoint-initdb.d:/docker-entrypoint-initdb.d:ro
         environment:
@@ -98,6 +98,7 @@ services:
           - "com.netflix.metacat.oss.test"
     postgresql:
         image: postgres:9.6
+        platform: linux/x86_64
         volumes:
             - ./datastores/postgres/docker-entrypoint-initdb.d:/docker-entrypoint-initdb.d:ro
         environment:
@@ -108,6 +109,7 @@ services:
           - "com.netflix.metacat.oss.test"
     hive-metastore:
         image: danielbwatson/metacat-test-hive-metastore:1.0.0
+        platform: linux/x86_64
         depends_on:
             - hive-metastore-db
         ports:
@@ -120,6 +122,7 @@ services:
           - "com.netflix.metacat.oss.test.hive"
     polaris-crdb:
         image: cockroachdb/cockroach:v21.1.11
+        platform: linux/x86_64
         ports:
             - '26257:26257'
             - '8081:8080'
@@ -129,6 +132,7 @@ services:
             - "com.netflix.metacat.oss.polaris.crdb"
     polaris-crdb-barrier:
         image: martin/wait:latest
+        platform: linux/x86_64
         depends_on:
             - polaris-crdb
         environment:
@@ -138,6 +142,7 @@ services:
             - "com.netflix.metacat.oss.polaris.crdb-barrier"
     polaris-crdb-init:
         image: cockroachdb/cockroach:v21.1.11
+        platform: linux/x86_64
         depends_on:
             - polaris-crdb-barrier
         volumes:
@@ -148,6 +153,7 @@ services:
             - "com.netflix.metacat.oss.polaris.crdb-init"
     storage-barrier:
         image: martin/wait:latest
+        platform: linux/x86_64
         depends_on:
             - hive-metastore-db
             - postgresql
@@ -158,6 +164,7 @@ services:
           - "com.netflix.metacat.oss.test"
     service-barrier:
         image: martin/wait:latest
+        platform: linux/x86_64
         depends_on:
             - hive-metastore
         environment:
@@ -166,6 +173,7 @@ services:
           - "com.netflix.metacat.oss.test"
     metacat-barrier:
         image: martin/wait:latest
+        platform: linux/x86_64
         depends_on:
             - metacat
         environment: