Github user hiteshk25 commented on a diff in the pull request:
https://github.com/apache/geode/pull/739#discussion_r135883931
--- Diff:
geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/GetAllRequestOperationHandler.java
---
@@ -50,26 +53,52
Github user hiteshk25 commented on a diff in the pull request:
https://github.com/apache/geode/pull/746#discussion_r135871358
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnectionFactory.java
---
@@ -57,15 +57,14 @@ private synchronized
Github user hiteshk25 commented on a diff in the pull request:
https://github.com/apache/geode/pull/746#discussion_r135871243
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnectionFactory.java
---
@@ -57,15 +57,14 @@ private synchronized
Github user hiteshk25 commented on a diff in the pull request:
https://github.com/apache/geode/pull/746#discussion_r135870888
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ServerConnectionFactory.java
---
@@ -41,8 +41,8 @@ private synchronized
Github user hiteshk25 commented on a diff in the pull request:
https://github.com/apache/geode/pull/702#discussion_r133506959
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/net/SocketCloser.java ---
@@ -96,46 +99,55 @@ public int getMaxThreads() {
return
Github user hiteshk25 commented on a diff in the pull request:
https://github.com/apache/geode/pull/702#discussion_r133251379
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/net/SocketCloser.java ---
@@ -96,46 +99,55 @@ public int getMaxThreads() {
return
Github user hiteshk25 commented on a diff in the pull request:
https://github.com/apache/geode/pull/702#discussion_r133249141
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/tcp/ConnectionTable.java ---
@@ -929,10 +929,6 @@ protected void removeEndpoint
Github user hiteshk25 commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r103499836
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java ---
@@ -259,6 +307,16 @@ public void
Github user hiteshk25 commented on a diff in the pull request:
https://github.com/apache/geode/pull/404#discussion_r103497182
--- Diff:
geode-core/src/main/java/org/apache/geode/redis/internal/RegionProvider.java ---
@@ -187,8 +217,26 @@ public boolean removeKey(ByteArrayWrapper
