[GitHub] geode issue #404: Geode 2469

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 on

[GitHub] geode issue #404: Geode 2469

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 on

[GitHub] geode pull request #404: Geode 2469

Github user ggreen closed the pull request at: https://github.com/apache/geode/pull/404 --- 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

[GitHub] geode pull request #404: Geode 2469

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) These

[GitHub] geode issue #404: Geode 2469

Github user ggreen commented on the issue: https://github.com/apache/geode/pull/404 @galen-pivotal and @metatype, please take a look at my latest change. It uses a java.util.concurrent.locks.Lock instead of a distribute lock for the hash and set synchronization. --- If your project

[GitHub] geode pull request #404: Geode 2469

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 to the

[GitHub] geode pull request #404: Geode 2469

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 to the

[GitHub] geode pull request #404: Geode 2469

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 issue #404: Geode 2469

Github user ggreen commented on the issue: https://github.com/apache/geode/pull/404 **RE: "huser:1000** The intent is to detect the use of objects in the Redis HASH commands. The character ":" will appear in the key if an object is used (note: this is u

[GitHub] geode pull request #404: Geode 2469

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

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

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

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...

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 wrote: > >

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

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 wrote: > > @metatype commented on this pull request. > > In .gitignore: >

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

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