[GitHub] helix issue #300: Daemonize SubscribeChangeEventProcessor thread in Callback...

2019-02-06 Thread mebe
Github user mebe commented on the issue: https://github.com/apache/helix/pull/300 Thanks! Any suggestions for a better fix? Currently I have this beautiful piece of Scala code in my shutdown hooks and I'd like to get rid of it 🙂 ```scala private def

[GitHub] helix pull request #300: Daemonize SubscribeChangeEventProcessor thread in C...

2019-02-05 Thread mebe
GitHub user mebe opened a pull request: https://github.com/apache/helix/pull/300 Daemonize SubscribeChangeEventProcessor thread in CallbackProcessor `CallbackProcessor.SubscribeChangeEventProcessor` is a `DedupEventProcessor extends Thread`. It is started in a static initializer

[GitHub] helix pull request #299: Remove unused _forceRebalanceTimer from GenericHeli...

2019-02-05 Thread mebe
GitHub user mebe opened a pull request: https://github.com/apache/helix/pull/299 Remove unused _forceRebalanceTimer from GenericHelixController `GenericHelixController. _forceRebalanceTimer` is a non-daemon `Timer` thread. It is started in the initializer of `GenericHelixController