Build failed in Jenkins: Geode-nightly #670

See Changes: [jiliao] GEODE-1955: make the tests more stable [bschuchardt] GEODE-2137 client membership IDs contain 17 bytes of useless information [hkhamesra] GEODE-2011 Removed flaky mark from test. [upthewaterspout] GEODE-2157: Addi

Broken: apache/incubator-geode#1508 (develop - 890e6c7)

Build Update for apache/incubator-geode - Build: #1508 Status: Broken Duration: 7 minutes and 13 seconds Commit: 890e6c7 (develop) Author: Udo Kohlmeyer Message: GEODE-1594: Reduced iteration counts of actions and removed unnecessary CacheFactory invocation V

Broken: apache/incubator-geode#1507 (develop - f0e5159)

Build Update for apache/incubator-geode - Build: #1507 Status: Broken Duration: 7 minutes and 26 seconds Commit: f0e5159 (develop) Author: Udo Kohlmeyer Message: GEODE-1931: Refactored tests to use Awaitility and lambdas instead of Wait.pause and WaitCriterion

Broken: apache/incubator-geode#1509 (develop - 423e04f)

Build Update for apache/incubator-geode - Build: #1509 Status: Broken Duration: 7 minutes and 12 seconds Commit: 423e04f (develop) Author: Dan Smith Message: GEODE-2157: Fixing sanctionedSerializables for PutAllPartialResultEx View the changeset: https://gith

Broken: apache/incubator-geode#1510 (develop - 5928459)

Build Update for apache/incubator-geode - Build: #1510 Status: Broken Duration: 7 minutes and 19 seconds Commit: 5928459 (develop) Author: Dave Barnes Message: Merge branch 'develop' of https://git-wip-us.apache.org/repos/asf/incubator-geode into develop * 'd

Re: Review Request 54240: hdos.toByteArray() is unneeded

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54240/#review157609 --- Ship it! Ship It! - Dan Smith On Dec. 1, 2016, 1:54 a.m., xi

Re: Review Request 54210: GEODE-2146: deploy should require more elevated privileges than just data:manage

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54210/#review157612 --- Ship it! Ship It! - Kirk Lund On Nov. 30, 2016, 9:27 p.m., J

[GitHub] incubator-geode issue #293: GEODE-1653: Executing a fire-and-forget function...

Github user kirklund commented on the issue: https://github.com/apache/incubator-geode/pull/293 The Travis failure is caused by the format not matching spotless. Unfortunately, the IntelliJ formatter doesn't currently match, so the easiest way to fix the format is to run: `$

Re: Review Request 54240: hdos.toByteArray() is unneeded

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54240/#review157619 --- Ship it! Ship It! - Darrel Schneider On Nov. 30, 2016, 5:54

[GitHub] incubator-geode issue #296: GEODE-2109 : Calling submit on ExecutionService ...

Github user kirklund commented on the issue: https://github.com/apache/incubator-geode/pull/296 We've been requiring unit tests or integration tests for all bug fixes. At a minimum, I would recommend adding a new test that fails due to at least of the conditions reported by G

[GitHub] incubator-geode issue #299: [ GEODE-2141 ] #comment Fix Issue #2141

Github user upthewaterspout commented on the issue: https://github.com/apache/incubator-geode/pull/299 +1 - looks good to me, I think it's ready to merge. --- 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 d

Re: Review Request 54235: GEODE-2159 add serialVersionUIDs to exception classes not having them

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54235/#review157623 --- Ship it! Ship It! - Udo Kohlmeyer On Nov. 30, 2016, 11:34 p.

[GitHub] incubator-geode pull request #297: GEODE-734: gfsh export stack-traces shoul...

Github user kirklund commented on a diff in the pull request: https://github.com/apache/incubator-geode/pull/297#discussion_r90507113 --- Diff: geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ShowStackTraceDUnitTest.java --- @@ -149,4 +163,92 @@ public v

[GitHub] incubator-geode pull request #297: GEODE-734: gfsh export stack-traces shoul...

Github user kirklund commented on a diff in the pull request: https://github.com/apache/incubator-geode/pull/297#discussion_r90506480 --- Diff: geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ShowStackTraceDUnitTest.java --- @@ -90,6 +90,20 @@ public voi

[GitHub] incubator-geode issue #293: GEODE-1653: Executing a fire-and-forget function...

Github user kirklund commented on the issue: https://github.com/apache/incubator-geode/pull/293 Oops, looks like the LocatorDUnitTest format problem is on develop, not in this PR. Sorry about my confusion. --- If your project is set up for it, you can reply to this email and have you

[GitHub] incubator-geode pull request #301: UITests actually take screenshots on fail...

GitHub user jaredjstewart opened a pull request: https://github.com/apache/incubator-geode/pull/301 UITests actually take screenshots on failure The ScreenShotOnFailureRule had been moved inside of another Rule, but this doesn't actually work. So I moved it back out into each UI te

Re: Review Request 54210: GEODE-2146: deploy should require more elevated privileges than just data:manage

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54210/#review157629 --- Ship it! Ship It! - Kevin Duling On Nov. 30, 2016, 1:27 p.m.

[GitHub] incubator-geode pull request #282: GEODE-1740: Inconsistencies with concurre...

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

