Skip to content

Commit

Permalink
[improve] Upgrade oxia-java to 0.4.10 and fix closing of OxiaMetadata…
Browse files Browse the repository at this point in the history
…Store (#23653)

(cherry picked from commit 7e6fa55)
  • Loading branch information
lhotari committed Nov 28, 2024
1 parent 142897c commit 2c05137
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 6 deletions.
4 changes: 2 additions & 2 deletions distribution/server/src/assemble/LICENSE.bin.txt
Original file line number Diff line number Diff line change
Expand Up @@ -481,8 +481,8 @@ The Apache Software License, Version 2.0
* Prometheus
- io.prometheus-simpleclient_httpserver-0.16.0.jar
* Oxia
- io.streamnative.oxia-oxia-client-api-0.4.9.jar
- io.streamnative.oxia-oxia-client-0.4.9.jar
- io.streamnative.oxia-oxia-client-api-0.4.10.jar
- io.streamnative.oxia-oxia-client-0.4.10.jar
* OpenHFT
- net.openhft-zero-allocation-hashing-0.16.jar
* Java JSON WebTokens
Expand Down
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ flexible messaging model and an intuitive client API.</description>
<apache-http-client.version>4.5.13</apache-http-client.version>
<apache-httpcomponents.version>4.4.15</apache-httpcomponents.version>
<jetcd.version>0.7.7</jetcd.version>
<oxia.version>0.4.9</oxia.version>
<oxia.version>0.4.10</oxia.version>
<snakeyaml.version>2.0</snakeyaml.version>
<ant.version>1.10.12</ant.version>
<seancfoley.ipaddress.version>5.5.0</seancfoley.ipaddress.version>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -297,10 +297,12 @@ private CompletableFuture<Void> createParents(String path) {

@Override
public void close() throws Exception {
if (client != null) {
client.close();
if (isClosed.compareAndSet(false, true)) {
if (client != null) {
client.close();
}
super.close();
}
super.close();
}

public Optional<MetadataEventSynchronizer> getMetadataEventSynchronizer() {
Expand Down

0 comments on commit 2c05137

Please sign in to comment.