[GitHub] [geode-native] pdxcodemonkey closed pull request #582: GEODE-7584: refactor pdx handling

2020-09-22 Thread GitBox
pdxcodemonkey closed pull request #582: URL: https://github.com/apache/geode-native/pull/582 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode-native] pdxcodemonkey merged pull request #654: GEODE-8514: python protocol tool

2020-09-22 Thread GitBox
pdxcodemonkey merged pull request #654: URL: https://github.com/apache/geode-native/pull/654 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode-native] pdxcodemonkey commented on pull request #582: GEODE-7584: refactor pdx handling

2020-09-22 Thread GitBox
pdxcodemonkey commented on pull request #582: URL: https://github.com/apache/geode-native/pull/582#issuecomment-696896376 Closing this PR, since it's clear I'm not going to get around to addressing the changes requested any time soon, if at all. GEODE-7584 in Jira can remain active as a

[GitHub] [geode] bschuchardt merged pull request #5531: Revert "GEODE-8463: server's log filled with SSLException: Tag mismatch! (#5482)"

2020-09-22 Thread GitBox
bschuchardt merged pull request #5531: URL: https://github.com/apache/geode/pull/5531 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [geode] jdeppe-pivotal merged pull request #5528: Redis StartUp tests use ephemeral port for Geode server

2020-09-22 Thread GitBox
jdeppe-pivotal merged pull request #5528: URL: https://github.com/apache/geode/pull/5528 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [geode] dschneider-pivotal merged pull request #5526: GEODE-8504: add info command

2020-09-22 Thread GitBox
dschneider-pivotal merged pull request #5526: URL: https://github.com/apache/geode/pull/5526 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode] sabbeyPivotal opened a new pull request #5535: GEODE-8516: Add Redis tests for multiple subscriptions for the same client

2020-09-22 Thread GitBox
sabbeyPivotal opened a new pull request #5535: URL: https://github.com/apache/geode/pull/5535 Fixed test flakiness by awaiting for successful subscribe and psubscribe from Lettuce client. This is an automated message from

[GitHub] [geode] onichols-pivotal merged pull request #5527: GEODE-8496: bump dependency versions

2020-09-22 Thread GitBox
onichols-pivotal merged pull request #5527: URL: https://github.com/apache/geode/pull/5527 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode] onichols-pivotal opened a new pull request #5537: Bump archunit from 0.12.0 to 0.14.1

2020-09-22 Thread GitBox
onichols-pivotal opened a new pull request #5537: URL: https://github.com/apache/geode/pull/5537 Adopt me! (otherwise I will be abandoned if no progress by Oct 16) This is an automated message from the Apache Git Service. To

[GitHub] [geode] sabbeyPivotal opened a new pull request #5533: GEODE-8498: Messages written to Netty channel sometimes delivered out of order

2020-09-22 Thread GitBox
sabbeyPivotal opened a new pull request #5533: URL: https://github.com/apache/geode/pull/5533 Error from .tcl tests: ``` [err]: Mix SUBSCRIBE and PSUBSCRIBE in tests/unit/pubsub.tcl Expected 'pmessage foo.* foo.bar hello' to be equal to 'message foo.bar hello' *** [err]:

[GitHub] [geode] dschneider-pivotal merged pull request #5534: Revert "GEODE-8516: Add Redis tests for multiple subscriptions for the same client

2020-09-22 Thread GitBox
dschneider-pivotal merged pull request #5534: URL: https://github.com/apache/geode/pull/5534 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode] onichols-pivotal opened a new pull request #5539: Bump assertj from 3.15.0 to 3.17.2

2020-09-22 Thread GitBox
onichols-pivotal opened a new pull request #5539: URL: https://github.com/apache/geode/pull/5539 Adopt me! (otherwise I will be abandoned if no progress by Oct 16) This is an automated message from the Apache Git Service. To

[GitHub] [geode] mhansonp commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

2020-09-22 Thread GitBox
mhansonp commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493093231 ## File path: geode-core/src/test/java/org/apache/geode/management/bean/stats/MemberLevelStatsTest.java ## @@ -339,6 +342,41 @@ public void

[GitHub] [geode] onichols-pivotal opened a new pull request #5538: Bump junit from 4.12 to 4.13

2020-09-22 Thread GitBox
onichols-pivotal opened a new pull request #5538: URL: https://github.com/apache/geode/pull/5538 Adopt me! (otherwise I will be abandoned if no progress by Oct 16) This is an automated message from the Apache Git Service. To

[GitHub] [geode] sabbeyPivotal opened a new pull request #5534: Revert "GEODE-8516: Add Redis tests for multiple subscriptions for the same client

2020-09-22 Thread GitBox
sabbeyPivotal opened a new pull request #5534: URL: https://github.com/apache/geode/pull/5534 Reverts GEODE-8516 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [geode-native] pdxcodemonkey commented on pull request #645: GEODE-8469: Enable no-missing-variable-declarations

