----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73296/#review222872 -----------------------------------------------------------
Ship it! Ship It! - Sarath Subramanian On April 26, 2021, 4:40 p.m., Ashutosh Mestry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73296/ > ----------------------------------------------------------- > > (Updated April 26, 2021, 4:40 p.m.) > > > Review request for atlas, Jayendra Parab and Sarath Subramanian. > > > Bugs: ATLAS-4256 and ATLAS-4258 > https://issues.apache.org/jira/browse/ATLAS-4256 > https://issues.apache.org/jira/browse/ATLAS-4258 > > > Repository: atlas > > > Description > ------- > > **Background** > Please read bug description. > > **Approach** > New: *TaskFactoryRegistry*: Registration process for individual factory. > Modified: *AdminResource* Removed the postConstruct calling of > *queuePendingTasks*. This is now part of *TaskManagement*, which is better > design. > > > Diffs > ----- > > common/src/main/java/org/apache/atlas/repository/Constants.java 4df38a5e6 > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java > 276343e0f > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java > 20d5e6fab > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasGraphUtilsV2.java > 8d4fdf388 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphMapper.java > d8ef32b32 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/tasks/ClassificationPropagateTaskFactory.java > 6244b2d5d > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/tasks/ClassificationPropagationTasks.java > 4fda34aca > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/tasks/ClassificationTask.java > 369db087b > repository/src/main/java/org/apache/atlas/tasks/TaskFactoryRegistry.java > PRE-CREATION > repository/src/main/java/org/apache/atlas/tasks/TaskManagement.java > 264aa8cfb > > repository/src/test/java/org/apache/atlas/repository/tagpropagation/ClassificationPropagationWithTasksTest.java > e309a7658 > webapp/src/main/java/org/apache/atlas/web/resources/AdminResource.java > 714b400c8 > > > Diff: https://reviews.apache.org/r/73296/diff/6/ > > > Testing > ------- > > **Functional tests** > Manually simulated failover scenarios. > > Steps: > - Start Atlas on a cluster with multiple instances. > - Start propagation task that takes few minutes to complete. > - Shutdown active instance. > > Expected behavior: Task should be started from the active node. > > > Thanks, > > Ashutosh Mestry > >
