-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25658/
-----------------------------------------------------------
(Updated Sept. 25, 2014, 7:16 p.m.)
Review request for lucene.
Changes
-------
Patch that integrates Steve's changes and also changes for AtomicReader ->
LeafReader refactor.
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/QueryTimeoutImpl.java
PRE-CREATION
trunk/lucene/core/src/java/org/apache/lucene/search/TimeLimitingCollector.java
1627584
trunk/lucene/core/src/test/org/apache/lucene/index/TestExitableDirectoryReader.java
PRE-CREATION
trunk/lucene/core/src/test/org/apache/lucene/search/TestTimeLimitingCollector.java
1627584
trunk/solr/core/src/java/org/apache/solr/handler/MoreLikeThisHandler.java
1627584
trunk/solr/core/src/java/org/apache/solr/handler/component/QueryComponent.java
1627584
trunk/solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java
1627584
trunk/solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java
1627584
trunk/solr/core/src/java/org/apache/solr/search/SolrQueryTimeoutImpl.java
PRE-CREATION
trunk/solr/core/src/test/org/apache/solr/TestDistributedSearch.java 1627584
trunk/solr/core/src/test/org/apache/solr/TestGroupingSearch.java 1627584
trunk/solr/core/src/test/org/apache/solr/cloud/CloudExitableDirectoryReaderTest.java
PRE-CREATION
trunk/solr/core/src/test/org/apache/solr/core/ExitableDirectoryReaderTest.java
PRE-CREATION
trunk/solr/solrj/src/java/org/apache/solr/common/params/SolrParams.java
1627584
Diff: https://reviews.apache.org/r/25658/diff/
Testing
-------
Added Lucene/Solr tests. Tested a bit manually.
Thanks,
Anshum Gupta
