-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70789/
-----------------------------------------------------------
(Updated June 6, 2019, 8:18 p.m.)
Review request for atlas, Ashutosh Mestry, Aadarsh Jajodia, keval bhatt,
Sridhar K, Le Ma, Nikhil Bonte, Nixon Rodrigues, Saqeeb Shaikh, and Sarath
Subramanian.
Changes
-------
addressed few review comments, also some refactoring
Bugs: ATLAS-3262
https://issues.apache.org/jira/browse/ATLAS-3262
Repository: atlas
Description
-------
- skipped full-text index computation & population when free-text is enabled
- added FreeTextRequestHandlerPatch to create /freetext request handler in Solr
- fixed free-text search handling of typeName filter
Diffs (updated)
-----
graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusGraph.java
edab08cc0
graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusGraphSolrIndexClient.java
a55fc3689
intg/src/main/java/org/apache/atlas/ApplicationProperties.java 6d0153f7d
repository/src/main/java/org/apache/atlas/discovery/ClassificationSearchProcessor.java
1cd507c3b
repository/src/main/java/org/apache/atlas/discovery/EntitySearchProcessor.java
c4b5ffb12
repository/src/main/java/org/apache/atlas/discovery/FreeTextSearchProcessor.java
b1a84c742
repository/src/main/java/org/apache/atlas/discovery/SearchContext.java
50edb9faf
repository/src/main/java/org/apache/atlas/discovery/SearchProcessor.java
2e09beda0
repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java
56655a845
repository/src/main/java/org/apache/atlas/repository/graph/SolrIndexHelper.java
20a517ff1
repository/src/main/java/org/apache/atlas/repository/patches/AtlasPatchManager.java
259f24674
repository/src/main/java/org/apache/atlas/repository/patches/FreeTextRequestHandlerPatch.java
PRE-CREATION
repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityChangeNotifier.java
035b02ce4
repository/src/main/java/org/apache/atlas/util/AtlasRepositoryConfiguration.java
1d296057b
repository/src/main/java/org/apache/atlas/util/SearchPredicateUtil.java
38de885cf
Diff: https://reviews.apache.org/r/70789/diff/2/
Changes: https://reviews.apache.org/r/70789/diff/1-2/
Testing
-------
- verified that free-text handler is created by the patch for the first time
this enhancement is applied in an existing env
- verified that the free-text search handles typeName filter correctly
- pre-commit tests run:
https://builds.apache.org/view/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/1176/
Thanks,
Madhan Neethiraj
