Github user upthewaterspout commented on a diff in the pull request:
https://github.com/apache/geode/pull/503#discussion_r115623920
--- Diff:
geode-lucene/src/main/java/org/apache/geode/cache/lucene/LuceneQuery.java ---
@@ -37,21 +42,38 @@
* Results are returned in order of their score with respect to this query.
* </p>
*/
-@Experimental
public interface LuceneQuery<K, V> {
/**
* Execute the query and return the region keys that match this query,
up to the limit specified
* by {@link #getLimit()}.
- *
+ *
+ * @return Collection of Apache Geode region keys that satisfy the
Lucene query.
* @throws LuceneQueryException if the query could not be parsed or
executed.
+ * @throws CacheClosedException if the cache was closed while the Lucene
query was being executed.
+ * @throws PrimaryBucketException if the primary bucket was moved from
the node on which the
--- End diff --
PrimaryBucketException is an internal exception, which should not be part
of the API. Do we actually throw this exception from this method? It seems like
we should be retrying in any case.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
