mosche commented on a change in pull request #16760:
URL: https://github.com/apache/beam/pull/16760#discussion_r808173287
##########
File path:
sdks/java/io/amazon-web-services2/src/main/java/org/apache/beam/sdk/io/aws2/dynamodb/DynamoDBIO.java
##########
@@ -173,33 +190,66 @@
@AutoValue.Builder
abstract static class Builder<T> {
+ abstract Builder<T> setClientConfiguration(ClientConfiguration config);
+
abstract Builder<T> setDynamoDbClientProvider(DynamoDbClientProvider
dynamoDbClientProvider);
abstract Builder<T> setScanRequestFn(SerializableFunction<Void,
ScanRequest> fn);
abstract Builder<T> setSegmentId(Integer segmentId);
- abstract Builder<T> setScanResponseMapperFn(
- SerializableFunction<ScanResponse, T> scanResponseMapperFn);
+ abstract Builder<T>
setScanResponseMapperFn(SerializableFunction<ScanResponse, T> fn);
abstract Builder<T> setCoder(Coder<T> coder);
abstract Read<T> build();
}
- public Read<T> withDynamoDbClientProvider(DynamoDbClientProvider
dynamoDbClientProvider) {
Review comment:
It is deprecated and not removed, please check a few lines down ... just
a presentation issue
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
