GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/398
Split the redis adapter into its own package
Under this PR, the redis adapter is moved to its own source root, and
registered as a service.
We're intending to make this a feature branch to improve the redis adapter.
@bschuchardt
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/galen-pivotal/incubator-geode
feature/GEODE-2449
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/geode/pull/398.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #398
----
commit c6dbc6d4e4ea82d65074e30c4c15085a5e3d8688
Author: Udo Kohlmeyer <ukohlme...@pivotal.io>
Date: 2017-02-10T21:33:31Z
GEODE-2449: Moved Redis out of core with minimal Extension work added
commit 81e64a9a27c74ffcff78103b66cc9ca9ec4f7cf3
Author: Udo Kohlmeyer <ukohlme...@pivotal.io>
Date: 2017-02-10T21:43:28Z
GEODE-2449: spotless
commit 318b56a5e6a8cb443929dbe3d80fa5711777c2ef
Author: Udo Kohlmeyer <ukohlme...@pivotal.io>
Date: 2017-02-10T22:45:45Z
GEODE-2449: Moved Coder.java from core to redis module.
fix up code from code review
commit 5562547e065906c3c1815875f8693ad0f4be93d0
Author: Udo Kohlmeyer <ukohlme...@pivotal.io>
Date: 2017-02-10T23:21:27Z
GEODE-2449: Do a null check on the stopRedisServer
commit f79beb1e9d5ec08d25bb02f93eb96468b3253a72
Author: Galen O'Sullivan <gosulli...@pivotal.io>
Date: 2017-02-13T18:53:05Z
GEODE-2449: changes in response to review.
* Move HyperLogLog back into geode-core.
* Bring back deprecated GeodeRedisServer for backwards compatibilty.
commit c2c5e07eb8ad7ae55a24f5d972678b78852e0a15
Author: Galen O'Sullivan <gosulli...@pivotal.io>
Date: 2017-02-13T23:53:16Z
Merge branch 'develop' into feature/GEODE-2449
commit 452ac17c90a48d13342ec25aa768aaa1e8359867
Author: Galen O'Sullivan <gosulli...@pivotal.io>
Date: 2017-02-14T00:29:13Z
Don't throw an error if Redis isn't supposed to start.
commit f990e7907cddad5522eea0b6919702489be7a49d
Author: Galen O'Sullivan <gosulli...@pivotal.io>
Date: 2017-02-14T01:25:06Z
update doc comment links
commit a1116189eb25c2209e21685386d9acfcf6fbdb9e
Author: Galen O'Sullivan <gosulli...@pivotal.io>
Date: 2017-02-14T07:48:01Z
GEODE-2449. Don't throw exception on redis port of zero
when Redis service is not found.
We would like to have the ability to tell the difference between
settings that are unset and those that are set to zero, but for the
moment we can't. So that's how it is. It may be that we'll have to not
allow starting a Redis host by setting a port number of zero in config,
or that it will just be less reliable.
commit 4ce902fce6759aed67bc1c321096326b7ce8bd60
Author: Galen OSullivan <gosulli...@pivotal.io>
Date: 2017-02-14T17:22:03Z
Fix a log message that was causing tests to fail.
commit 2fbdb0cd8958051cb807a1674bf8344c92473802
Author: Galen OSullivan <gosulli...@pivotal.io>
Date: 2017-02-14T22:30:56Z
GEODE-2449. Don't expect moved Redis classes in core.
----
---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
