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

(Updated Sept. 16, 2014, 9:40 p.m.)


Review request for lucene.


Changes
-------

Changed the design to have an abstract/base QueryTimeout class and 
implementations that use local var and ThreadLocal for Lucene/Solr. Changed the 
Lucene test to use the non-ThreadLocal implementation and SolrIndexSearcher 
uses the ThreadLocal implementation of QueryTimeout.


Bugs: SOLR-5986
    https://issues.apache.org/jira/browse/SOLR-5986


Repository: lucene


Description
-------

Timeout queries when they take too long to rewrite/enumerate over terms.


Diffs (updated)
-----

  
trunk/lucene/core/src/java/org/apache/lucene/index/ExitableDirectoryReader.java 
PRE-CREATION 
  trunk/lucene/core/src/java/org/apache/lucene/index/QueryTimeout.java 
PRE-CREATION 
  trunk/lucene/core/src/java/org/apache/lucene/index/QueryTimeoutBase.java 
PRE-CREATION 
  
trunk/lucene/core/src/test/org/apache/lucene/index/TestExitableDirectoryReader.java
 PRE-CREATION 
  trunk/solr/core/src/java/org/apache/solr/handler/MoreLikeThisHandler.java 
1625349 
  trunk/solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java 
1625349 
  trunk/solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java 
1625349 
  trunk/solr/core/src/java/org/apache/solr/search/SolrQueryTimeout.java 
PRE-CREATION 
  trunk/solr/core/src/test/org/apache/solr/TestDistributedSearch.java 1625349 
  trunk/solr/core/src/test/org/apache/solr/TestGroupingSearch.java 1625349 
  
trunk/solr/core/src/test/org/apache/solr/core/ExitableDirectoryReaderTest.java 
PRE-CREATION 

Diff: https://reviews.apache.org/r/25658/diff/


Testing
-------

Added Lucene/Solr tests. Tested a bit manually.


Thanks,

Anshum Gupta

Reply via email to