-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72127/
-----------------------------------------------------------

(Updated Feb. 21, 2020, 5:24 a.m.)


Review request for atlas, Madhan Neethiraj, Nixon Rodrigues, and Sarath 
Subramanian.


Changes
-------

removed empty line and combined if condition


Bugs: ATLAS-3615
    https://issues.apache.org/jira/browse/ATLAS-3615


Repository: atlas


Description
-------

Issues:
1) Combination of typeName:_ALL_ENTITY_TYPES and classification doesn't return 
result
Why? As _ALL_ENTITY_TYPES value is assigned as [* TO *] results show up when 
searched from solr but not from graphdb

2) Combination of typeName:_ALL_ENTITY_TYPES, classification, 
query(SearchByText) doesn't return result
Why? result from solr, thru "/freetext" is correct, but onsearching if resulted 
entitytype is in typeAndSubtypes of given entityType, it breaks because 
typeAndSubTypes of _ALL_ENTITY_TYPES is [* TO *]

3) Combination of typeName, classification:_ALL_CLASSIFICATION_TYPES doesn't 
return result

4) Combination of typeName:_ALL_ENTITY_TYPES, 
classification:_ALL_CLASSIFICATION_TYPES, query(SearchByText) doesn't return 
correct result
why? query formed with above combination in solr /freetext is 'SAMPLEQUERY' AND 
[* TO *] AND [* TO *]

5) Combination of classification:_ALL_CLASSIFICATION_TYPES and 
classificationFilter doesn't return correct result


Diffs (updated)
-----

  
repository/src/main/java/org/apache/atlas/discovery/ClassificationSearchProcessor.java
 c0a5a46dd 
  
repository/src/main/java/org/apache/atlas/discovery/EntitySearchProcessor.java 
b5606d046 
  
repository/src/main/java/org/apache/atlas/discovery/FreeTextSearchProcessor.java
 d628f1352 
  repository/src/main/java/org/apache/atlas/discovery/SearchContext.java 
353411363 
  repository/src/main/java/org/apache/atlas/discovery/SearchProcessor.java 
8c17a9044 


Diff: https://reviews.apache.org/r/72127/diff/4/

Changes: https://reviews.apache.org/r/72127/diff/3-4/


Testing
-------

For issue1:
1) tested with values _ALL_ENTITY_TYPES + EntityFilter + classification -> 
result fetches from gremlin and shows up correct result
2) tested with values _ALL_ENTITY_TYPES + classification -> result fetches from 
gremlin and shows up correct result

For issue2:
1) tested with values _ALL_ENTITY_TYPES + classification + query(SearchByText) 
-> shows up correct result


Thanks,

Pinal Shah

Reply via email to