----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60010/#review177650 -----------------------------------------------------------
Ship it! Ship It! - Jinmei Liao On June 12, 2017, 5:40 p.m., Kirk Lund wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60010/ > ----------------------------------------------------------- > > (Updated June 12, 2017, 5:40 p.m.) > > > Review request for geode, Emily Yeh, Jinmei Liao, Jared Stewart, Ken Howe, > and Patrick Rhomberg. > > > Bugs: GEODE-3062 > https://issues.apache.org/jira/browse/GEODE-3062 > > > Repository: geode > > > Description > ------- > > Add new test to ClusterConfigWithSecurityDUnitTest that fails due to bug > GEODE-3062. > > Remove unused Cache param from applyClusterPropertiesConfiguration so it can > be called during Cache construction. > > Move cluster config request to Cache construction and handle jars and > properties there. Create new SecurityService in constructor and overwrite the > SecurityService in InternalDistributedSystem. > > NOTE: We will later have to fix GEODE-3061 by moving cluster config request > from Cache to InternalDistributedSystem construction so that IDS can be > created with gemfire.properties from cluster config. At that time we would > rip out both cluster config request and creation of security service from > Cache construction and pass both into Cache construction. > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java > 22edb6f06c7791929cc9a033ca1a1bfed5751a47 > > geode-core/src/main/java/org/apache/geode/internal/cache/ClusterConfigurationLoader.java > 4f4881fe39116faa505bc2fbec74efd669efe0ef > > geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java > 40df0c7dcac8827a381c268c1f90e6acfb97a7f1 > > geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigWithSecurityDUnitTest.java > c551ca9104a85dcf54c0bebbc4178fce4114a416 > > > Diff: https://reviews.apache.org/r/60010/diff/1/ > > > Testing > ------- > > Precheckin passes > > > Thanks, > > Kirk Lund > >
