----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73271/#review222816 -----------------------------------------------------------
repository/src/test/java/org/apache/atlas/discovery/AtlasDiscoveryServiceTest.java Lines 614 (patched) <https://reviews.apache.org/r/73271/#comment311999> Please let me know in that case how the "search_name_with\ nameblank*" retunring one result where as "search_qf_with\ qfblank*" returning 0 result. repository/src/test/java/org/apache/atlas/discovery/AtlasDiscoveryServiceTest.java Lines 706 (patched) <https://reviews.apache.org/r/73271/#comment311998> nit: Please add one space after searchResult.getSearchResults(). Similarly please do other places. - Sidharth Mishra On April 8, 2021, 4:13 a.m., Pinal Shah wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73271/ > ----------------------------------------------------------- > > (Updated April 8, 2021, 4:13 a.m.) > > > Review request for atlas, Jayendra Parab, Madhan Neethiraj, Nixon Rodrigues, > and Sarath Subramanian. > > > Bugs: ATLAS-4238 > https://issues.apache.org/jira/browse/ATLAS-4238 > > > Repository: atlas > > > Description > ------- > > **Current working:** During freetext/quick search, wildcard '*' is appended > after the search string from UI. > Whenever attribute search value has "**+-&|!(){}[]^"~?:**" special > characters, then '*' will not be appended to get the correct result. > > ('*' is removed because for TEXT type attribute, it tokenize the value on > special characters) > > **Issue:** '@' or 'digit+.' etc there are many other special characters which > gets tokenize for 'TEXT' type attribute. hence if the search value has these > special characters which are not added in above list, it doesn't give results. > > **Solution:** We need to handle all the special characters which tokenize the > string > > > Diffs > ----- > > > repository/src/main/java/org/apache/atlas/discovery/EntityDiscoveryService.java > 4ae2d1279 > > repository/src/test/java/org/apache/atlas/discovery/AtlasDiscoveryServiceTest.java > 9846d4310 > > > Diff: https://reviews.apache.org/r/73271/diff/1/ > > > Testing > ------- > > added test cases > Precommit: > https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/487/console > > > Thanks, > > Pinal Shah > >
