[jira] [Created] (FLINK-3210) Unnecessary call to deserializer#deserialize() in LegacyFetcher#SimpleConsumerThread#run()

[Shang Yuanchun (Jira)]

Ted Yu created FLINK-3210:
-----------------------------

             Summary: Unnecessary call to deserializer#deserialize() in LegacyFetcher#SimpleConsumerThread#run()
                 Key: FLINK-3210
                 URL: https://issues.apache.org/jira/browse/FLINK-3210
             Project: Flink
          Issue Type: Bug
            Reporter: Ted Yu
            Priority: Minor


Here is related code:
{code}
                byte[] valueBytes;
                if (payload == null) {
                  deletedMessages++;
                  valueBytes = null;
                } else {
...
                final T value = deserializer.deserialize(keyBytes, valueBytes, fp.topic, offset);
{code}
When valueBytes is null, there is no need to call deserializer#deserialize()



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)