[ https://issues.apache.org/jira/browse/FLINK-8841?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16453099#comment-16453099 ]
ASF GitHub Bot commented on FLINK-8841: --------------------------------------- Github user bowenli86 commented on the issue: https://github.com/apache/flink/pull/5910 +1 > 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 > Priority: Major > 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 [~stefanrichte...@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)