[GitHub] incubator-geode issue #282: GEODE-1740: Inconsistencies with concurrent clea...

Github user gemzdude commented on the issue: https://github.com/apache/incubator-geode/pull/282 Merged to develop by Ken Howe --- 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 enab

Review Request 54266: GEODE-2153 temporary docs update for post processing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54266/ --- Review request for geode, Dave Barnes, Galen O'Sullivan, Jinmei Liao, Joey McAll

Re: Review Request 54266: GEODE-2153 temporary docs update for post processing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54266/#review157659 --- Ship it! Ship It! - Joey McAllister On Dec. 1, 2016, 9:42 p.

Re: Review Request 54266: GEODE-2153 temporary docs update for post processing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54266/#review157660 --- Ship it! Ship It! - Jinmei Liao On Dec. 1, 2016, 9:42 p.m.,

Review Request 54268: geode-2137 client membership IDs contain 17 bytes of useless information

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54268/ --- Review request for geode, Galen O'Sullivan, Hitesh Khamesra, and Udo Kohlmeyer.

Re: Review Request 54268: geode-2137 client membership IDs contain 17 bytes of useless information

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54268/#review157665 --- Ship it! Ship It! - Hitesh Khamesra On Dec. 1, 2016, 10:42 p

Re: Review Request 54266: GEODE-2153 temporary docs update for post processing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54266/#review157667 --- Ship it! Ship It! - Dave Barnes On Dec. 1, 2016, 9:42 p.m.,

Re: Review Request 54266: GEODE-2153 temporary docs update for post processing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54266/#review157671 --- Ship it! Ship It! - Jared Stewart On Dec. 1, 2016, 9:42 p.m.

Review Request 54272: GEODE-2162 AsyncEventListenerDUnitTest testParallelAsyncEventQueueWithOneAccessor hang on tearDown

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

[GitHub] incubator-geode pull request #293: GEODE-1653: Executing a fire-and-forget f...

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

Re: Review Request 54272: GEODE-2162 AsyncEventListenerDUnitTest testParallelAsyncEventQueueWithOneAccessor hang on tearDown

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54272/#review157680 --- I'd like to talk with you about this change. The whole wait/notif

Review Request 54283: GEODE-1978: Waiting for the queue to drain in testReplicatedSerialProp...

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

TLP Transition Changes Coming

Hi Everyone, I have received notification that the INFRA team will hopefully start our TLP transition tasks either tomorrow, Friday, or Monday. These tasks consist of, but not limited to: mailing list migration Apache Git URL migration GitHub URL migration JIRA/GitHub Bot I am sure things might

Re: Review Request 54272: GEODE-2162 AsyncEventListenerDUnitTest testParallelAsyncEventQueueWithOneAccessor hang on tearDown

> On Dec. 2, 2016, 12:31 a.m., Bruce Schuchardt wrote: > > I'd like to talk with you about this change. The whole wait/notify thing > > seems fragile to me - a Future might be a lot better device to use here. yeah lets talk tomorrow on this. Thanks. - Hitesh ---

Re: Review Request 54283: GEODE-1978: Waiting for the queue to drain in testReplicatedSerialProp...

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54283/#review157685 --- Ship it! Ship It! - nabarun nag On Dec. 2, 2016, 1:18 a.m.,

[GitHub] incubator-geode pull request #299: [ GEODE-2141 ] #comment Fix Issue #2141

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

[GitHub] incubator-geode issue #275: [GEODE-1871] - Updating and cleaning up Dockerfi...

Github user metatype commented on the issue: https://github.com/apache/incubator-geode/pull/275 Is this ready to be merged? --- 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 enable

Re: Review Request 54283: GEODE-1978: Waiting for the queue to drain in testReplicatedSerialProp...

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54283/#review157690 --- Ship it! Ship It! - Barry Oglesby On Dec. 2, 2016, 1:18 a.m.

Re: TLP Transition Changes Coming

I might recommend they setup redirects if people have bookmarked the old URLs. On Thu, Dec 1, 2016 at 5:23 PM, Mark Bretl wrote: > Hi Everyone, > > I have received notification that the INFRA team will hopefully start our > TLP transition tasks either tomorrow, Friday, or Monday. > > These tasks

[GitHub] incubator-geode pull request #297: GEODE-734: gfsh export stack-traces shoul...

Github user deepakddixit commented on a diff in the pull request: https://github.com/apache/incubator-geode/pull/297#discussion_r90585692 --- Diff: geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ShowStackTraceDUnitTest.java --- @@ -90,6 +90,20 @@ public

Re: TLP Transition Changes Coming

Thanks Mark! Can you let us know what has been done so far according to [1]? I see that p.a.o [2] has been updated, but I’m curious about the committer list. It looks quite different compared to during incubation. Is that correct? Anthony [1] http://incubator.apache.org/guides/graduation.htm

Re: TLP Transition Changes Coming

Also, I think we need additional subtasks on INFRA-12937 for migrating: - GitHub mirror - release folders - Travis-CI - JIRA - ReviewBoard - Sonar See [1] for examples. LMK if you want me to create the subtasks. Anthony [1] https://issues.apache.org/jira/issues/?jql=project%20%3D%20INFRA%20AN