> On July 19, 2017, 7:42 p.m., Apoorv Naik wrote: > > repository/src/main/java/org/apache/atlas/discovery/SearchProcessor.java > > Line 181 (original), 181 (patched) > > <https://reviews.apache.org/r/60980/diff/1/?file=1779714#file1779714line181> > > > > Do it make more sense to pass a boolean flag here to control the > > addition of the state to the query ? > > > > EntityProcessor will pass true whereas ClassificationProcessor will > > pass false
I choose to read from context, as it is already available in SearchProcessor - avoid the need to pass additional parameter. - Madhan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60980/#review180967 ----------------------------------------------------------- On July 19, 2017, 9:59 p.m., Madhan Neethiraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60980/ > ----------------------------------------------------------- > > (Updated July 19, 2017, 9:59 p.m.) > > > Review request for atlas. > > > Bugs: ATLAS-1961 > https://issues.apache.org/jira/browse/ATLAS-1961 > > > Repository: atlas > > > Description > ------- > > - fixed NPE when search includes a free-text query & a classification that > has large number of sub-types > - fixed entity search in Solr to include state filter > > > Diffs > ----- > > > repository/src/main/java/org/apache/atlas/discovery/ClassificationSearchProcessor.java > 77b2c7c2 > > repository/src/main/java/org/apache/atlas/discovery/EntitySearchProcessor.java > 50376ef1 > > repository/src/main/java/org/apache/atlas/discovery/FullTextSearchProcessor.java > 83368c2b > repository/src/main/java/org/apache/atlas/discovery/SearchProcessor.java > 596b43bc > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasGraphUtilsV1.java > cd9a47ad > > > Diff: https://reviews.apache.org/r/60980/diff/2/ > > > Testing > ------- > > - validated that the fix addressed the error cases listed above > > > Thanks, > > Madhan Neethiraj > >
