----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20182/#review39955 -----------------------------------------------------------
helix-core/src/main/java/org/apache/helix/model/ClusterConstraints.java <https://reviews.apache.org/r/20182/#comment72737> remove this todo helix-core/src/test/java/org/apache/helix/integration/TestPartitionLevelMsgThrottle.java <https://reviews.apache.org/r/20182/#comment72739> this is not thread-safe -- use a concurrent list - Kanak Biscuitwala On April 9, 2014, 3:29 p.m., Zhen Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20182/ > ----------------------------------------------------------- > > (Updated April 9, 2014, 3:29 p.m.) > > > Review request for helix, Kanak Biscuitwala and Kishore Gopalakrishna. > > > Repository: helix-git > > > Description > ------- > > add support for partition level constraint in 0.6 > > > Diffs > ----- > > helix-core/src/main/java/org/apache/helix/model/ClusterConstraints.java > f69a7ce > > helix-core/src/test/java/org/apache/helix/integration/TestPartitionLevelMsgThrottle.java > e69de29 > > Diff: https://reviews.apache.org/r/20182/diff/ > > > Testing > ------- > > add a new test: TestPartitionLevelMsgThrottle > > > Thanks, > > Zhen Zhang > >
