----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47850/#review134835 -----------------------------------------------------------
geode-core/src/test/java/com/gemstone/gemfire/management/internal/security/JSONAuthorization.java (line 14) <https://reviews.apache.org/r/47850/#comment199766> What about the ExampleJSONAuthorization? Do we need to change that also? geode-core/src/test/resources/com/gemstone/gemfire/management/internal/security/cacheServer.json (line 82) <https://reviews.apache.org/r/47850/#comment199765> we can get rid of the null now. - Jinmei Liao On May 25, 2016, 8:27 p.m., Jens Deppe wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47850/ > ----------------------------------------------------------- > > (Updated May 25, 2016, 8:27 p.m.) > > > Review request for geode, Jinmei Liao, Kevin Duling, and Kirk Lund. > > > Repository: geode > > > Description > ------- > > - Also converted to Jackson. Be gone org.json!! > > > Diffs > ----- > > > geode-core/src/test/java/com/gemstone/gemfire/management/internal/security/JSONAuthorization.java > e14d1def97a06e479cbeeffe56a05e8e8de16d65 > > geode-core/src/test/resources/com/gemstone/gemfire/management/internal/security/cacheServer.json > fbbda8d0369bd7fdf400ba226c67d4c1861de62e > > Diff: https://reviews.apache.org/r/47850/diff/ > > > Testing > ------- > > > Thanks, > > Jens Deppe > >
