Skip to content

Commit 01a2014

Browse files
committed
Update deps
Signed-off-by: Luca Burgazzoli <lburgazzoli@gmail.com>
1 parent acbfa6f commit 01a2014

File tree

4 files changed

+7
-32
lines changed

4 files changed

+7
-32
lines changed

gradle/libs.versions.toml

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
[versions]
2-
grpc = "1.68.0"
2+
grpc = "1.68.1"
33
log4j = "2.24.1"
44
mockito = "5.14.2"
55
slf4j = "2.0.16"
@@ -26,8 +26,8 @@ shadowPlugin = "8.1.1"
2626
testLoggerPlugin = "4.0.0"
2727
protobufPlugin = "0.9.4"
2828
nexusPublishPlugin = "2.0.0"
29-
axionReleasePlugin = "1.18.4"
30-
testRetryPlugin = "1.5.10"
29+
axionReleasePlugin = "1.18.15"
30+
testRetryPlugin = "1.6.0"
3131

3232

3333
[libraries]

gradle/wrapper/gradle-wrapper.properties

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
distributionBase=GRADLE_USER_HOME
22
distributionPath=wrapper/dists
3-
distributionUrl=https\://services.gradle.org/distributions/gradle-8.10-bin.zip
3+
distributionUrl=https\://services.gradle.org/distributions/gradle-8.11-bin.zip
44
networkTimeout=10000
55
validateDistributionUrl=true
66
zipStoreBase=GRADLE_USER_HOME

jetcd-core/src/main/java/io/etcd/jetcd/resolver/AbstractNameResolver.java

+1-14
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
import java.net.URI;
2020
import java.util.List;
2121
import java.util.Objects;
22-
import java.util.concurrent.Executor;
2322

2423
import org.slf4j.Logger;
2524
import org.slf4j.LoggerFactory;
@@ -30,8 +29,6 @@
3029
import io.grpc.EquivalentAddressGroup;
3130
import io.grpc.NameResolver;
3231
import io.grpc.Status;
33-
import io.grpc.internal.GrpcUtil;
34-
import io.grpc.internal.SharedResourceHolder;
3532

3633
import com.google.common.base.Preconditions;
3734

@@ -47,7 +44,6 @@ public abstract class AbstractNameResolver extends NameResolver {
4744
private volatile boolean shutdown;
4845
private volatile boolean resolving;
4946

50-
private Executor executor;
5147
private Listener listener;
5248

5349
public AbstractNameResolver(URI targetUri) {
@@ -69,7 +65,6 @@ public String getServiceAuthority() {
6965
public void start(Listener listener) {
7066
synchronized (lock) {
7167
Preconditions.checkState(this.listener == null, "already started");
72-
this.executor = SharedResourceHolder.get(GrpcUtil.SHARED_CHANNEL_EXECUTOR);
7368
this.listener = Objects.requireNonNull(listener, "listener");
7469
resolve();
7570
}
@@ -86,21 +81,13 @@ public void shutdown() {
8681
return;
8782
}
8883
shutdown = true;
89-
90-
synchronized (lock) {
91-
if (executor != null) {
92-
executor = SharedResourceHolder.release(GrpcUtil.SHARED_CHANNEL_EXECUTOR, executor);
93-
}
94-
}
9584
}
9685

9786
private void resolve() {
9887
if (resolving || shutdown) {
9988
return;
10089
}
101-
synchronized (lock) {
102-
executor.execute(this::doResolve);
103-
}
90+
doResolve();
10491
}
10592

10693
private void doResolve() {

jetcd-test/src/main/java/io/etcd/jetcd/test/EtcdClusterNameResolver.java

+2-14
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
import java.util.ArrayList;
2121
import java.util.List;
2222
import java.util.Objects;
23-
import java.util.concurrent.Executor;
2423

2524
import org.slf4j.Logger;
2625
import org.slf4j.LoggerFactory;
@@ -31,8 +30,6 @@
3130
import io.grpc.EquivalentAddressGroup;
3231
import io.grpc.NameResolver;
3332
import io.grpc.Status;
34-
import io.grpc.internal.GrpcUtil;
35-
import io.grpc.internal.SharedResourceHolder;
3633

3734
import com.google.common.base.Preconditions;
3835

@@ -47,7 +44,6 @@ public class EtcdClusterNameResolver extends NameResolver {
4744
private volatile boolean shutdown;
4845
private volatile boolean resolving;
4946

50-
private Executor executor;
5147
private Listener listener;
5248

5349
public EtcdClusterNameResolver(URI targetUri) {
@@ -65,7 +61,6 @@ public String getServiceAuthority() {
6561
public void start(Listener listener) {
6662
synchronized (lock) {
6763
Preconditions.checkState(this.listener == null, "already started");
68-
this.executor = SharedResourceHolder.get(GrpcUtil.SHARED_CHANNEL_EXECUTOR);
6964
this.listener = Objects.requireNonNull(listener, "listener");
7065
resolve();
7166
}
@@ -82,21 +77,14 @@ public void shutdown() {
8277
return;
8378
}
8479
shutdown = true;
85-
86-
synchronized (lock) {
87-
if (executor != null) {
88-
executor = SharedResourceHolder.release(GrpcUtil.SHARED_CHANNEL_EXECUTOR, executor);
89-
}
90-
}
9180
}
9281

9382
private void resolve() {
9483
if (resolving || shutdown) {
9584
return;
9685
}
97-
synchronized (lock) {
98-
executor.execute(this::doResolve);
99-
}
86+
87+
doResolve();
10088
}
10189

10290
private void doResolve() {

0 commit comments

Comments
 (0)