Skip to content

Commit 9e7f83e

Browse files
authored
---
yaml --- r: 5537 b: refs/heads/master c: 78346c3 h: refs/heads/master i: 5535: f330cb2
1 parent 9895f60 commit 9e7f83e

3 files changed

Lines changed: 12 additions & 11 deletions

File tree

[refs]

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
refs/heads/master: 3a039931aaaaf19b128d96ba94107299c2a82749
2+
refs/heads/master: 78346c3527c36bc4581d124de7a862b06f3d86f4
33
refs/heads/travis: dae77e558b884bc1b165155482d76c8e40b0fca4
44
refs/heads/gh-pages: 049a9668af6afce3b5840dee8be005b0e6b43506
55
refs/tags/0.0.9: 22f1839238f66c39e67ed4dfdcd273b1ae2e8444

trunk/google-cloud-core/src/main/java/com/google/cloud/GrpcTransportOptions.java

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -210,23 +210,23 @@ public UnaryCallSettings.Builder getApiCallSettings(RetrySettings retrySettings)
210210
}
211211

212212
/**
213-
* Returns a channel provider.
213+
* Returns a channel provider from the given default provider.
214214
*/
215-
public static ChannelProvider getChannelProvider(
216-
ServiceOptions<?, ?> serviceOptions) {
215+
public static ChannelProvider setUpChannelProvider(
216+
InstantiatingChannelProvider.Builder providerBuilder, ServiceOptions<?, ?> serviceOptions) {
217217
HostAndPort hostAndPort = HostAndPort.fromString(serviceOptions.getHost());
218-
InstantiatingChannelProvider.Builder builder = InstantiatingChannelProvider.newBuilder()
219-
.setServiceAddress(hostAndPort.getHostText())
218+
providerBuilder.setServiceAddress(hostAndPort.getHostText())
220219
.setPort(hostAndPort.getPort())
221-
.setClientLibHeader(serviceOptions.getGoogApiClientLibName(),
222-
firstNonNull(serviceOptions.getLibraryVersion(), ""));
220+
.setClientLibHeader(ServiceOptions.getGoogApiClientLibName(),
221+
firstNonNull(ServiceOptions.getLibraryVersion(), ""));
223222
Credentials scopedCredentials = serviceOptions.getScopedCredentials();
224223
if (scopedCredentials != null && scopedCredentials != NoCredentials.getInstance()) {
225-
builder.setCredentialsProvider(FixedCredentialsProvider.create(scopedCredentials));
224+
providerBuilder.setCredentialsProvider(FixedCredentialsProvider.create(scopedCredentials));
226225
}
227-
return builder.build();
226+
return providerBuilder.build();
228227
}
229228

229+
230230
/**
231231
* Returns the timeout for the initial RPC, in milliseconds. Subsequent calls will use this value
232232
* adjusted according to {@link #getTimeoutMultiplier()}. Default value is 20000.

trunk/google-cloud-logging/src/main/java/com/google/cloud/logging/spi/v2/GrpcLoggingRpc.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,8 @@ public GrpcLoggingRpc(LoggingOptions options) throws IOException {
9090
.build();
9191
channelProvider = FixedChannelProvider.create(managedChannel);
9292
} else {
93-
channelProvider = transportOptions.getChannelProvider(options);
93+
channelProvider = GrpcTransportOptions.setUpChannelProvider(
94+
LoggingSettings.defaultChannelProviderBuilder(), options);
9495
}
9596
providerManager = ProviderManager.newBuilder()
9697
.setChannelProvider(channelProvider)

0 commit comments

Comments
 (0)