> On Oct. 26, 2016, 9:39 p.m., Kirk Lund wrote: > > geode-core/src/test/java/org/apache/geode/internal/security/IntegratedSecurityServiceTest.java, > > line 21 > > <https://reviews.apache.org/r/53168/diff/2/?file=1546442#file1546442line21> > > > > This should be: > > > > import static org.assertj.core.api.Assertions.assertThatThrownBy; > > > > Don't use Java6Assertions.
Oh, yeah, thanks for catching this. - Jinmei ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53168/#review153948 ----------------------------------------------------------- On Oct. 26, 2016, 9:12 p.m., Jinmei Liao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53168/ > ----------------------------------------------------------- > > (Updated Oct. 26, 2016, 9:12 p.m.) > > > Review request for geode, John Blum, Kevin Duling, and Kirk Lund. > > > Repository: geode > > > Description > ------- > > GEODE-2030: security support for SDG > > * added cacheFactory.setSecurityManager and cacheFactory.setPostProcessor > * The isIntegragtedSecurity flag is set by checking if the Shiro's > securityManger is configured or not. > > > Diffs > ----- > > geode-core/src/main/java/org/apache/geode/cache/CacheFactory.java > b62feac4b2a3ae15e940afb50f71abf61e5eee50 > geode-core/src/main/java/org/apache/geode/internal/cache/CacheConfig.java > 91ae33304a1e82fa7c02c97edce2318d359220e2 > > geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java > d9d572c45ac559e611fe2dc86d1028d39d1e592c > > geode-core/src/main/java/org/apache/geode/internal/security/IntegratedSecurityService.java > 9f4697f6c9f06dbfd671e9e0d80dc52f447e5829 > > geode-core/src/main/java/org/apache/geode/internal/security/SecurityService.java > 41b08d510876f95733c96d103b0826726d9a09bb > > geode-core/src/test/java/org/apache/geode/internal/security/IntegratedSecurityServiceTest.java > ee76dfc095c3aeb43a04cee899ba4b434c1e552e > > geode-core/src/test/java/org/apache/geode/security/CacheFactoryWithSecurityObjectTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/53168/diff/ > > > Testing > ------- > > precheckin successful > > > Thanks, > > Jinmei Liao > >
