Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/661#discussion_r130494367
--- Diff:
geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/utilities/ProtobufPrimitiveTypes.java
---
@@ -0,0 +1,47
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/676#discussion_r130781170
--- Diff:
geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/ProtocolErrorCode.java
---
@@ -0,0 +1,39 @@
+/*
+ * Licensed to the
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/676#discussion_r130781436
--- Diff:
geode-protobuf/src/test/java/org/apache/geode/protocol/protobuf/operations/RemoveRequestOperationHandlerJUnitTest.java
---
@@ -95,8 +97,8
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/676
It looks like this is failing in Travis due to `:rat`, but you've got a
license in your only new file. I'm a bit mystified. Closing the pull request
and reopening it can be a g
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/673#discussion_r130781962
--- Diff:
geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/GetAvailableServersOperationHandler.java
---
@@ -0,0 +1,110
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/673#discussion_r130782011
--- Diff:
geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/GetAvailableServersOperationHandler.java
---
@@ -0,0 +1,110
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/676#discussion_r130987753
--- Diff:
geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/ProtocolErrorCode.java
---
@@ -0,0 +1,39 @@
+/*
+ * Licensed to the
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/676
Looks like there are some conflicts that need to be resolved.
---
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
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/683
GEODE-3314 - additional refactoring for developer QoL.
* Write characterization tests for DLockService.
* Remove debugging code.
* Remove dead code.
* Remove comments.
* Extract
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/683
@kohlmu-pivotal @hiteshk25 @bschuchardt @pivotal-amurmann @WireBaron
---
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
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/683#discussion_r131290034
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/internal/locks/DLockToken.java
---
@@ -87,7 +87,8 @@
private Thread thread
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/683#discussion_r131290142
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/internal/locks/DLockRequestProcessor.java
---
@@ -196,6 +196,13 @@ long
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/683#discussion_r131290383
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/internal/locks/DLockToken.java
---
@@ -302,14 +302,14 @@ boolean checkForExpiration
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/683#discussion_r131290606
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/internal/locks/DLockToken.java
---
@@ -302,14 +302,14 @@ boolean checkForExpiration
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/682#discussion_r132626571
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/admin/SSLConfig.java ---
@@ -33,11 +34,11 @@
private String ciphers
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/698
What you've done looks good. However, it looks like there are ten or so
files in `geode-protobuf` that you didn't mark as experimental. Is there a
reason for this?
Have a l
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/683#discussion_r132864196
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/internal/locks/DLockService.java
---
@@ -74,17 +73,6 @@
public static final
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/702#discussion_r132869478
--- Diff: geode-book/master_middleman/bookbinder_helpers.rb ---
@@ -1,298 +0,0 @@
-# Licensed to the Apache Software Foundation (ASF) under one
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/707#discussion_r132877993
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/ConfigurationProperties.java
---
@@ -1378,6 +1379,18 @@
*/
String NAME
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/707#discussion_r132878110
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/internal/DistributionConfig.java
---
@@ -2433,6 +2433,25 @@
String
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/707#discussion_r132878378
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnectionFactory.java
---
@@ -63,6 +65,31 @@ private static
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/707#discussion_r132878512
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnectionFactory.java
---
@@ -72,9 +99,15 @@ public static
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/716#discussion_r134021523
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/internal/tcpserver/TcpServer.java
---
@@ -334,107 +351,109 @@ protected void run
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/716#discussion_r134021675
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/internal/tcpserver/TcpServer.java
---
@@ -334,107 +351,109 @@ protected void run
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/726
GEODE-3474: protobuf auth with ExampleSecurityManager
It looks like we were using the wrong constants for the username and
password strings; ExampleSecurityManager uses some defined constants
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/716#discussion_r134023037
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnectionFactory.java
---
@@ -49,20 +48,13 @@ private
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/716#discussion_r134024644
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/TcpServerFactory.java
---
@@ -0,0 +1,39 @@
+package
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/716#discussion_r134026218
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/TcpServerFactory.java
---
@@ -0,0 +1,39 @@
+package
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/716#discussion_r134358612
--- Diff:
geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/GetAvailableServersOperationHandler.java
---
@@ -50,51 +37,23
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/716#discussion_r134022599
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientProtoclMessageHandlerLoader.java
---
@@ -0,0 +1,64
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/716#discussion_r134270934
--- Diff:
geode-core/src/test/java/org/apache/geode/test/dunit/standalone/DUnitLauncher.java
---
@@ -297,6 +297,8 @@ public Object call() throws
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/719#discussion_r134633950
--- Diff:
geode-core/src/main/java/org/apache/geode/security/StreamAuthorizer.java ---
@@ -0,0 +1,19 @@
+/*
+ * Licensed to the Apache Software
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/719#discussion_r134635734
--- Diff:
geode-protobuf/src/test/java/org/apache/geode/protocol/AuthorizationIntegrationTest.java
---
@@ -0,0 +1,206 @@
+/*
+ * Licensed to
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/719#discussion_r134634705
--- Diff:
geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/registry/OperationContextRegistry.java
---
@@ -47,41 +48,57 @@ private void
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/730
I haven't reviewed this, but :+1: deleting commented code is a GOOD THING.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/739
GEODE-3385: Change GetAllRequest to return list of errors.
Also:
* Rename `KeyedErrorResponse` to `KeyedError`.
* move `ErrorResponse` to `clientProtocol.proto`.
* Check for null
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/649
closing for #739 .
---
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
Github user galen-pivotal closed the pull request at:
https://github.com/apache/geode/pull/649
---
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
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/737
Looks like JSONCodecJUnitTest has multiple failures.
---
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
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/737#discussion_r135157183
--- Diff:
geode-protobuf/src/test/java/org/apache/geode/serialization/codec/JSONCodecJUnitTest.java
---
@@ -0,0 +1,226 @@
+/*
+ * Licensed to
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/719#discussion_r135158491
--- Diff:
geode-core/src/main/java/org/apache/geode/security/StreamAuthenticator.java ---
@@ -46,6 +44,14 @@ void receiveMessage(InputStream
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/719
@metatype We need the `StreamAuthenticator` to receive and send
(Protobuf-encoded) messages containing the credentials that get passed to the
`SecurityManager`. I would think that ideally it&#
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/739#discussion_r135351811
--- Diff:
geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/GetAllRequestOperationHandler.java
---
@@ -50,26 +53,52
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/739
Rebased on develop.
---
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
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/730
@PurelyApplied I think you can also close and reopen a PR to restart Travis.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/739
rebased on develop again, fixed a test to use the new
MessageExecutionContext.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/746
@bschuchardt try closing and reopening; I think that will make Travis
restart.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/305
[GEODE-1923] Fix a test race condition.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/galen-pivotal/incubator-geode
feature/GEODE-1923
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/305#discussion_r91569800
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/cache/FixedPRSinglehopDUnitTest.java
---
@@ -293,18 +295,24 @@ public void
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/306
[GEODE-2191] Test randomized string serialization/deserialization.
I added this test partly as an exercise in getting used to the Geode
codebase, and partly because I'd like to see
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/306
@kirklund I disagree. We expect string serialization to work reliably on
*all* inputs, and since we can't test all of them, we may as well test a bunch
picked at random. If we fail,
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/313
[GEODE-1407] Change a FlakyTest to distributedTest.
* remove `FlakyTest` from `ReconnectDUnitTest.testReconnectALocator`.
* use Awaitility instead of Wait.
This test doesn't
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/305#discussion_r94657673
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/cache/FixedPRSinglehopDUnitTest.java
---
@@ -293,18 +294,23 @@ public void
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/305#discussion_r94660604
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/cache/FixedPRSinglehopDUnitTest.java
---
@@ -293,18 +294,23 @@ public void
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/305
@kohlmu-pivotal I just think it's a bit cleaner syntactically; there's no
other real benefit.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/313#discussion_r94888520
--- Diff:
geode-core/src/test/java/org/apache/geode/cache30/ReconnectDUnitTest.java ---
@@ -534,41 +532,35 @@ public Object call() throws CacheException
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/305
@kohlmu-pivotal Could you merge this for me please?
---
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
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/337
[GEODE-2295] Add constructors for LocatorCancelException
and update one place where it's used.
You can merge this pull request into a Git repository by running:
$ git pull
Github user galen-pivotal closed the pull request at:
https://github.com/apache/geode/pull/337
---
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
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/337
Moving to review board.
---
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
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/349
[GEODE-2324] fixes to AcceptorImpl.close()
If the thread is interrupted during closing, just continue to shut down
what we can.
* Catch InterruptedException so cleanup continues
Github user galen-pivotal closed the pull request at:
https://github.com/apache/geode/pull/313
---
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
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/313
Merged after rebase.
---
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
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/359
[GEODE-2295] Add constructors for `LocatorCancelException`.
[GEODE-2295] Add constructors for `LocatorCancelException`.
You can merge this pull request into a Git repository by running
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/360
[GEODE-1793] Fix a flaky test, clean up SSL tests.
* Use less code.
* Get random port numbers a bit more robustly.
* Check to make sure that locator2 has died at the end.
You can merge
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/368
[GEODE-2381] make complex enums not look terrible.
* Change the style XML file to insert a line break after each constant
of an enum.
There may be a better way to do this and
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/368
@jaredjstewart you know more about this than I do.
Do you know why we decided to go with the Eclipse formatter rather than
[Google's java formatter](https://github.com/google/google
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/368
I don't really understand what the `48` means, but it's used elsewhere and
looks something like "put everything on its own line".
---
If your project is set up for it, y
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/382
GEODE-2414: mark a failing test as flaky.
This is just a temporary fix for CI until we can diagnose the issue.
@kohlmu-pivotal @hiteshk25
You can merge this pull request into a Git
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/383
GEODE-2206: Add junit-quickcheck to geode-core.
* Rewrite a data serialization test to use junit-quickcheck.
You can merge this pull request into a Git repository by running:
$ git pull
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/383
@scmbuildguy : done.
---
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
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/383#discussion_r99265598
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/InternalDataSerializerQuickcheckStringTest.java
---
@@ -0,0 +1,63
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/383#discussion_r99371844
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/InternalDataSerializerQuickcheckStringTest.java
---
@@ -0,0 +1,63
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/392
[GEODE-2414] Ignore a consistently failing test.
This test only seems to be failing on Linux. I haven't figured out the
cause yet. Ignore it until I have time to fix it properly.
Yo
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/392
@kirklund I'm not a committer; you're welcome to commit this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/368
@kirklund no problem, the change is what counts. ð
---
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
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/349
merged in cd2291102744f4bb1b8bb5170e7388b31295ef6f
---
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
Github user galen-pivotal closed the pull request at:
https://github.com/apache/geode/pull/349
---
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
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/359
Merged in a61c55b9b0d4fffc70a1502f06cbca59098c9418
---
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
Github user galen-pivotal closed the pull request at:
https://github.com/apache/geode/pull/359
---
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
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 fe
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/398#discussion_r101354013
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/GeodeRedisService.java ---
@@ -0,0 +1,33 @@
+/*
+ * Licensed to the Apache Software
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/398#discussion_r101353985
--- Diff:
geode-redis/src/test/java/org/apache/geode/redis/AuthJUnitTest.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed to the Apache Software
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/398#discussion_r101354076
--- Diff:
geode-redis/src/test/java/org/apache/geode/redis/AuthJUnitTest.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed to the Apache Software
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/398
@metatype The import ordering is the rather unfortunate result of
IntelliJ's import cleanup.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/398#discussion_r101374277
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java
---
@@ -1330,21 +1360,12 @@ private void startMemcachedServer
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/399
Minor non-functional changes in response to PR comments.
@bschuchardt
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/galen-pivotal
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/398#discussion_r101388584
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java
---
@@ -1330,21 +1360,12 @@ private void startMemcachedServer
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/398#discussion_r101388974
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java
---
@@ -1330,21 +1360,12 @@ private void startMemcachedServer
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/400
Add geode-redis to assembly.
This makes starting a server with Redis actually work, at least on my
machine.
You can merge this pull request into a Git repository by running:
$ git pull
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/398#discussion_r101401136
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java
---
@@ -1330,21 +1360,12 @@ private void startMemcachedServer
Github user galen-pivotal closed the pull request at:
https://github.com/apache/geode/pull/398
---
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
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/398
We're now running this as a feature branch for the current round of Redis
work, `feature/GEODE-2444`. I'm going to close this PR for now.
---
If your project is set up for it, you ca
Github user galen-pivotal closed the pull request at:
https://github.com/apache/geode/pull/400
---
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
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/400
It doesn't look like it, but this has been merged.
---
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
GitHub user galen-pivotal reopened a pull request:
https://github.com/apache/geode/pull/400
Add geode-redis to assembly.
This makes starting a server with Redis actually work, at least on my
machine.
You can merge this pull request into a Git repository by running:
$ git pull
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/402#discussion_r101873819
--- Diff:
geode-core/src/test/java/org/apache/geode/distributed/internal/DistributionManagerDUnitTest.java
---
@@ -159,77 +159,74 @@ public void
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/402#discussion_r102345903
--- Diff:
geode-core/src/test/java/org/apache/geode/distributed/internal/DistributionManagerDUnitTest.java
---
@@ -159,77 +159,74 @@ public void
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/402#discussion_r102345774
--- Diff:
geode-core/src/test/java/org/apache/geode/distributed/internal/DistributionManagerDUnitTest.java
---
@@ -159,77 +159,74 @@ public void
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/402#discussion_r102345794
--- Diff:
geode-core/src/test/java/org/apache/geode/distributed/internal/DistributionManagerDUnitTest.java
---
@@ -159,77 +159,74 @@ public void
101 - 200 of 201 matches
Mail list logo
