@tzulitai We can remove the non-timestamped version of the `deserialize` method completely. It will break the interface and if you implemented this method it quite easy to migrate to the new method. I've only introduced the new `deserialize` method as default to make this change backwards compatible. When backwards compatibility is required we could mark the non-timestamped version of the `deserialize` method as `@Deprecated` and give it a default implementation that throws a `NotImplementedException` or a `FlinkRuntimeException` with the message "you must implement the deserialize method when implementingt the KeyedDeserializationSchema interface"
[ Full content available at: https://github.com/apache/flink/pull/6105 ] This message was relayed via gitbox.apache.org for [email protected]
