Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2241#discussion_r158676771
--- Diff:
storm-client/src/jvm/org/apache/storm/serialization/KryoTupleDeserializer.java
---
@@ -39,7 +38,7 @@ public KryoTupleDeserializer(final Map<String, Object>
conf, final GeneralTopolo
_kryoInput = new Input(1);
}
- public Tuple deserialize(byte[] ser) {
+ public TupleImpl deserialize(byte[] ser) {
--- End diff --
Just a reminder: Let's add `@Override`, and just revert back if we don't
leverage the explicit implementation type.
---
