Github user yanghua commented on a diff in the pull request: https://github.com/apache/flink/pull/6134#discussion_r193976690 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/state/KeyedStateFunction.java --- @@ -25,13 +25,13 @@ * <p>This functionality is only available through the * {@code BroadcastConnectedStream.process(final KeyedBroadcastProcessFunction function)}. */ -public abstract class KeyedStateFunction<K, S extends State> { +public interface KeyedStateFunction<K, S extends State> { --- End diff -- hi @dawidwys the Travis is green, that means the lambda is right. I think there isn't necessary to have a extra test? I think changing from anonymous implementation to lambda function can be treat as refactor.
---