Github user kl0u commented on a diff in the pull request:
https://github.com/apache/flink/pull/6097#discussion_r191460316
--- Diff:
flink-streaming-java/src/test/java/org/apache/flink/streaming/api/operators/KeyedProcessOperatorTest.java
---
@@ -50,6 +52,30 @@
@Rule
public ExpectedException expectedException = ExpectedException.none();
+ @Test
+ public void testKeyQuerying() throws Exception {
+
+ KeyedProcessOperator<Integer, Tuple2<Integer, String>, String>
operator =
--- End diff --
The `OneInputStreamOperatorTestHarness` is `AutoCloseable` so I would
recommend to go with
```
try(harness=...) {
...
}
```
And remove the explicit call to `harness.close()`. This is a nice practice
to start enforcing in new tests as it cleans up any leaks in case of exceptions
and stuff.
---
