----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36714/#review92822 -----------------------------------------------------------
Ship it! Ship It! - Kirk Lund On July 23, 2015, 12:15 a.m., Darrel Schneider wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36714/ > ----------------------------------------------------------- > > (Updated July 23, 2015, 12:15 a.m.) > > > Review request for geode and Kirk Lund. > > > Repository: geode > > > Description > ------- > > A more powerful StringUtils.forceToString now exists that will also > print out the elements of arrays. It used to just to a toString on the > array which only shows the array identity. The new code will (by default) > show the first 16 element values and after that say how many more the array > had that are not shown. If you set the sys prop > StringUtils.MAX_ARRAY_ELEMENTS_TO_CONVERT > you can change the default of 16. > > > Diffs > ----- > > > gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegionEntry.java > 809996bb62862895c4de64713ef3347885d3d5b9 > > gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/EntryEventImpl.java > 25466dabdbe6e6e189ae465e6a36a122f33a99f9 > > gemfire-core/src/main/java/com/gemstone/gemfire/internal/lang/StringUtils.java > f5a467c3d941a9ea060180666d08ccf3e45fb9ac > > gemfire-core/src/main/java/com/gemstone/gemfire/internal/offheap/OffHeapCachedDeserializable.java > abba827f86ba37d7257071f48a3def1c604664e1 > > gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/ChunkValueWrapperJUnitTest.java > 229719f59bbf7db83647e9b25b36a8ba48c446bf > > gemfire-core/src/test/java/com/gemstone/gemfire/internal/lang/StringUtilsJUnitTest.java > 1420fbd977b4a1361213a2baf54947afd8aa00b7 > > Diff: https://reviews.apache.org/r/36714/diff/ > > > Testing > ------- > > precheckin > > > Thanks, > > Darrel Schneider > >