[GitHub] helix pull request: [Helix-624] Fix NPE in TestMessageService.Test...

2016-01-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/helix/pull/42 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enable

[GitHub] helix pull request: [Helix-624] Fix NPE in TestMessageService.Test...

2016-01-13 Thread kishoreg
Github user kishoreg commented on the pull request: https://github.com/apache/helix/pull/42#issuecomment-171563196 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled an

[GitHub] helix pull request: [Helix-624] Fix NPE in TestMessageService.Test...

2016-01-13 Thread jicongrui
GitHub user jicongrui opened a pull request: https://github.com/apache/helix/pull/42 [Helix-624] Fix NPE in TestMessageService.TestMultiMessageCriteria Fix NPE in TestMessageService.TestMultiMessageCriteria. The messageHandlerFactory is not correctly set and cause the pipeline b