2020-09-22 Thread GitBox
pdxcodemonkey commented on pull request #645: URL: https://github.com/apache/geode-native/pull/645#issuecomment-697028109 So the following 3 tests are now consistently failing on Windows: ``` The following tests FAILED: 14 - testRegionAccessThreadSafe (Timeout) 24 -

[GitHub] [geode] pivotal-eshu commented on a change in pull request #5530: GEODE-8517: GatewaySenderEventImpl's 2 new attributes were introduced…

2020-09-22 Thread GitBox
pivotal-eshu commented on a change in pull request #5530: URL: https://github.com/apache/geode/pull/5530#discussion_r492888962 ## File path: geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventImpl.java ## @@ -710,7 +710,7 @@ public int getDSFID() {

[GitHub] [geode] pivotal-jbarrett opened a new pull request #5532: GEODE-8513: Remove (de)serialization of local sessions.

2020-09-22 Thread GitBox
pivotal-jbarrett opened a new pull request #5532: URL: https://github.com/apache/geode/pull/5532 Removes query that resulted in casting issue in antlr. Clears all locally cached entries. Thank you for submitting a contribution to Apache Geode. In order to streamline the

[GitHub] [geode] jinmeiliao opened a new pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

2020-09-22 Thread GitBox
jinmeiliao opened a new pull request #5536: URL: https://github.com/apache/geode/pull/5536 …total GC count and GC time. This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [geode] jhutchison commented on a change in pull request #5526: GEODE-8504: add info command

2020-09-22 Thread GitBox
jhutchison commented on a change in pull request #5526: URL: https://github.com/apache/geode/pull/5526#discussion_r492821512 ## File path: geode-redis/src/main/java/org/apache/geode/redis/internal/netty/Command.java ## @@ -183,8 +184,16 @@ public boolean

[GitHub] [geode] dschneider-pivotal merged pull request #5529: GEODE-8516: Add Redis tests for multiple subscriptions for the same client

2020-09-22 Thread GitBox
dschneider-pivotal merged pull request #5529: URL: https://github.com/apache/geode/pull/5529 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode] bschuchardt opened a new pull request #5531: Revert "GEODE-8463: server's log filled with SSLException: Tag mismatch! (#5482)"

2020-09-22 Thread GitBox
bschuchardt opened a new pull request #5531: URL: https://github.com/apache/geode/pull/5531 This reverts commit 20a35ece18054e96eccda70c65a015f4af26b4c7. Changes for GEODE-8506 have fixed the problems with using TLSv1.3 in Java 8 so this commit needs to be reverted. Thank

[GitHub] [geode] onichols-pivotal opened a new pull request #5541: Bump tomcat6 from 6.0.37 to 6.0.53

2020-09-22 Thread GitBox
onichols-pivotal opened a new pull request #5541: URL: https://github.com/apache/geode/pull/5541 Adopt me! (otherwise I will be abandoned if no progress by Oct 16) This is an automated message from the Apache Git Service. To

[GitHub] [geode] onichols-pivotal opened a new pull request #5543: GEODE-8496: fix rest management test after dependency bump

2020-09-22 Thread GitBox
onichols-pivotal opened a new pull request #5543: URL: https://github.com/apache/geode/pull/5543 I've uncommented the test that started failing after 37 dependencies were bumped for GEODE-8496. It appears as if the test was relying on a quirk of an underlying library to generate the

[GitHub] [geode] mhansonp commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

2020-09-22 Thread GitBox
mhansonp commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493093231 ## File path: geode-core/src/test/java/org/apache/geode/management/bean/stats/MemberLevelStatsTest.java ## @@ -339,6 +342,41 @@ public void

[GitHub] [geode] onichols-pivotal merged pull request #5542: GEODE-8469: fix windows test failure

2020-09-22 Thread GitBox
onichols-pivotal merged pull request #5542: URL: https://github.com/apache/geode/pull/5542 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode-native] pdxcodemonkey commented on pull request #645: GEODE-8469: Enable no-missing-variable-declarations

2020-09-22 Thread GitBox
pdxcodemonkey commented on pull request #645: URL: https://github.com/apache/geode-native/pull/645#issuecomment-697028109 So the following 3 tests are now consistently failing on Windows: ``` The following tests FAILED: 14 - testRegionAccessThreadSafe (Timeout) 24 -

[GitHub] [geode] dschneider-pivotal merged pull request #5534: Revert "GEODE-8516: Add Redis tests for multiple subscriptions for the same client

2020-09-22 Thread GitBox
dschneider-pivotal merged pull request #5534: URL: https://github.com/apache/geode/pull/5534 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode] onichols-pivotal merged pull request #5527: GEODE-8496: bump dependency versions

2020-09-22 Thread GitBox
onichols-pivotal merged pull request #5527: URL: https://github.com/apache/geode/pull/5527 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode] onichols-pivotal opened a new pull request #5540: Bump mockito from 3.3.3 to 3.5.11

