[GitHub] flink pull request #5130: [FLINK-8216] Unify test utils in flink-connector-k...

2017-12-07 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/5130#discussion_r155451196 --- Diff: flink-connectors/flink-connector-kinesis/src/test/java/org/apache/flink/streaming/connectors/kinesis/testutils/TestUtils.java --- @@ -0,0 +1,39

[GitHub] flink pull request #5130: [FLINK-8216] Unify test utils in flink-connector-k...

2017-12-06 Thread bowenli86
GitHub user bowenli86 opened a pull request: https://github.com/apache/flink/pull/5130 [FLINK-8216] Unify test utils in flink-connector-kinesis ## What is the purpose of the change currently there are a few ways to get a `Properties` object with required fields (aws access