----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18167/#review34587 -----------------------------------------------------------
helix-core/src/main/java/org/apache/helix/controller/GenericHelixController.java <https://reviews.apache.org/r/18167/#comment64747> Why was this annotation removed? helix-core/src/main/java/org/apache/helix/controller/GenericHelixController.java <https://reviews.apache.org/r/18167/#comment64746> whitespace helix-core/src/main/java/org/apache/helix/controller/GenericHelixController.java <https://reviews.apache.org/r/18167/#comment64745> Please use _eventQueue.put(event) instead of handleEvent(event). - Kanak Biscuitwala On Feb. 15, 2014, 11:28 p.m., Sandeep Nayak wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18167/ > ----------------------------------------------------------- > > (Updated Feb. 15, 2014, 11:28 p.m.) > > > Review request for helix. > > > Bugs: HELIX-382 > > > Repository: helix-git > > > Description > ------- > > commit 4c792fd2908cbf8cbfe051dbe31f25bf0a1094df > Author: Sandeep Nayak <[email protected]> > Date: Sat Feb 15 23:25:15 2014 -0800 > > [HELIX-382] GenericHelixController now implements > InstanceConfigChangeListener, ConfigChangeListener delegates call to > InstanceConfigChangeListener > > :100644 100644 e9924a2... 9481802... M > helix-core/src/main/java/org/apache/helix/controller/GenericHelixController.java > > > Diffs > ----- > > > helix-core/src/main/java/org/apache/helix/controller/GenericHelixController.java > e9924a2 > > Diff: https://reviews.apache.org/r/18167/diff/ > > > Testing > ------- > > > Thanks, > > Sandeep Nayak > >
