GitHub user zhan849 opened a pull request:

    https://github.com/apache/helix/pull/181

    [HELIX-690] batch message execution should not share same context

    In this PR, I added deep copy methods to NotificationContext so when 
processing messages in batch, different thread would not share the same 
notification context.
    
    This solves the problem that when processing BatchMessages, each thread 
will have their own current state delta to work on, so current states won't be 
messed up.
    
    Also modified some logs to make it more useful when debugging

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/zhan849/helix harry/batch-msg-fix

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/helix/pull/181.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 #181
    
----
commit bb7751b0f52aadcf04b7813fa3e99c8e266a3d0b
Author: Harry Zhang <zhan849@...>
Date:   2018-04-16T16:55:43Z

    [HELIX-690] batch message execution should not share same context

----


---

Reply via email to