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
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/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
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 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 your project
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.
You can
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 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, you
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 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
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/402#discussion_r102357674
--- 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
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 can reply
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 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 have
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 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/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 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 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/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
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_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_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_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 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 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_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 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 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 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 more
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, it's
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 https
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 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 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 appear
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/426
@bschuchardt Check it out again; I force-pushed with just the annotation on
GeodeRedisServer.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/426
@bschuchardt This is just the experimental annotation, not changes.
---
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 the issue:
https://github.com/apache/geode/pull/426
@bschuchardt want to merge 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 your project does not have this feature
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/426
GEODE-2660 Add @Experimental to the Redis adapter.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/galen-pivotal/incubator-geode
feature
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/441
I'm not a committer, so someone else will have to push this for me.
@kohlmu-pivotal ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/404
@ggreen If you can clean up the history from where my PR was merged this
will be much easier to review. In general, even though we've added the
`@Experimental` tag, I would like the PR
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/437
@bschuchardt this is https://reviews.apache.org/r/58155/
---
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
GitHub user galen-pivotal opened a pull request:
https://github.com/apache/geode/pull/441
Fix a repetition in the README.
and update numbers.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/galen-pivotal/geode fix-README
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/450#discussion_r112004128
--- Diff:
geode-core/src/jmh/java/org/apache/geode/internal/cache/tier/sockets/command/ClientCachePutBench.java
---
@@ -0,0 +1,233
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/450#discussion_r112005541
--- Diff:
geode-core/src/jmh/java/org/apache/geode/internal/cache/tier/sockets/command/ClientCachePutBench.java
---
@@ -0,0 +1,233
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/450
Do you know why I get so many of these messages when I run the benchmark?
```
Encountered duplicate path
"javassist/bytecode/annotation/ArrayMemberValue.class" during copy
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/450#discussion_r112005377
--- Diff:
geode-core/src/jmh/java/org/apache/geode/internal/cache/tier/sockets/command/ClientCachePutBench.java
---
@@ -0,0 +1,233
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/450
This will set the same keys to the same values each iteration, because. I
ran a benchmark last night with a benchmark that used only 99 values, but I
didn't see a significant difference
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105059140
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/set/SPopExecutor.java
---
@@ -36,24 +37,26 @@ public void executeCommand
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105061296
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/set/SetOpExecutor.java
---
@@ -47,23 +44,23 @@ public void executeCommand
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105008854
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/ExecutionHandlerContext.java
---
@@ -135,8 +135,14 @@ private void writeToChannel
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105060019
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/set/SRandMemberExecutor.java
---
@@ -42,9 +42,8 @@ public void
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105046939
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/set/SMembersExecutor.java
---
@@ -39,16 +39,18 @@ public void
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105048647
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/set/SIsMemberExecutor.java
---
@@ -40,23 +42,35 @@ public void
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105008897
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/ExecutionHandlerContext.java
---
@@ -235,6 +242,8 @@ private void
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105060205
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/set/SRandMemberExecutor.java
---
@@ -42,9 +42,8 @@ public void
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105010443
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java ---
@@ -187,8 +215,18 @@ public boolean removeKey
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105053042
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/set/SMoveExecutor.java
---
@@ -45,30 +48,44 @@ public void executeCommand
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105009858
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java ---
@@ -95,18 +98,38 @@ public RegionProvider(Region
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r104552749
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/hash/HashExecutor.java
---
@@ -75,14 +96,76
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105010837
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java ---
@@ -289,7 +331,9 @@ public void
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/412#discussion_r104258182
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/internal/tcpserver/TcpServer.java
---
@@ -360,6 +360,13 @@ private void processRequest
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r105510564
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/hash/HSetExecutor.java
---
@@ -55,9 +69,11 @@ public void executeCommand
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r104041093
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/hash/HDelExecutor.java
---
@@ -40,25 +38,25 @@ public void executeCommand
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r104031973
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java ---
@@ -442,9 +502,10 @@ protected void checkDataType
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r104041266
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java ---
@@ -442,9 +502,10 @@ protected void checkDataType
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r104059948
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/executor/hash/HashInterpreter.java
---
@@ -0,0 +1,126 @@
+/*
+ * Licensed
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/437
@kohlmu-pivotal
---
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
Github user galen-pivotal commented on the issue:
https://github.com/apache/geode/pull/663
@pivotal-amurmann I'd like to write a unit test. @hiteshk25 and I took a
stab at it this afternoon but after an hour or two, got lost in mocks and the
tangle of
DistributedSystem
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 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
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/661#discussion_r130494791
--- 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/661#discussion_r130779627
--- Diff:
geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/PutRequestOperationHandler.java
---
@@ -50,10 +50,7
Github user galen-pivotal commented on a diff in the pull request:
https://github.com/apache/geode/pull/661#discussion_r130495450
--- 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/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 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 good way
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
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_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 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 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_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_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 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 look
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 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/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
1 - 100 of 170 matches
Mail list logo