Build failed in Jenkins: Geode-nightly #578

2016-09-01 Thread Apache Jenkins Server
See Changes: [hkhamesra] GEODE-900 Test was using stale cache. [hkhamesra] GEODE-1372 Updated java docs. [hkhamesra] GEODE-1828 Added logging to debug the test [boglesby] GEODE-1819: modified BRQ so that all initialization is done befor

[GitHub] incubator-geode pull request #235: GEODE-1818: Bug37377DUnitTest passes with...

2016-09-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/235 --- 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 featur

Review Request 51577: GEODE-1834: initilize the socketcreator correctly

2016-09-01 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51577/ --- Review request for geode, Bruce Schuchardt and Kirk Lund. Repository: geode D

Re: Review Request 51577: GEODE-1834: initilize the socketcreator correctly

2016-09-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51577/#review147579 --- geode-core/src/main/java/com/gemstone/gemfire/internal/SocketCrea

Re: Review Request 51577: GEODE-1834: initilize the socketcreator correctly

2016-09-01 Thread Jinmei Liao
> On Sept. 1, 2016, 5:26 p.m., Kirk Lund wrote: > > geode-core/src/main/java/com/gemstone/gemfire/internal/SocketCreator.java, > > line 331 > > > > > > No calls ever pass in "true" for reinitialize. I think this sho

[GitHub] incubator-geode pull request #236: Feature/geode 1570

2016-09-01 Thread kjduling
GitHub user kjduling opened a pull request: https://github.com/apache/incubator-geode/pull/236 Feature/geode 1570 temp pull request - don't accept You can merge this pull request into a Git repository by running: $ git pull https://github.com/kjduling/incubator-geode feature/GE

Re: Review Request 51577: GEODE-1834: initilize the socketcreator correctly

2016-09-01 Thread Kirk Lund
> On Sept. 1, 2016, 5:26 p.m., Kirk Lund wrote: > > geode-core/src/main/java/com/gemstone/gemfire/internal/SocketCreator.java, > > line 331 > > > > > > No calls ever pass in "true" for reinitialize. I think this sho

Build failed in Jenkins: Geode-nightly #579

2016-09-01 Thread Apache Jenkins Server
See Changes: [abaker] GEODE-1809: Improve error handling [gzhou] GEODE-11: create index repository using raw Lucene directory. [gzhou] GEODE-11: need to fix the junit test case when adding FSDirectory ---

Fwd: Build failed in Jenkins: Geode-nightly #579

2016-09-01 Thread Kirk Lund
We're getting OOME failures in Hudson fairly frequently in the nightly build. Any ideas what the issue is? Do we need to change the job configuration or exclude more build machines? -Kirk -- Forwarded message -- From: Apache Jenkins Server Date: Thu, Sep 1, 2016 at 1:23 PM Subjec

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

2016-09-01 Thread Dan Smith
I think we've seen something like this before when tests are logging too much. Jenkins is trying to parse the test result xml files, which contain all of the stdout from the test. -Dan On Thu, Sep 1, 2016 at 1:41 PM, Kirk Lund wrote: > We're getting OOME failures in Hudson fairly frequently in

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

2016-09-01 Thread Kirk Lund
#577 hit this same OOME failure. However, #578 did not. It failed with a BindException in DiskRegionJUnitTest. -Kirk On Thu, Sep 1, 2016 at 1:56 PM, Dan Smith wrote: > I think we've seen something like this before when tests are logging too > much. Jenkins is trying to parse the test result x

Review Request 51585: let getId() return unique name

2016-09-01 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51585/ --- Review request for geode and Dan Smith. Bugs: GEODE-11 https://issues.apach

Re: Review Request 51585: let getId() return unique name

2016-09-01 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51585/#review147613 --- I think the ID needs to include something indicating that this pro

[GitHub] incubator-geode issue #236: Feature/geode 1570

2016-09-01 Thread kjduling
Github user kjduling commented on the issue: https://github.com/apache/incubator-geode/pull/236 Closing as this was only a temp. --- 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 e

[GitHub] incubator-geode pull request #236: Feature/geode 1570

2016-09-01 Thread kjduling
Github user kjduling closed the pull request at: https://github.com/apache/incubator-geode/pull/236 --- 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 feat

Re: Review Request 51585: let getId() return unique name

2016-09-01 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51585/ --- (Updated Sept. 1, 2016, 10:55 p.m.) Review request for geode and Dan Smith. C

Review Request 51589: GEODE-1832: remove dependency on cacheRunner package

2016-09-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51589/ --- Review request for geode, Jason Huynh and nabarun nag. Bugs: GEODE-1832 htt

Review Request 51590: GEODE-1838: remove unused test classes

2016-09-01 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51590/ --- Review request for geode, Jason Huynh and nabarun nag. Bugs: GEODE-1838 htt

Re: Review Request 51589: GEODE-1832: remove dependency on cacheRunner package

2016-09-01 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51589/#review147620 --- Ship it! Ship It! - Jason Huynh On Sept. 2, 2016, 12:15 a.m.

Re: Review Request 51590: GEODE-1838: remove unused test classes

2016-09-01 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51590/#review147621 --- Ship it! Ship It! - Jason Huynh On Sept. 2, 2016, 12:18 a.m.

Re: Review Request 51585: let getId() return unique name

2016-09-01 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51585/ --- (Updated Sept. 2, 2016, 12:48 a.m.) Review request for geode and Dan Smith. C