[GitHub] geode issue #518: GEODE-2913 Update Lucene index documentation

2017-05-24 Thread karensmolermiller
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] geode issue #518: GEODE-2913 Update Lucene index documentation

2017-05-24 Thread karensmolermiller
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] geode issue #518: GEODE-2913 Update Lucene index documentation

2017-05-24 Thread boglesby
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] geode issue #518: GEODE-2913 Update Lucene index documentation

2017-05-23 Thread karensmolermiller
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] geode issue #518: GEODE-2913 Update Lucene index documentation

2017-05-18 Thread karensmolermiller
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] geode issue #518: GEODE-2913 Update Lucene index documentation

2017-05-18 Thread davebarnes97
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