----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36747/#review92926 -----------------------------------------------------------
Ship it! Ship It! - Kirk Lund On July 23, 2015, 9:49 p.m., Darrel Schneider wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36747/ > ----------------------------------------------------------- > > (Updated July 23, 2015, 9:49 p.m.) > > > Review request for geode and Kirk Lund. > > > Repository: geode > > > Description > ------- > > The code now uses an ArrayList and only adds non-null elements to it. > It then allocates a array of that ArrayList size and returns it with > the contents of the ArrayList. > > > Diffs > ----- > > > gemfire-core/src/main/java/com/gemstone/gemfire/management/internal/beans/MemberMBeanBridge.java > 6a2d0850d40dac2ef247d021fb8ee755afc02dd8 > > Diff: https://reviews.apache.org/r/36747/diff/ > > > Testing > ------- > > MemberMBeanAttributesDUnitTest (this code that calls fetchJvmThreads). > > > Thanks, > > Darrel Schneider > >