[GitHub] geode-examples pull request #10: Feature/geode 3202

2017-07-13 Thread dihardman
GitHub user dihardman opened a pull request: https://github.com/apache/geode-examples/pull/10 Feature/geode 3202 Here is a new simple example that uses Lucene indexes and searching. Please review and comment. You can merge this pull request into a Git repository by running

[GitHub] geode issue #555: GEODE-2947: Document revised gfsh destroy error message

2017-06-02 Thread dihardman
Github user dihardman commented on the issue: https://github.com/apache/geode/pull/555 +1 Looks good. Thanks! On Fri, Jun 2, 2017 at 9:26 AM, Karen Miller wrote: > @dihardman <https://github.com/dihardman> @boglesby > <https://github.com/bogle

[GitHub] geode issue #554: GEODE-3014 Document server/region/Lucene index start-up se...

2017-05-31 Thread dihardman
Github user dihardman commented on the issue: https://github.com/apache/geode/pull/554 +1 On Wed, May 31, 2017 at 4:24 PM, Karen Miller wrote: > @joeymcallister <https://github.com/joeymcallister> @dihardman > <https://github.com/dihardman>

[GitHub] geode issue #544: GEODE-2994 Take backups when region ops are quiescent

2017-05-31 Thread dihardman
Github user dihardman commented on the issue: https://github.com/apache/geode/pull/544 +1, looks good. On Wed, May 31, 2017 at 2:46 PM, Barrett Oglesby wrote: > *@boglesby* approved this pull request. > > — > You are receiving this beca

[GitHub] geode issue #553: GEODE-3011 Correct a Lucene gfsh query example

2017-05-31 Thread dihardman
Github user dihardman commented on the issue: https://github.com/apache/geode/pull/553 +1 looks good! On Wed, May 31, 2017 at 1:33 PM, Karen Miller wrote: > @dihardman <https://github.com/dihardman> @boglesby > <https://github.com/boglesby>

[GitHub] geode pull request #544: GEODE-2994 Take backups when region ops are quiesce...

2017-05-31 Thread dihardman
Github user dihardman commented on a diff in the pull request: https://github.com/apache/geode/pull/544#discussion_r119445495 --- Diff: geode-docs/tools_modules/lucene_integration.html.md.erb --- @@ -294,8 +294,11 @@ will return `object_2`. will return `object_3

[GitHub] geode pull request #544: GEODE-2994 Take backups when region ops are quiesce...

2017-05-31 Thread dihardman
Github user dihardman commented on a diff in the pull request: https://github.com/apache/geode/pull/544#discussion_r119445110 --- Diff: geode-docs/managing/disk_storage/backup_restore_disk_store.html.md.erb --- @@ -38,6 +38,15 @@ Do not try to create backup files from a running

[GitHub] geode pull request #544: GEODE-2994 Take backups when region ops are quiesce...

2017-05-31 Thread dihardman
Github user dihardman commented on a diff in the pull request: https://github.com/apache/geode/pull/544#discussion_r119439870 --- Diff: geode-docs/managing/disk_storage/backup_restore_disk_store.html.md.erb --- @@ -38,6 +38,15 @@ Do not try to create backup files from a running

[GitHub] geode pull request #544: GEODE-2994 Take backups when region ops are quiesce...

2017-05-31 Thread dihardman
Github user dihardman commented on a diff in the pull request: https://github.com/apache/geode/pull/544#discussion_r119438727 --- Diff: geode-docs/managing/disk_storage/backup_restore_disk_store.html.md.erb --- @@ -38,6 +38,15 @@ Do not try to create backup files from a running

[GitHub] geode issue #550: GEODE-3002 Clarify __REGION_VALUE_FIELD in Lucene docs

2017-05-30 Thread dihardman
Github user dihardman commented on the issue: https://github.com/apache/geode/pull/550 +1, thanks the new description is more accurate. On Tue, May 30, 2017 at 2:12 PM, Dave Barnes wrote: > *@davebarnes97* approved this pull request. > >

[GitHub] geode issue #545: GEODE-2952 document quoting of exact match Lucene queries

2017-05-26 Thread dihardman
Github user dihardman commented on the issue: https://github.com/apache/geode/pull/545 +1 Looks good! On Fri, May 26, 2017 at 3:15 PM, Karen Miller wrote: > @DivineEnder <https://github.com/divineender> @upthewaterspout > <https://github.com/

[GitHub] geode issue #542: GEODE-2951 Remove --pageSize from docs of gfsh search luce...

2017-05-26 Thread dihardman
Github user dihardman commented on the issue: https://github.com/apache/geode/pull/542 +1 - Looks fine to me. On Fri, May 26, 2017 at 12:30 PM, Dave Barnes wrote: > *@davebarnes97* approved this pull request. > > LGTM +1 > > â€

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

2017-05-18 Thread dihardman
Github user dihardman commented on a diff in the pull request: https://github.com/apache/geode/pull/518#discussion_r117387903 --- Diff: geode-docs/tools_modules/lucene_integration.html.md.erb --- @@ -219,7 +219,7 @@ at TestClient.main(TestClient.java:59) ``` - If the