----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71282/#review217742 -----------------------------------------------------------
intg/src/main/java/org/apache/atlas/model/discovery/AtlasSearchResult.java Line 82 (original), 84 (patched) <https://reviews.apache.org/r/71282/#comment305101> approximateCount should initialized in this constructor as well. It might be simpler to initialize in line #55. repository/src/main/java/org/apache/atlas/discovery/ClassificationSearchProcessor.java Lines 356 (patched) <https://reviews.apache.org/r/71282/#comment305102> Shouldn't the return value be "-1", since the processor wouldn't know the result-count for non-index query case. repository/src/main/java/org/apache/atlas/discovery/EntitySearchProcessor.java Lines 320 (patched) <https://reviews.apache.org/r/71282/#comment305103> Shouldn't the return value be "-1", since the processor wouldn't know the result-count for non-index query case. - Madhan Neethiraj On Sept. 14, 2019, 6:03 p.m., Bolke de Bruin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71282/ > ----------------------------------------------------------- > > (Updated Sept. 14, 2019, 6:03 p.m.) > > > Review request for atlas, Ashutosh Mestry, Aadarsh Jajodia, Sridhar K, Le Ma, > Madhan Neethiraj, and Nixon Rodrigues. > > > Bugs: https://issues.apache.org/jira/browse/ATLAS-3367 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/ATLAS-3367 > > > Repository: atlas > > > Description > ------- > > Both ElasticSearch and SOLR can expose the total count > of results without returning all results. This is useful > to give the user or client an idea how many results there are. > > This patch ensures the total is returned if available. This total > is an approximate as scrubbing of the results still needs to happen. > Therefore, one should not only rely on this information to provide > ,for example, pagination. > > > Diffs > ----- > > intg/src/main/java/org/apache/atlas/model/discovery/AtlasSearchResult.java > d274cc07e > > repository/src/main/java/org/apache/atlas/discovery/ClassificationSearchProcessor.java > 479ddfd89 > > repository/src/main/java/org/apache/atlas/discovery/EntityDiscoveryService.java > f667aa399 > > repository/src/main/java/org/apache/atlas/discovery/EntitySearchProcessor.java > f7d8f08c7 > > repository/src/main/java/org/apache/atlas/discovery/FreeTextSearchProcessor.java > 1dd1afaa3 > > repository/src/main/java/org/apache/atlas/discovery/FullTextSearchProcessor.java > 0ffd61c07 > repository/src/main/java/org/apache/atlas/discovery/SearchProcessor.java > f7847edff > > repository/src/main/java/org/apache/atlas/discovery/TermSearchProcessor.java > c253d544b > > > Diff: https://reviews.apache.org/r/71282/diff/4/ > > > Testing > ------- > > > Thanks, > > Bolke de Bruin > >