[GitHub] geode issue #404: Geode 2469

2017-04-10 Thread ggreen
Github user ggreen commented on the issue: https://github.com/apache/geode/pull/404 @galen-pivotal and @metatype, are there any suggestions on how to move this pull request forward? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] geode issue #404: Geode 2469

2017-04-10 Thread ggreen
Github user ggreen commented on the issue: https://github.com/apache/geode/pull/404 @galen-pivotal and @metatype, are there any suggestions on how to move this pull request forward? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] geode pull request #404: Geode 2469

2017-03-21 Thread ggreen
GitHub user ggreen reopened a pull request: https://github.com/apache/geode/pull/404 Geode 2469 The updated Geode Redis Adapter now works with a sample Spring Data Redis Example [GEODE-2469.pdf](https://github.com/apache/geode/files/785580/GEODE-2469.pdf

[GitHub] geode pull request #404: Geode 2469

2017-03-08 Thread ggreen
Github user ggreen commented on a diff in the pull request: https://github.com/apache/geode/pull/404#discussion_r104974260 --- Diff: geode-core/src/main/java/org/apache/geode/redis/internal/executor/hash/HashInterpreter.java --- @@ -0,0 +1,126 @@ +/* + * Licensed

[GitHub] geode pull request #404: Geode 2469

2017-03-06 Thread ggreen
Github user ggreen commented on a diff in the pull request: https://github.com/apache/geode/pull/404#discussion_r104402728 --- Diff: geode-core/src/main/java/org/apache/geode/redis/internal/executor/hash/HashInterpreter.java --- @@ -0,0 +1,126 @@ +/* + * Licensed

[GitHub] geode pull request #404: Geode 2469

2017-03-02 Thread ggreen
Github user ggreen commented on a diff in the pull request: https://github.com/apache/geode/pull/404#discussion_r104033312 --- Diff: geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java --- @@ -259,6 +307,16 @@ public void

[GitHub] geode pull request #404: Geode 2469

2017-03-02 Thread ggreen
Github user ggreen commented on a diff in the pull request: https://github.com/apache/geode/pull/404#discussion_r103993535 --- Diff: geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java --- @@ -187,8 +217,26 @@ public boolean removeKey(ByteArrayWrapper key

[GitHub] geode pull request #404: Geode 2469

2017-03-02 Thread ggreen
Github user ggreen commented on a diff in the pull request: https://github.com/apache/geode/pull/404#discussion_r103928210 --- Diff: geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java --- @@ -259,6 +307,16 @@ public void

[GitHub] geode issue #404: Geode 2469

2017-02-22 Thread ggreen
Github user ggreen commented on the issue: https://github.com/apache/geode/pull/404 Hello, Sorry about that. It is there now geode/SetExecutor.java at GEODE-2469 · ggreen/geode · GitHub

[GitHub] geode pull request #404: Geode 2469

2017-02-18 Thread ggreen
GitHub user ggreen opened a pull request: https://github.com/apache/geode/pull/404 Geode 2469 The updated Geode Redis Adapter now works with a sample Spring Data Redis Example [GEODE-2469.pdf](https://github.com/apache/geode/files/785580/GEODE-2469.pdf) These changes

[GitHub] geode issue #353: GEODE-2269 update to allow region entries non null empty k...

2017-01-24 Thread ggreen
Github user ggreen commented on the issue: https://github.com/apache/geode/pull/353 Hello I did, but feel free ignore the .gitignore if it causes any issues Sent from mobile phone > On Jan 24, 2017, at 9:19 PM, Anthony Baker <notificati...@github.com&g

[GitHub] geode issue #353: GEODE-2269 update to allow region entries non null empty k...

2017-01-24 Thread ggreen
Github user ggreen commented on the issue: https://github.com/apache/geode/pull/353 Sent from mobile phone > On Jan 24, 2017, at 9:19 PM, Anthony Baker <notificati...@github.com> wrote: > > @metatype commented on th

[GitHub] geode pull request #353: GEODE-2269 update to allow region entries non null ...

2017-01-24 Thread ggreen
GitHub user ggreen opened a pull request: https://github.com/apache/geode/pull/353 GEODE-2269 update to allow region entries non null empty key such as … update to allow region entries non null empty key such as zero length strings to be removed You can merge this pull request