[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #165 was SUCCESSFUL (with 1227 tests)

2015-12-21 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #165 was successful. --- Scheduled 1231 tests in total. https://build.spring.io/browse/SGF-NAG-165/ -- This

Re: Review Request 41615: Capture lines read from status file in finally-block of FileControlProcessor

2015-12-21 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41615/#review111556 --- gemfire-core/src/main/java/com/gemstone/gemfire/internal/process/F

Review Request 41615: Capture lines read from status file in finally-block of FileControlProcessor

2015-12-21 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41615/ --- Review request for geode, Jens Deppe and Jinmei Liao. Bugs: GEODE-473 https

Re: Annoyed with transitive = false for tests

2015-12-21 Thread Mark Bretl
Well, keeping track of the actual dependencies was one of the reasons, however, the main reason was the underlying dependency management done by Gradle (via Maven repositories) had to be 'turned off' due to incompatible versions in other dependencies or their downstream dependencies. There were dep

Annoyed with transitive = false for tests

2015-12-21 Thread Jens Deppe
I see that we're setting transitive to false for all dependencies. I guess that was done so that we're explicit about what is absolutely required as a dependency for GemFire. I'd like to propose we relax that for test dependencies. I really don't see the need to track every transitive dependency t

Re: Stats for GMS Health Monitor

2015-12-21 Thread Bruce Schuchardt
+1 Le 12/18/2015 3:46 PM, Jianxia Chen a écrit : Hi all, I am trying to implement the stats for Geode membership service health monitor, which monitors the health of the members of the distributed system by heartbeats. I will describe the stats that will be implemented. Please take a look and l

Re: Review Request 41581: Shutdown a distributed system will close the cache in another thread, make sure the cache is really closed

2015-12-21 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41581/#review111527 --- Ship it! Ship It! - Dan Smith On Dec. 21, 2015, 7:37 p.m., xiao

Re: Stats for GMS Health Monitor

2015-12-21 Thread Kirk Lund
+1 On Fri, Dec 18, 2015 at 3:46 PM, Jianxia Chen wrote: > Hi all, > > I am trying to implement the stats for Geode membership service health > monitor, which monitors the health of the members of the distributed system > by heartbeats. I will describe the stats that will be implemented. Please

Re: Review Request 41581: Shutdown a distributed system will close the cache in another thread, make sure the cache is really closed

2015-12-21 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41581/ --- (Updated Dec. 21, 2015, 7:37 p.m.) Review request for geode and Dan Smith. Re

Re: Stats for GMS Health Monitor

2015-12-21 Thread pulkit chandra
+1 In the past I have looked at logs and tried to match the timeline in stats to figure out why the event happened. This will reduce switching between logs and stats. Hence, I would say a very good addition ! Best Regards, *Pulkit Chandra* *412-641-9176* On Fri, Dec 18, 2015 at 6:46 PM, Jianxia

Re: Review Request 41581: Shutdown a distributed system will close the cache in another thread, make sure the cache is really closed

2015-12-21 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41581/ --- (Updated Dec. 21, 2015, 6:57 p.m.) Review request for geode and Dan Smith. Ch

Re: Review Request 41481: GEODE-660: Fixing inconsistencies and reliance on randomness in TableBuilderJUnitTest

2015-12-21 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41481/#review111513 --- Ship it! Ship It! - Kirk Lund On Dec. 17, 2015, 12:26 a.m., Jen

[GitHub] incubator-geode pull request: GEODE-543: upgrade the Jline and Spr...

2015-12-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/57 --- 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] incubator-geode pull request: Feature/geode 663: adding more secur...

2015-12-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/58 --- 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

Re: Review Request 41581: Shutdown a distributed system will close the cache in another thread, make sure the cache is really closed

2015-12-21 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41581/#review111506 --- Ship it! The change looks good, but it would be nice to use a stan

Review Request 41611: GEODE-707: cache loader not invoked on concurrent load if first load fails with an exception

2015-12-21 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41611/ --- Review request for geode and Darrel Schneider. Bugs: geode-707 https://issu

[GitHub] incubator-geode pull request: GEODE-568: SharedConfiguration regio...

2015-12-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/63 --- 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] incubator-geode pull request: GEODE-568: SharedConfiguration regio...

2015-12-21 Thread jdeppe-pivotal
Github user jdeppe-pivotal commented on the pull request: https://github.com/apache/incubator-geode/pull/63#issuecomment-166356173 +1 --- 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 fea

Build failed in Jenkins: Geode-nightly #321

2015-12-21 Thread Apache Jenkins Server
See -- [...truncated 256 lines...] :gemfire-web-api:processResources UP-TO-DATE :gemfire-web-api:classes :gemfire-web-api:jar :gemfire-web-api:war :gemfire-assembly:distTar :gemfire-assembly:distZip :gemfir