----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62679/#review186644 -----------------------------------------------------------
repository/src/main/java/org/apache/atlas/util/SearchPredicateUtil.java Lines 564 (patched) <https://reviews.apache.org/r/62679/#comment263400> Why ALWAYS_TRUE here? Shouldn't the result be true only when the attribute value is NULL? repository/src/main/java/org/apache/atlas/util/SearchPredicateUtil.java Lines 591 (patched) <https://reviews.apache.org/r/62679/#comment263401> Why ALWAYS_TRUE here? Shouldn't the result be true only when the attribute value is NOT NULL? - Madhan Neethiraj On Sept. 29, 2017, 1:08 a.m., Apoorv Naik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62679/ > ----------------------------------------------------------- > > (Updated Sept. 29, 2017, 1:08 a.m.) > > > Review request for atlas and Madhan Neethiraj. > > > Bugs: ATLAS-2184 > https://issues.apache.org/jira/browse/ATLAS-2184 > > > Repository: atlas > > > Description > ------- > > Search parameters in basic search now include is_null, not_null operators > which can prove useful in scenarios where user is interested in checking the > presence/absence of certain entity attributes > > > Diffs > ----- > > intg/src/main/java/org/apache/atlas/model/discovery/SearchParameters.java > a7a71b78 > repository/src/main/java/org/apache/atlas/discovery/SearchProcessor.java > b380e1e4 > repository/src/main/java/org/apache/atlas/util/SearchPredicateUtil.java > 18d77ee7 > > > Diff: https://reviews.apache.org/r/62679/diff/1/ > > > Testing > ------- > > Tested using various combinations of attributes and IS_NULL/NOT_NULL operator > via REST client/ curl > > > Thanks, > > Apoorv Naik > >
