Github user karensmolermiller commented on the issue:
https://github.com/apache/geode/pull/518
Closed by merge: 096c22d5c73dc609651caf2887b4d95f162230ad.
---
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 doe
Github user karensmolermiller commented on the issue:
https://github.com/apache/geode/pull/518
Thanks @boglesby . I've incorporated all suggestions but the last one in
your review. When we change the error message for destroying a region (before
the associated index), then let's also
Github user boglesby commented on the issue:
https://github.com/apache/geode/pull/518
The default analyzer is Geode's not Lucene's so I think the sentence should
be something like:
When no analyzer is specified, the
org.apache.lucene.analysis.standard.StandardAnalyzer will be
Github user karensmolermiller commented on the issue:
https://github.com/apache/geode/pull/518
Commit
https://github.com/apache/geode/pull/518/commits/012983e58311572d155835a95d8a51a784bf1283
adds reference information about the XSD (xml) definitions for Lucene indexes.
Can all devs
Github user karensmolermiller commented on the issue:
https://github.com/apache/geode/pull/518
Thanks joeymcallister. All suggested corrections have been made.
---
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 proj
Github user davebarnes97 commented on the issue:
https://github.com/apache/geode/pull/518
+1 with Joey's changes.
---
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 wish