You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flink.apache.org by "Kostas Kloudas (JIRA)" <ji...@apache.org> on 2018/03/02 17:53:00 UTC

[jira] [Created] (FLINK-8841) Duplicate MapSerializer and HashMapSerializer.

Kostas Kloudas created FLINK-8841:
-------------------------------------

             Summary: Duplicate MapSerializer and HashMapSerializer.
                 Key: FLINK-8841
                 URL: https://issues.apache.org/jira/browse/FLINK-8841
             Project: Flink
          Issue Type: Bug
          Components: State Backends, Checkpointing, Type Serialization System
    Affects Versions: 1.5.0
            Reporter: Kostas Kloudas
            Assignee: Kostas Kloudas
             Fix For: 1.5.0


Currently there are two class MapSerializer and HashMapSerializer whose code is the same with the only difference being that one includes elements of type Map and the other HashMap. 

In addition, these two were merged on the same commit. 

I would like to remove the HashMapSerializer. I already created a branch without the HashMapSerialzer and nothing seems to be failing on Travis. The reasons why I hesitate to do it, is because I am not sure if this may create problems with Backwards Compatibility.

[~xiaogang.sxg] could you elaborate a bit on why they were both added and if there is any danger in removing the HashMapSerializer?

Also [~StephanEwen] and [~stefanrichter83@gmail.com] it is worth having a look and if you are ok, I can remove the redundant serializer.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)