----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70543/#review214872 -----------------------------------------------------------
Fix it, then Ship it! repository/src/main/java/org/apache/atlas/repository/store/bootstrap/AtlasTypeDefStoreInitializer.java Lines 99 (patched) <https://reviews.apache.org/r/70543/#comment301150> AtlasPatchRegistry doesn't have to be a member; instance created in startInternal() can be passed down to loadBootstrapTypeDefs() and other methods that are called from there. - Madhan Neethiraj On April 25, 2019, 12:02 a.m., Ashutosh Mestry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70543/ > ----------------------------------------------------------- > > (Updated April 25, 2019, 12:02 a.m.) > > > Review request for atlas, Madhan Neethiraj, Nixon Rodrigues, and Sarath > Subramanian. > > > Bugs: ATLAS-3168 > https://issues.apache.org/jira/browse/ATLAS-3168 > > > Repository: atlas > > > Description > ------- > > PatchService should implement _ActiveStateChangeHandler_ so that it gets > events from ZK electoral election. > > > Diffs > ----- > > > repository/src/main/java/org/apache/atlas/repository/audit/HBaseBasedAuditRepository.java > 6e8dbe9ad > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java > c57f8e3c7 > > repository/src/main/java/org/apache/atlas/repository/patches/AtlasPatchService.java > fc2128529 > > repository/src/main/java/org/apache/atlas/repository/store/bootstrap/AtlasTypeDefStoreInitializer.java > c3356edd4 > > server-api/src/main/java/org/apache/atlas/listener/ActiveStateChangeHandler.java > c38e53c37 > > webapp/src/main/java/org/apache/atlas/web/service/ActiveInstanceElectorService.java > 10081ac9c > > > Diff: https://reviews.apache.org/r/70543/diff/3/ > > > Testing > ------- > > **Functional tests** > Verified in HA environment with kerberos and ssl enabled. > > **Unit tests** > Existing tests. > > > Thanks, > > Ashutosh Mestry > >
