GitHub user zhan849 opened a pull request:
https://github.com/apache/helix/pull/175
[HELIX-692] use map instead of list to avoid deleting redundant message
during cleanup
Currently in MessageGenerationPhase, we are using list to store messages to
GC. However, pending message is stored per resource/partition/instance, and
under batch message mode, same message is stored once for each partition in the
batch, which lead to the fact that we are cleaning up same message a lot of
times.
This RB changes list to map to avoid redundant cleanup
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zhan849/helix harry/HELIX-692
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/helix/pull/175.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #175
----
commit ce7d2e9d275e1403375edd94d63afa94ea1a2234
Author: Harry Zhang <zhan849@...>
Date: 2018-04-06T23:27:02Z
[HELIX-692] use map instead of list to avoid deleting redundant message
during cleanup
----
---