2020-09-22 Thread GitBox
onichols-pivotal opened a new pull request #5540: URL: https://github.com/apache/geode/pull/5540 Adopt me! (otherwise I will be abandoned if no progress by Oct 16) This is an automated message from the Apache Git Service. To

[GitHub] [geode] onichols-pivotal opened a new pull request #5542: GEODE-8469: fix windows test failure

2020-09-22 Thread GitBox
onichols-pivotal opened a new pull request #5542: URL: https://github.com/apache/geode/pull/5542 newer dependency version uses platform-specific line separators, test did not This is an automated message from the Apache Git

[GitHub] [geode] jinmeiliao commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

2020-09-22 Thread GitBox
jinmeiliao commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493153715 ## File path: geode-core/src/test/java/org/apache/geode/management/bean/stats/MemberLevelStatsTest.java ## @@ -339,6 +342,41 @@ public void

[GitHub] [geode-native] pdxcodemonkey commented on pull request #582: GEODE-7584: refactor pdx handling

2020-09-22 Thread GitBox
pdxcodemonkey commented on pull request #582: URL: https://github.com/apache/geode-native/pull/582#issuecomment-696896376 Closing this PR, since it's clear I'm not going to get around to addressing the changes requested any time soon, if at all. GEODE-7584 in Jira can remain active as a

[GitHub] [geode] jdeppe-pivotal merged pull request #5528: Redis StartUp tests use ephemeral port for Geode server

2020-09-22 Thread GitBox
jdeppe-pivotal merged pull request #5528: URL: https://github.com/apache/geode/pull/5528 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [geode] dschneider-pivotal merged pull request #5526: GEODE-8504: add info command

2020-09-22 Thread GitBox
dschneider-pivotal merged pull request #5526: URL: https://github.com/apache/geode/pull/5526 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode] bschuchardt merged pull request #5531: Revert "GEODE-8463: server's log filled with SSLException: Tag mismatch! (#5482)"

2020-09-22 Thread GitBox
bschuchardt merged pull request #5531: URL: https://github.com/apache/geode/pull/5531 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [geode] pivotal-eshu commented on a change in pull request #5530: GEODE-8517: GatewaySenderEventImpl's 2 new attributes were introduced…

2020-09-22 Thread GitBox
pivotal-eshu commented on a change in pull request #5530: URL: https://github.com/apache/geode/pull/5530#discussion_r492888962 ## File path: geode-core/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventImpl.java ## @@ -710,7 +710,7 @@ public int getDSFID() {

[GitHub] [geode] jdeppe-pivotal commented on a change in pull request #5526: GEODE-8504: add info command

2020-09-22 Thread GitBox
jdeppe-pivotal commented on a change in pull request #5526: URL: https://github.com/apache/geode/pull/5526#discussion_r492343203 ## File path: geode-redis/src/main/java/org/apache/geode/redis/internal/netty/Command.java ## @@ -183,8 +184,16 @@ public boolean

[GitHub] [geode-native] pdxcodemonkey merged pull request #654: GEODE-8514: python protocol tool

2020-09-22 Thread GitBox
pdxcodemonkey merged pull request #654: URL: https://github.com/apache/geode-native/pull/654 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode] dschneider-pivotal merged pull request #5529: GEODE-8516: Add Redis tests for multiple subscriptions for the same client

2020-09-22 Thread GitBox
dschneider-pivotal merged pull request #5529: URL: https://github.com/apache/geode/pull/5529 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode-native] pdxcodemonkey closed pull request #582: GEODE-7584: refactor pdx handling

2020-09-22 Thread GitBox
pdxcodemonkey closed pull request #582: URL: https://github.com/apache/geode-native/pull/582 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [geode] jhutchison commented on a change in pull request #5526: GEODE-8504: add info command

2020-09-22 Thread GitBox
jhutchison commented on a change in pull request #5526: URL: https://github.com/apache/geode/pull/5526#discussion_r492821512 ## File path: geode-redis/src/main/java/org/apache/geode/redis/internal/netty/Command.java ## @@ -183,8 +184,16 @@ public boolean

[GitHub] [geode] jinmeiliao commented on a change in pull request #5536: GEODE-8520: GCStatsMonitor should sum up all the GC stats to get the …

2020-09-22 Thread GitBox
jinmeiliao commented on a change in pull request #5536: URL: https://github.com/apache/geode/pull/5536#discussion_r493153715 ## File path: geode-core/src/test/java/org/apache/geode/management/bean/stats/MemberLevelStatsTest.java ## @@ -339,6 +342,41 @@ public void

[GitHub] [geode] onichols-pivotal merged pull request #5542: GEODE-8469: fix windows test failure

2020-09-22 Thread GitBox
onichols-pivotal merged pull request #5542: URL: https://github.com/apache/geode/pull/5542 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above