[
https://issues.apache.org/jira/browse/FLINK-8841?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16453701#comment-16453701
]
ASF GitHub Bot commented on FLINK-8841:
---------------------------------------
Github user kl0u commented on the issue:
https://github.com/apache/flink/pull/5910
Thanks @yuqi1129, @bowenli86 and @StefanRRichter for the reviews. I
integrated your comments. If you are done with reviewing, I will merge it as
soon as Travis gives the green light.
> 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)
