diff --git a/docs/src/main/sphinx/connector/hive-alluxio.rst b/docs/src/main/sphinx/connector/hive-alluxio.rst index 103149a1efc2..c9ad97ab6a9e 100644 --- a/docs/src/main/sphinx/connector/hive-alluxio.rst +++ b/docs/src/main/sphinx/connector/hive-alluxio.rst @@ -80,7 +80,7 @@ the following: .. code-block:: text connector.name=hive - hive.metastore=alluxio + hive.metastore=alluxio-deprecated hive.metastore.alluxio.master.address=HOSTNAME:PORT Replace ``HOSTNAME`` with the Alluxio master hostname, and replace ``PORT`` diff --git a/plugin/trino-delta-lake/src/test/java/io/trino/plugin/deltalake/TestDeltaLakePlugin.java b/plugin/trino-delta-lake/src/test/java/io/trino/plugin/deltalake/TestDeltaLakePlugin.java index 7c3157ada182..19c21f511681 100644 --- a/plugin/trino-delta-lake/src/test/java/io/trino/plugin/deltalake/TestDeltaLakePlugin.java +++ b/plugin/trino-delta-lake/src/test/java/io/trino/plugin/deltalake/TestDeltaLakePlugin.java @@ -104,6 +104,13 @@ public void testAlluxioMetastore() new TestingConnectorContext())) .hasMessageMatching("(?s)Unable to create injector, see the following errors:.*" + "Explicit bindings are required and HiveMetastoreFactory .* is not explicitly bound.*"); + + assertThatThrownBy(() -> factory.create( + "test", + ImmutableMap.of("hive.metastore", "alluxio-deprecated"), + new TestingConnectorContext())) + .hasMessageMatching("(?s)Unable to create injector, see the following errors:.*" + + "Explicit bindings are required and HiveMetastoreFactory .* is not explicitly bound.*"); } @Test diff --git a/plugin/trino-hive-hadoop2/src/test/java/io/trino/plugin/hive/TestHivePlugin.java b/plugin/trino-hive-hadoop2/src/test/java/io/trino/plugin/hive/TestHivePlugin.java index c88bdb673461..a50798c847a9 100644 --- a/plugin/trino-hive-hadoop2/src/test/java/io/trino/plugin/hive/TestHivePlugin.java +++ b/plugin/trino-hive-hadoop2/src/test/java/io/trino/plugin/hive/TestHivePlugin.java @@ -134,7 +134,7 @@ public void testAlluxioMetastore() factory.create( "test", ImmutableMap.of( - "hive.metastore", "alluxio", + "hive.metastore", "alluxio-deprecated", "hive.metastore.alluxio.master.address", "dummy:1234"), new TestingConnectorContext()) .shutdown(); diff --git a/plugin/trino-hive/src/main/java/io/trino/plugin/hive/metastore/HiveMetastoreModule.java b/plugin/trino-hive/src/main/java/io/trino/plugin/hive/metastore/HiveMetastoreModule.java index c8b58145f16c..4b3ab0a38a78 100644 --- a/plugin/trino-hive/src/main/java/io/trino/plugin/hive/metastore/HiveMetastoreModule.java +++ b/plugin/trino-hive/src/main/java/io/trino/plugin/hive/metastore/HiveMetastoreModule.java @@ -50,7 +50,7 @@ protected void setup(Binder binder) // Load Alluxio metastore support through reflection. This makes Alluxio effectively an optional dependency // and allows deploying Trino without the Alluxio jar. Can be useful if the integration is unused and is flagged // by a security scanner. - bindMetastoreModule("alluxio", deferredModule("io.trino.plugin.hive.metastore.alluxio.AlluxioMetastoreModule")); + bindMetastoreModule("alluxio-deprecated", deferredModule("io.trino.plugin.hive.metastore.alluxio.AlluxioMetastoreModule")); } install(new DecoratedHiveMetastoreModule());