----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72182/#review219876 -----------------------------------------------------------
Ship it! Ship It! - Nixon Rodrigues On March 5, 2020, 9:50 a.m., mayank jain wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72182/ > ----------------------------------------------------------- > > (Updated March 5, 2020, 9:50 a.m.) > > > Review request for atlas, Madhan Neethiraj, Nixon Rodrigues, and Sarath > Subramanian. > > > Bugs: ATLAS-3647 > https://issues.apache.org/jira/browse/ATLAS-3647 > > > Repository: atlas > > > Description > ------- > > In entity definition , > isIncomplete is null when is entity is complete > isIncomplete is 1 when is entity is incomplete (shell/ghost entities) > > when isIncomplete = false is expected to return all complete entities > (non-shell entities) but since isIncomplete is null , it doesn't return any > entity. > > In system attributes search, > isIncomplete takes conditions =, != , not null , null with values true , > false. > > > Solution : > Basically while creating normal entities isIncomplete attribute is nowhere > into consideration and also it only comes into picture for shell entities. > > So, when we try searching for isIncomplete = false , the normal entities > which does not hold any value for this particular attribute we can directly > alter the graph query search with an OR condition. > > i.e _isIncomplete = false OR _isIncomplete is null > > This will return all the entities which were once a shell entity and then > got updated to full entity and all the normal entities which never went into > the process of shell entities and have isIncomplete attribute as null. > > > Diffs > ----- > > repository/src/main/java/org/apache/atlas/discovery/SearchProcessor.java > 356363db0 > > > Diff: https://reviews.apache.org/r/72182/diff/3/ > > > Testing > ------- > > Tested the complete working of isIncomplete attribute and it works fine. > > > Thanks, > > mayank jain > >
