> On Sept. 14, 2019, 7:16 p.m., Madhan Neethiraj wrote: > > repository/src/main/java/org/apache/atlas/discovery/EntitySearchProcessor.java > > Lines 205 (patched) > > <https://reviews.apache.org/r/71431/diff/2/?file=2165082#file2165082line205> > > > > graphQuery requires 'qualifiedName' of the attribute (i.e. Asset.name, > > Asset.owner) - refer to SearchProcessor.toGraphFilterQuery(). Is 'sortBy' > > here qualifiedName of the attribute? > > Bolke de Bruin wrote: > Good point. It seems (but please correct me if I am wrong) given the > tests that it is not required. However, I assume I will need to update > SearchParameters to validate whether the attribute exists? > > Madhan Neethiraj wrote: > Validation of attribute names is handled in > SearchContext.validateAttributes(); consider updating this method to validate > the new field SearchParameters.sortBy. > > Bolke de Bruin wrote: > Can you search on classificationType and entityType at the same time? I'm > assuming that I need to verify the sortBy parameter against entityType and > classifictionType. It seems a bit odd though
Ah used the ui to find out a bit more. Now I am going to assume I just need to validate against entityType. - Bolke ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71431/#review217738 ----------------------------------------------------------- On Sept. 16, 2019, 6:45 p.m., Bolke de Bruin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71431/ > ----------------------------------------------------------- > > (Updated Sept. 16, 2019, 6:45 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-3399 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/ATLAS-3399 > > > Repository: atlas > > > Description > ------- > > Add indexed order by to basic search. > > > Diffs > ----- > > > graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasIndexQuery.java > f3722b827 > > graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusIndexQuery.java > 7c258b7a3 > graphdb/janus/src/main/java/org/janusgraph/diskstorage/solr/Solr6Index.java > e457866af > intg/src/main/java/org/apache/atlas/model/discovery/SearchParameters.java > aac6b5aa3 > > repository/src/main/java/org/apache/atlas/discovery/ClassificationSearchProcessor.java > 479ddfd89 > > repository/src/main/java/org/apache/atlas/discovery/EntitySearchProcessor.java > f7d8f08c7 > repository/src/main/java/org/apache/atlas/discovery/SearchContext.java > 9bd0382eb > webapp/src/main/java/org/apache/atlas/web/rest/DiscoveryREST.java e6b338b84 > webapp/src/test/java/org/apache/atlas/web/integration/BasicSearchIT.java > 06931b38c > webapp/src/test/resources/json/search-parameters/entity-filters.json > f4b2efe63 > webapp/src/test/resources/json/search-parameters/tag-filters.json 5e74328d6 > > > Diff: https://reviews.apache.org/r/71431/diff/4/ > > > Testing > ------- > > - production and tests added > > > Thanks, > > Bolke de Bruin > >
