Skip to content
This repository was archived by the owner on Apr 7, 2026. It is now read-only.

Commit 2bd6884

Browse files
committed
merge fixes
1 parent 29dac67 commit 2bd6884

2 files changed

Lines changed: 1 addition & 11 deletions

File tree

google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/GapicSpannerRpc.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -457,8 +457,6 @@ public <ReqT, RespT> ClientCall<ReqT, RespT> interceptCall(
457457
TransportChannelProvider baseChannelProvider =
458458
MoreObjects.firstNonNull(
459459
options.getChannelProvider(), defaultChannelProviderBuilder.build());
460-
boolean enableLocationApi =
461-
Boolean.parseBoolean(System.getenv(EXPERIMENTAL_LOCATION_API_ENV_VAR));
462460
TransportChannelProvider channelProvider =
463461
enableLocationApi && baseChannelProvider instanceof InstantiatingGrpcChannelProvider
464462
? new KeyAwareTransportChannelProvider(
@@ -699,13 +697,13 @@ private InstantiatingGrpcChannelProvider.Builder createChannelProviderBuilder(
699697
.setAllowNonDefaultServiceAccount(true);
700698
if (isEnableDirectAccess) {
701699
defaultChannelProviderBuilder.setAttemptDirectPath(true);
702-
defaultChannelProviderBuilder.setAttemptDirectPathXds();
703700
if (isEnableDirectPathBoundToken()) {
704701
// This will let the credentials try to fetch a hard-bound access token if the runtime
705702
// environment supports it.
706703
defaultChannelProviderBuilder.setAllowHardBoundTokenTypes(
707704
Collections.singletonList(InstantiatingGrpcChannelProvider.HardBoundTokenTypes.ALTS));
708705
}
706+
defaultChannelProviderBuilder.setAttemptDirectPathXds();
709707
}
710708

711709
options.enablegRPCMetrics(defaultChannelProviderBuilder);

google-cloud-spanner/src/test/java/com/google/cloud/spanner/spi/v1/GapicSpannerRpcTest.java

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1134,14 +1134,6 @@ private SpannerOptions createSpannerOptions() {
11341134
.build();
11351135
}
11361136

1137-
private boolean isJava8() {
1138-
return JavaVersionUtil.getJavaMajorVersion() == 8;
1139-
}
1140-
1141-
private boolean isWindows() {
1142-
return System.getProperty("os.name").toLowerCase().contains("windows");
1143-
}
1144-
11451137
static class TestableGapicSpannerRpc extends GapicSpannerRpc {
11461138
public TestableGapicSpannerRpc(SpannerOptions options) {
11471139
super(options);

0 commit comments

Comments
 (0)