[ https://issues.apache.org/jira/browse/FLINK-9124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16434617#comment-16434617 ]
ASF GitHub Bot commented on FLINK-9124: --------------------------------------- Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/5803#discussion_r180909552 --- Diff: flink-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/proxy/KinesisProxy.java --- @@ -176,6 +179,16 @@ protected KinesisProxy(Properties configProps) { } + /** + * Create the Kinesis client, using the provided configuration properties and default {@link ClientConfiguration}. + * Derived classes can override this method to customize the client configuration. + * @param configProps + * @return + */ + protected AmazonKinesis createKinesisClient(Properties configProps) { --- End diff -- My main concern with allowing overrides of this method, is that override implementations can potentially completely ignore the `configProps` settings and create a Kinesis client entirely irrelevant from the original configuration. IMO, this is not nice design-wise. As a different approach, would it be possible to traverse keys in the `configProps` and set the `ClientConfiguration` appropriately, such that we won't need to be aware of all updated / new keys in the AWS Kinesis SDK? Ideally, Flink should not need to maintain its own set of config keys and just rely on AWS's keys (for example, Flink actually should not need to define its own config keys for AWS credentials). > Allow customization of KinesisProxy.getRecords read timeout and retry > --------------------------------------------------------------------- > > Key: FLINK-9124 > URL: https://issues.apache.org/jira/browse/FLINK-9124 > Project: Flink > Issue Type: Task > Components: Kinesis Connector > Affects Versions: 1.4.2 > Reporter: Thomas Weise > Assignee: Thomas Weise > Priority: Minor > > It should be possible to change the socket read timeout and all other > configuration parameters of the underlying AWS ClientConfiguration and also > have the option to retry after a socket timeout exception. -- This message was sent by Atlassian JIRA (v7.6.3#76005)