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

--- Spring Data GemFire > Nightly-ApacheGeode > #154 was successful. --- Scheduled 1231 tests in total. https://build.spring.io/browse/SGF-NAG-154/ -- This

Re: Review Request 41229: StateFlushOperation flushing/waiting for multicast messages

> On Dec. 11, 2015, 12:05 a.m., anilkumar gingade wrote: > > gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/JGroupsMessenger.java, > > line 517 > > > > > > Just wonderi

[GitHub] incubator-geode pull request: GEM-164: move the security tests in ...

Github user metatype commented on the pull request: https://github.com/apache/incubator-geode/pull/52#issuecomment-163792393 Can you remove the Pivotal copyright and add the ASF license header? --- If your project is set up for it, you can reply to this email and have your reply appea

Re: Review Request 41229: StateFlushOperation flushing/waiting for multicast messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41229/#review109890 --- gemfire-core/src/main/java/com/gemstone/gemfire/distributed/intern

[GitHub] incubator-geode pull request: Additional offheap test coverage bas...

GitHub user gemzdude opened a pull request: https://github.com/apache/incubator-geode/pull/53 Additional offheap test coverage based on jira ticket GEODE-624 New unit test for RefCountChangeInfo Modified format of stack trace maintained within RefCountChangeInfo Test covera

Re: Review Request 41044: Update process stream handling in tests that fork JVMs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41044/ --- (Updated Dec. 10, 2015, 11:54 p.m.) Review request for geode, Darrel Schneider

[GitHub] incubator-geode pull request: GEM-164: move the security tests in ...

GitHub user jinmeiliao opened a pull request: https://github.com/apache/incubator-geode/pull/52 GEM-164: move the security tests in gemfire-test module inside the co… …m.gemstone.gemfire.security packages to the open side. and related resource files You can merge this p

Re: Review Request 41044: Update process stream handling in tests that fork JVMs

> On Dec. 10, 2015, 11:16 p.m., Jens Deppe wrote: > > gemfire-core/src/test/java/com/gemstone/gemfire/test/process/ProcessWrapper.java, > > line 53 > > > > > > Can the timeouts be consolidated at all? > > > >

Re: Review Request 41044: Update process stream handling in tests that fork JVMs

> On Dec. 10, 2015, 11:16 p.m., Jens Deppe wrote: > > gemfire-core/src/test/java/com/gemstone/gemfire/test/process/ProcessWrapper.java, > > line 53 > > > > > > Can the timeouts be consolidated at all? > > > >

Re: Review Request 41044: Update process stream handling in tests that fork JVMs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41044/#review109886 --- gemfire-core/src/test/java/com/gemstone/gemfire/test/process/Proce

Review Request 41229: StateFlushOperation flushing/waiting for multicast messages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41229/ --- Review request for geode, anilkumar gingade, Hitesh Khamesra, Jason Huynh, and J

Re: Review Request 41222: GEODE-528: Locator not stopping correctly if jmx-manager-port=0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41222/#review109883 --- Ship it! Ship It! - Kirk Lund On Dec. 10, 2015, 9:26 p.m., Jens

Re: Review Request 41224: added null check in checkAddressesForUUIDs method

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41224/#review109880 --- Ship it! Ship It! - Bruce Schuchardt On Dec. 10, 2015, 9:57 p.m

[GitHub] incubator-geode pull request: Feature/geode 624

Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/51 --- 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 41224: added null check in checkAddressesForUUIDs method

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41224/#review109869 --- gemfire-core/src/main/java/com/gemstone/gemfire/distributed/intern

Re: Review Request 41224: added null check in checkAddressesForUUIDs method

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41224/#review109867 --- gemfire-core/src/main/java/com/gemstone/gemfire/distributed/intern

Review Request 41224: added null check in checkAddressesForUUIDs method

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41224/ --- Review request for geode and Bruce Schuchardt. Repository: geode Description

Review Request 41222: GEODE-528: Locator not stopping correctly if jmx-manager-port=0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41222/ --- Review request for geode and Kirk Lund. Repository: geode Description ---

Re: Review Request 41219: GEODE-503: Addresses config passwords written to logs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41219/#review109845 --- Ship it! Ship It! - Darrel Schneider On Dec. 10, 2015, 11:29 a.

[GitHub] incubator-geode pull request: Feature/geode 624

GitHub user gemzdude opened a pull request: https://github.com/apache/incubator-geode/pull/51 Feature/geode 624 Additional offheap test coverage based on jira ticket GEODE-624 You can merge this pull request into a Git repository by running: $ git pull https://github.com/gemzdu

[GitHub] incubator-geode pull request: Feature/geode 623

Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/48 --- 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: Fixes Geode 644: DataAsAddress needs...

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

Review Request 41219: GEODE-503: Addresses config passwords written to logs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41219/ --- Review request for geode and Darrel Schneider. Repository: geode Description

Re: Build failed in Jenkins: Geode-nightly #307

On Thu, Dec 10, 2015 at 12:09 PM, Kirk Lund wrote: > If all Apache projects use Git, Git is actually *very* new at Apache. In fact, there's this little project that's part of the foundation called Apache SVN ;-) > why would one of their Jenkins machines be > missing Git? Sounds like something bu

Re: Build failed in Jenkins: Geode-nightly #307

If all Apache projects use Git, why would one of their Jenkins machines be missing Git? Sounds like something builds@apache might want to know about and remove. -Kirk On Thu, Dec 10, 2015 at 9:48 AM, Mark Bretl wrote: > We can do it via the job configuration. I will look at it and add a filter

Re: Review Request 41016: GEODE-18: Add missing license headers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41016/#review109802 --- Ship it! - Dick Cavender On Dec. 9, 2015, 10:53 p.m., Anthony Ba

Re: Build failed in Jenkins: Geode-nightly #307

We can do it via the job configuration. I will look at it and add a filter for the macMini. --Mark On Thu, Dec 10, 2015 at 9:43 AM, Kirk Lund wrote: > Do we need to email bui...@apache.org to request that they remove this > machine from the Jenkins rotation? > > -Kirk > > > On Thu, Dec 10, 2015

RAT support

RAT is a tool that checks for source headers in files and it is now integrated with our build. You can execute the checks directly using `gradle rat` I’ve also added it as a dependency for the check task. This means that the build will fail if you add a new file without a proper source header

Re: Review Request 41016: GEODE-18: Add missing license headers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41016/#review109800 --- Ship it! Ship It! - William Markito On Dec. 9, 2015, 10:53 p.m.

Re: Build failed in Jenkins: Geode-nightly #307

Do we need to email bui...@apache.org to request that they remove this machine from the Jenkins rotation? -Kirk On Thu, Dec 10, 2015 at 9:21 AM, Dan Smith wrote: > Looks like this tried to run the nightly build on a mac mini with no git > installed? > > -Dan > > On Wed, Dec 9, 2015 at 7:44 PM,

Re: Build failed in Jenkins: Geode-nightly #307

Looks like this tried to run the nightly build on a mac mini with no git installed? -Dan On Wed, Dec 9, 2015 at 7:44 PM, Apache Jenkins Server < jenk...@builds.apache.org> wrote: > See > > -- > Started by