[GitHub] helix pull request #87: Make map in NotificationContext synchronized

2017-05-11 Thread kishoreg
Github user kishoreg commented on a diff in the pull request: https://github.com/apache/helix/pull/87#discussion_r116129454 --- Diff: helix-core/src/main/java/org/apache/helix/messaging/handling/HelixTaskExecutor.java --- @@ -143,6 +145,7 @@ public HelixTaskExecutor(ParticipantSta

[GitHub] helix pull request #87: Make map in NotificationContext synchronized

2017-05-11 Thread dasahcc
Github user dasahcc commented on a diff in the pull request: https://github.com/apache/helix/pull/87#discussion_r116132045 --- Diff: helix-core/src/main/java/org/apache/helix/messaging/handling/HelixTaskExecutor.java --- @@ -143,6 +145,7 @@ public HelixTaskExecutor(ParticipantStat