Skip to content

Commit cdd4a5d

Browse files
committed
wip
1 parent 2016fbf commit cdd4a5d

File tree

3 files changed

+18
-16
lines changed

3 files changed

+18
-16
lines changed

extensions/cli/hbase-embed/src/main/java/org/locationtech/geowave/datastore/hbase/cli/ZookeeperMiniCluster.java

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -54,30 +54,32 @@ public void setup() throws Exception {
5454
System.setProperty(
5555
"test.build.data.basedirectory",
5656
conf.get("zookeeper.temp.dir", zkDataDir));
57-
zookeeperLocalCluster =
58-
Class.forName(
59-
"org.apache.hadoop.hbase.HBaseTestingUtility",
60-
true,
61-
hbaseMiniClusterCl).getConstructor(Configuration.class).newInstance(conf);
62-
zookeeperLocalCluster.getClass().getMethod("startMiniZKCluster").invoke(
63-
zookeeperLocalCluster);
57+
final Class<?> htuClass =
58+
Class.forName("org.apache.hadoop.hbase.HBaseTestingUtility", true, hbaseMiniClusterCl);
59+
final Object instance = htuClass.getConstructor(Configuration.class).newInstance(conf);
60+
if (instance == null) {
61+
throw new IllegalStateException(
62+
"Mini Zookeeper cluster failed to instantiate HBaseTestingUtility");
63+
}
64+
zookeeperLocalCluster = instance;
65+
htuClass.getMethod("startMiniZKCluster").invoke(zookeeperLocalCluster);
6466
} catch (final Exception e) {
6567
LOGGER.error("Exception starting zookeeperLocalCluster: " + e, e);
6668
throw e;
6769
} finally {
6870
Thread.currentThread().setContextClassLoader(prevCl);
6971
}
70-
if (zookeeperLocalCluster == null) {
71-
throw new IllegalStateException(
72-
"Mini Zookeeper cluster failed to start; see logs for details");
73-
}
7472
final Object zkCluster =
7573
zookeeperLocalCluster.getClass().getMethod("getZkCluster").invoke(zookeeperLocalCluster);
7674
zookeeper = "127.0.0.1:" + zkCluster.getClass().getMethod("getClientPort").invoke(zkCluster);
7775
}
7876
}
7977

8078
public void tearDown() throws Exception {
79+
if (zookeeperLocalCluster == null) {
80+
zookeeper = null;
81+
return;
82+
}
8183
try {
8284
zookeeperLocalCluster.getClass().getMethod("shutdownMiniZKCluster").invoke(
8385
zookeeperLocalCluster);

pom.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,8 @@
8383
<skipITs>true</skipITs>
8484
<vfs2.version>2.9.0</vfs2.version>
8585
<configuration2.version>2.9.0</configuration2.version>
86-
<hbase.version>2.5.12</hbase.version>
87-
<hadoop.version>3.3.6</hadoop.version>
86+
<hbase.version>2.6.3</hbase.version>
87+
<hadoop.version>3.4.1</hadoop.version>
8888
<avro.version>1.7.6</avro.version>
8989
<awssdk.version>1.11.1015</awssdk.version>
9090
<httpclient.version>4.5.13</httpclient.version>
@@ -96,7 +96,7 @@
9696
<jetty.version>11.0.17</jetty.version>
9797
<bouncycastle.version>1.70</bouncycastle.version>
9898
<lz4.version>1.8.0</lz4.version>
99-
<zookeeper.version>3.5.7</zookeeper.version>
99+
<zookeeper.version>3.8.4</zookeeper.version>
100100
<jersey.version>3.1.3</jersey.version>
101101
<scala.version>2.12.18</scala.version>
102102
<kafka.artifact>kafka_2.12</kafka.artifact>
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
py4j==0.10.8.1
22
shapely==2.0.4
3-
pytest==4.3.1
4-
pytest-cov==2.7.1
3+
pytest==8.3.3
4+
pytest-cov==5.0.0
55
pdoc3==0.6.3

0 commit comments

Comments
 (0)