----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54767/#review159339 -----------------------------------------------------------
extensions/geode-modules/src/test/java/org/apache/geode/modules/util/ClassLoaderObjectInputStreamTest.java (line 109) <https://reviews.apache.org/r/54767/#comment230306> Maybe this should not be an inner class, if it's something we want to reuse for other tests? - Dan Smith On Dec. 15, 2016, 1:05 a.m., Jason Huynh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54767/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2016, 1:05 a.m.) > > > Review request for geode, Kirk Lund and Dan Smith. > > > Repository: geode > > > Description > ------- > > Use the tccl if the regular classloader fails > > > Diffs > ----- > > extensions/geode-modules/build.gradle 0ac8c9a > > extensions/geode-modules/src/main/java/org/apache/geode/modules/util/ClassLoaderObjectInputStream.java > d5ebbab > > extensions/geode-modules/src/test/java/org/apache/geode/modules/util/ClassLoaderObjectInputStreamTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/54767/diff/ > > > Testing > ------- > > > Thanks, > > Jason Huynh > >
