Friendly ping. Sent from my iPhone
Begin forwarded message: > From: Bolke de Bruin <[email protected]> > Date: 16 September 2019 at 21:11:17 CEST > To: Aadarsh Jajodia <[email protected]>, Ashutosh Mestry > <[email protected]>, Sridhar K <[email protected]>, Le Ma > <[email protected]>, Nixon Rodrigues > <[email protected]>, Madhan Neethiraj <[email protected]> > Cc: Sarath Subramanian <[email protected]>, atlas > <[email protected]>, Bolke de Bruin <[email protected]> > Subject: Re: Review Request 71431: Add indexed order by (sort) to basic search > Reply-To: Bolke de Bruin <[email protected]> > > > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71431/ > > On September 16th, 2019, 7:05 p.m. UTC, Madhan Neethiraj wrote: > > repository/src/main/java/org/apache/atlas/discovery/SearchContext.java (Diff > revision 4) > 272 > FilterCriteria filterCriteria = new FilterCriteria(); > Instead of creating FilterCriteria, consider using lines #263 - #265 here: > if (StringUtils.isNotEmpty(attributeName) && > structType.getAttributeType(attributeName) == null) { > throw new AtlasBaseException(AtlasErrorCode.UNKNOWN_ATTRIBUTE, > attributeName, structType.getTypeName()); > > Better yet, update 'else' block at #260 to call this method: > } else { > validateAttribute(structType, filterCriteria.getAttributeName(); > } > Nice. I was struggling with that a bit. Fixed. > > - Bolke > > > On September 16th, 2019, 7:10 p.m. UTC, Bolke de Bruin wrote: > > Review request for atlas, Ashutosh Mestry, Aadarsh Jajodia, Sridhar K, Le Ma, > Madhan Neethiraj, and Nixon Rodrigues. > By Bolke de Bruin. > Updated Sept. 16, 2019, 7:10 p.m. > > Bugs: https://issues.apache.org/jira/browse/ATLAS-3399 > Repository: atlas > Description > > Add indexed order by to basic search. > Testing > > production and tests added > 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) > View Diff
