alb3rtobr commented on pull request #654:
URL: https://github.com/apache/geode-native/pull/654#issuecomment-697558085
@pdxcodemonkey thanks for contributing this! It's also a good excuse to
refresh my Python knowledge
This
pdxcodemonkey commented on pull request #654:
URL: https://github.com/apache/geode-native/pull/654#issuecomment-697629814
> @pdxcodemonkey thanks for contributing this! It's also a good excuse to
refresh my Python knowledge
thanks, @alb3rtobr, hope you find it helpful. Just FYI, I
jinmeiliao commented on a change in pull request #5516:
URL: https://github.com/apache/geode/pull/5516#discussion_r493631271
##
File path:
geode-core/src/distributedTest/java/org/apache/geode/internal/cache/PartitionedRegionClearWithAlterRegionDUnitTest.java
##
@@ -0,0 +1,424
sabbeyPivotal opened a new pull request #5544:
URL: https://github.com/apache/geode/pull/5544
From PING documentation (https://redis.io/commands/ping):
If the client is subscribed to a channel or a pattern, it will instead
return a multi-bulk with a "pong" in the first position and
demery-pivotal commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493761722
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
nabarunnag opened a new pull request #5546:
URL: https://github.com/apache/geode/pull/5546
* geode-tcp and geode-membership added to the list
* removed geode-json as it does not exist anymore.
This is an
jhuynh1 commented on pull request #5546:
URL: https://github.com/apache/geode/pull/5546#issuecomment-697809643
This might have to be backported to previous releases based on when these
jars were introduced...
This is an
jchen21 commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493856692
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
kirklund commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493739655
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
dschneider-pivotal merged pull request #5535:
URL: https://github.com/apache/geode/pull/5535
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
kirklund commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493738369
##
File path:
geode-core/src/test/java/org/apache/geode/management/bean/stats/MemberLevelStatsTest.java
##
@@ -339,6 +342,41 @@ public void
upthewaterspout opened a new pull request #5545:
URL: https://github.com/apache/geode/pull/5545
This log message happens during the course of normal startup of multiple
locators. We should not be logging a full stack trace during normal startup.
Thank you for submitting a
pdxcodemonkey commented on pull request #645:
URL: https://github.com/apache/geode-native/pull/645#issuecomment-697875154
I think we can diagnose the test problem with a couple of (very tedious)
hours of debugging, which is a lot better than rewriting the test. I think
we're much closer
demery-pivotal commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493853151
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
jchen21 commented on a change in pull request #5516:
URL: https://github.com/apache/geode/pull/5516#discussion_r493858900
##
File path:
geode-core/src/distributedTest/java/org/apache/geode/internal/cache/PartitionedRegionClearWithAlterRegionDUnitTest.java
##
@@ -0,0 +1,424 @@
jchen21 commented on a change in pull request #5516:
URL: https://github.com/apache/geode/pull/5516#discussion_r493858361
##
File path:
geode-core/src/distributedTest/java/org/apache/geode/internal/cache/PartitionedRegionClearWithAlterRegionDUnitTest.java
##
@@ -0,0 +1,424 @@
kirklund commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493738369
##
File path:
geode-core/src/test/java/org/apache/geode/management/bean/stats/MemberLevelStatsTest.java
##
@@ -339,6 +342,41 @@ public void
pdxcodemonkey opened a new pull request #655:
URL: https://github.com/apache/geode-native/pull/655
Found a log file this morning from an app that was calling `getKeys()` and
`getAll()`, and discovered these message that hadn't yet been added to the
parser.
@alb3rtobr @mreddington
sabbeyPivotal commented on pull request #5538:
URL: https://github.com/apache/geode/pull/5538#issuecomment-697674111
I'll take a look at this one!
This is an automated message from the Apache Git Service.
To respond to the
jinmeiliao commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493821071
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
jinmeiliao commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493822296
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
demery-pivotal commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493890530
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
nabarunnag merged pull request #5546:
URL: https://github.com/apache/geode/pull/5546
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
jinmeiliao opened a new pull request #5547:
URL: https://github.com/apache/geode/pull/5547
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
### For all
demery-pivotal commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493844666
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
moleske commented on pull request #645:
URL: https://github.com/apache/geode-native/pull/645#issuecomment-697695288
@pdxcodemonkey do you think it will be easier to rewrite those tests in the
new framework rather than find and fix the problem? It may take more time but
long term be more
jinmeiliao commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493896136
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
jinmeiliao commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493896436
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
onichols-pivotal merged pull request #5547:
URL: https://github.com/apache/geode/pull/5547
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
pivotal-jbarrett commented on pull request #645:
URL: https://github.com/apache/geode-native/pull/645#issuecomment-697999232
> I think we can diagnose the test problem with a couple of (very tedious)
hours of debugging, which is a lot better than rewriting the test. I think
we're much
sabbeyPivotal commented on pull request #5544:
URL: https://github.com/apache/geode/pull/5544#issuecomment-698047025
Distributed test failure is unrelated to this PR.
This is an automated message from the Apache Git Service.
kohlmu-pivotal opened a new pull request #5548:
URL: https://github.com/apache/geode/pull/5548
for the direct feature usage of ClassLoader and ClassLoaderPath
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask
onichols-pivotal merged pull request #5540:
URL: https://github.com/apache/geode/pull/5540
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
jinmeiliao commented on a change in pull request #5536:
URL: https://github.com/apache/geode/pull/5536#discussion_r493961781
##
File path:
geode-core/src/main/java/org/apache/geode/management/internal/beans/stats/GCStatsMonitor.java
##
@@ -39,69 +39,71 @@
* @see
onichols-pivotal commented on pull request #5543:
URL: https://github.com/apache/geode/pull/5543#issuecomment-697997186
Fixed by #5547 thanks!
This is an automated message from the Apache Git Service.
To respond to the
onichols-pivotal closed pull request #5543:
URL: https://github.com/apache/geode/pull/5543
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
36 matches
Mail list logo