Skip to content

Commit d0d1aa3

Browse files
docs: fix connection pool size for samples (#880)
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly: - [ ] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/java-bigtable/issues/new/choose) before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea - [ ] Ensure the tests and linter pass - [ ] Code coverage does not decrease (if any source code was changed) - [ ] Appropriate docs were updated (if necessary) Fixes #<issue_number_goes_here> ☕️
1 parent 4614912 commit d0d1aa3

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

samples/snippets/src/main/java/com/example/bigtable/ConfigureConnectionPool.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public static void configureConnectionPool(String projectId, String instanceId)
3737
settingsBuilder.stubSettings()
3838
.setTransportChannelProvider(
3939
EnhancedBigtableStubSettings.defaultGrpcTransportProviderBuilder()
40-
.setPoolSize(250)
40+
.setPoolSize(10)
4141
.build());
4242

4343
BigtableDataSettings settings = settingsBuilder.build();

samples/snippets/src/test/java/com/example/bigtable/ConfigureConnectionPoolTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,6 @@ public void testConfigureConnectionPool() {
4141
ConfigureConnectionPool.configureConnectionPool(projectId, instanceId);
4242

4343
String output = bout.toString();
44-
assertThat(output, CoreMatchers.containsString("Connected with pool size of 250"));
44+
assertThat(output, CoreMatchers.containsString("Connected with pool size of 10"));
4545
}
4646
}

0 commit comments

Comments
 (0)