This is an automated email from the ASF dual-hosted git repository. madhan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/atlas.git
The following commit(s) were added to refs/heads/master by this push: new 636c2b74c ATLAS-4923: Bump versions of commons-fileupload, jetty, testng 636c2b74c is described below commit 636c2b74c0b917e6ca5e828d1eea01bf8ae8e389 Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> AuthorDate: Mon Nov 18 07:58:51 2024 -0800 ATLAS-4923: Bump versions of commons-fileupload, jetty, testng Signed-off-by: Madhan Neethiraj <mad...@apache.org> --- .../atlas/hbase/TestHBaseTestingUtilSpinup.java | 50 +++++++++++++--------- addons/hive-bridge/pom.xml | 7 +++ graphdb/common/pom.xml | 2 +- pom.xml | 2 +- test-tools/pom.xml | 2 +- 5 files changed, 40 insertions(+), 23 deletions(-) diff --git a/addons/hbase-testing-util/src/test/java/org/apache/atlas/hbase/TestHBaseTestingUtilSpinup.java b/addons/hbase-testing-util/src/test/java/org/apache/atlas/hbase/TestHBaseTestingUtilSpinup.java index 0beb03552..b870d97ea 100644 --- a/addons/hbase-testing-util/src/test/java/org/apache/atlas/hbase/TestHBaseTestingUtilSpinup.java +++ b/addons/hbase-testing-util/src/test/java/org/apache/atlas/hbase/TestHBaseTestingUtilSpinup.java @@ -18,13 +18,11 @@ package org.apache.atlas.hbase; import org.apache.hadoop.hbase.HBaseTestingUtility; -import org.testng.annotations.AfterClass; -import org.testng.annotations.BeforeClass; +import org.apache.hadoop.hbase.MiniHBaseCluster; import org.testng.annotations.Test; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - +import java.io.IOException; +import java.net.ServerSocket; import java.util.List; import static org.testng.AssertJUnit.assertFalse; @@ -34,26 +32,38 @@ import static org.testng.AssertJUnit.assertFalse; * Make sure we can spin up a HBTU without a hbase-site.xml */ public class TestHBaseTestingUtilSpinup { - private static final Logger LOG = LoggerFactory.getLogger(TestHBaseTestingUtilSpinup.class); - private final static HBaseTestingUtility UTIL = new HBaseTestingUtility(); - - @BeforeClass - public static void beforeClass() throws Exception { - UTIL.startMiniCluster(); - if (!UTIL.getHBaseCluster().waitForActiveAndReadyMaster(30000)) { - throw new RuntimeException("Active master not ready"); - } - } + private final HBaseTestingUtility UTIL = new HBaseTestingUtility(); - @AfterClass - public static void afterClass() throws Exception { - UTIL.shutdownMiniCluster(); + public TestHBaseTestingUtilSpinup() throws Exception { + UTIL.getConfiguration().set("test.hbase.zookeeper.property.clientPort", String.valueOf(getFreePort())); + UTIL.getConfiguration().set("hbase.master.port", String.valueOf(getFreePort())); + UTIL.getConfiguration().set("hbase.master.info.port", String.valueOf(getFreePort())); + UTIL.getConfiguration().set("hbase.regionserver.port", String.valueOf(getFreePort())); + UTIL.getConfiguration().set("hbase.regionserver.info.port", String.valueOf(getFreePort())); + UTIL.getConfiguration().set("zookeeper.znode.parent", "/hbase-unsecure"); + UTIL.getConfiguration().set("hbase.table.sanity.checks", "false"); } @Test public void testGetMetaTableRows() throws Exception { - List<byte[]> results = UTIL.getMetaTableRows(); - assertFalse("results should have some entries and is empty.", results.isEmpty()); + try (MiniHBaseCluster miniCluster = UTIL.startMiniCluster()) { + if (!UTIL.getHBaseCluster().waitForActiveAndReadyMaster(30000)) { + throw new RuntimeException("Active master not ready"); + } + + List<byte[]> results = UTIL.getMetaTableRows(); + assertFalse("results should have some entries and is empty.", results.isEmpty()); + } finally { + UTIL.shutdownMiniCluster(); + } } + private static int getFreePort() throws IOException { + ServerSocket serverSocket = new ServerSocket(0); + int port = serverSocket.getLocalPort(); + + serverSocket.close(); + + return port; + } } diff --git a/addons/hive-bridge/pom.xml b/addons/hive-bridge/pom.xml index 3dfbb33a7..fae42b810 100755 --- a/addons/hive-bridge/pom.xml +++ b/addons/hive-bridge/pom.xml @@ -42,6 +42,13 @@ <artifactId>slf4j-log4j12</artifactId> </dependency> + <dependency> + <groupId>junit</groupId> + <artifactId>junit</artifactId> + <version>${junit.version}</version> + <scope>test</scope> + </dependency> + <dependency> <groupId>org.apache.hive</groupId> <artifactId>hive-metastore</artifactId> diff --git a/graphdb/common/pom.xml b/graphdb/common/pom.xml index ce6e6fdef..e4e5dbe11 100644 --- a/graphdb/common/pom.xml +++ b/graphdb/common/pom.xml @@ -42,7 +42,7 @@ under the License. --> <dependency> <groupId>org.testng</groupId> <artifactId>testng</artifactId> - <version>7.0.0</version> + <version>7.5.1</version> <scope>test</scope> </dependency> diff --git a/pom.xml b/pom.xml index 91046fc51..60dcc3f68 100644 --- a/pom.xml +++ b/pom.xml @@ -773,7 +773,7 @@ <jersey.version>1.19</jersey.version> <jettison.version>1.5.4</jettison.version> <jetty-maven-plugin.stopWait>10</jetty-maven-plugin.stopWait> - <jetty.version>9.4.53.v20231009</jetty.version> + <jetty.version>9.4.56.v20240826</jetty.version> <joda-time.version>2.10.6</joda-time.version> <json.version>3.2.11</json.version> <json-simple.version>1.1.1</json-simple.version> diff --git a/test-tools/pom.xml b/test-tools/pom.xml index a7b9a47d9..a01494f3b 100644 --- a/test-tools/pom.xml +++ b/test-tools/pom.xml @@ -91,7 +91,7 @@ <dependency> <groupId>commons-fileupload</groupId> <artifactId>commons-fileupload</artifactId> - <version>1.3.3</version> + <version>1.5</version> </dependency> <dependency>