Skip to content

Commit 13d1e3d

Browse files
committed
cleanup(S3ClientProvider): Reduce visibility of methods
1 parent cf5ae74 commit 13d1e3d

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

src/main/java/software/amazon/nio/spi/s3/S3ClientProvider.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ public void asyncClientBuilder(final S3CrtAsyncClientBuilder builder) {
121121
*
122122
* @return a S3Client not bound to a region
123123
*/
124-
public S3Client universalClient() {
124+
S3Client universalClient() {
125125
return universalClient(false);
126126
}
127127

@@ -133,7 +133,7 @@ public S3Client universalClient() {
133133
* @param <T> type of AwsClient
134134
* @return a S3Client not bound to a region
135135
*/
136-
public <T extends AwsClient> T universalClient(boolean async) {
136+
<T extends AwsClient> T universalClient(boolean async) {
137137
return (T)((async) ? DEFAULT_ASYNC_CLIENT : DEFAULT_CLIENT);
138138
}
139139

@@ -160,7 +160,7 @@ protected S3AsyncClient generateAsyncClient(String bucket) {
160160
*
161161
* @return an S3 client appropriate for the region of the named bucket
162162
*/
163-
protected S3Client generateSyncClient(String bucketName, S3Client locationClient) {
163+
S3Client generateSyncClient(String bucketName, S3Client locationClient) {
164164
return getClientForBucket(bucketName, locationClient, this::clientForRegion);
165165
}
166166

@@ -174,11 +174,11 @@ protected S3Client generateSyncClient(String bucketName, S3Client locationClient
174174
*
175175
* @return an S3 client appropriate for the region of the named bucket
176176
*/
177-
protected S3AsyncClient generateAsyncClient (String bucketName, S3Client locationClient) {
177+
S3AsyncClient generateAsyncClient (String bucketName, S3Client locationClient) {
178178
return getClientForBucket(bucketName, locationClient, this::asyncClientForRegion);
179179
}
180180

181-
protected <T extends AwsClient> T getClientForBucket(
181+
private <T extends AwsClient> T getClientForBucket(
182182
String bucketName,
183183
S3Client locationClient,
184184
Function<String, T> getClientForRegion

src/test/java/software/amazon/nio/spi/s3/FixedS3ClientProvider.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ public FixedS3ClientProvider(S3AsyncClient client) {
2424
}
2525

2626
@Override
27-
public S3Client universalClient() {
27+
S3Client universalClient() {
2828
return (S3Client)client;
2929
}
3030

0 commit comments

Comments
 (0)