Jenkins build is back to normal : Geode-nightly-flaky #102

2017-08-25 Thread Apache Jenkins Server
See

[GitHub] geode pull request #742: GEODE-3473: Initial commit of the internal package ...

2017-08-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/742 --- 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 is enabl

Re: Review Request 61895: GEDOE-3516: TXManagerImpl.tryResume call may add itself again into the waiting thread queue

2017-08-25 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61895/ --- (Updated Aug. 25, 2017, 4:53 p.m.) Review request for geode, anilkumar gingade,

[GitHub] geode pull request #737: GEODE-3503: Removal of Codec classes left behind.

2017-08-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/737 --- 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 is enabl

Broken: apache/geode#3691 (develop - 4ea83c3)

2017-08-25 Thread Travis CI
Build Update for apache/geode - Build: #3691 Status: Broken Duration: 9 minutes and 57 seconds Commit: 4ea83c3 (develop) Author: Kirk Lund Message: GEODE-3461: GEODE-3505: mark test with FlakyTest category * also increase timeout to wait for process to die Vi

Fixed: apache/geode#3692 (develop - 92bf7fa)

2017-08-25 Thread Travis CI
Build Update for apache/geode - Build: #3692 Status: Fixed Duration: 19 minutes and 49 seconds Commit: 92bf7fa (develop) Author: Udo Kohlmeyer Message: GEODE-3503: fixing the build by correcting the formatting View the changeset: https://github.com/apache/geo

Broken: apache/geode#3690 (develop - d775dfb)

2017-08-25 Thread Travis CI
Build Update for apache/geode - Build: #3690 Status: Broken Duration: 25 minutes and 51 seconds Commit: d775dfb (develop) Author: Udo Kohlmeyer Message: GEODE-3503: Removal of Codec classes left behind. This closes #737 Added tests to test the remaining JSO

Re: Review Request 61895: GEDOE-3516: TXManagerImpl.tryResume call may add itself again into the waiting thread queue

2017-08-25 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61895/#review183871 --- geode-core/src/main/java/org/apache/geode/internal/cache/TXManage

Re: Review Request 61860: GEODE-3510: GfshRule displays output from StdError

2017-08-25 Thread Jared Stewart
> On Aug. 24, 2017, 9:29 p.m., Kirk Lund wrote: > > geode-assembly/src/test/java/org/apache/geode/test/dunit/rules/gfsh/ProcessLogger.java > > Lines 96 (patched) > > > > > > I'm curious, what's the result of this meth

[GitHub] geode issue #719: GEODE-3447 Implement client authorization for the new prot...

2017-08-25 Thread galen-pivotal
Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/719 @metatype We need the `StreamAuthenticator` to receive and send (Protobuf-encoded) messages containing the credentials that get passed to the `SecurityManager`. I would think that ideally it's n

[GitHub] geode pull request #739: GEODE-3385: Change GetAllRequest to return list of ...

2017-08-25 Thread galen-pivotal
Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/739#discussion_r135351811 --- Diff: geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/GetAllRequestOperationHandler.java --- @@ -50,26 +53,52 @@

[GitHub] geode issue #719: GEODE-3447 Implement client authorization for the new prot...

2017-08-25 Thread metatype
Github user metatype commented on the issue: https://github.com/apache/geode/pull/719 I'm not suggesting changing the protobuf message. That seems ok to me. Since the `StreamAuthenticator` is part of protobuf messaging I think it should remain in the `geode-protobuf` module.

Re: Review Request 61860: GEODE-3510: GfshRule displays output from StdError

2017-08-25 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61860/ --- (Updated Aug. 25, 2017, 9:12 p.m.) Review request for geode, Emily Yeh, Jared S

Re: Review Request 61701: GEODE-3277: Fix error path constructors of Launcher inner State classess

2017-08-25 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61701/#review183875 --- Ship it! Ship It! - Kirk Lund On Aug. 24, 2017, 10:02 p.m.,

[Discuss] Enable region set operations to start JTA transactions

2017-08-25 Thread Eric Shu
Hi Team, Currently in GEODE JTA implementation, if GEODE transaction is not yet bootstrapped (there is no TXState created yet for the transaction), region set operations will not bootstrap the JTA transaction. In other word, if the first operation of the JTA transaction is region set operation, th

[GitHub] geode pull request #744: GEODE-3522: create via load needs to add event to A...

2017-08-25 Thread ladyVader
GitHub user ladyVader opened a pull request: https://github.com/apache/geode/pull/744 GEODE-3522: create via load needs to add event to AEQ under same lock… … as update to local region * create via load was invoking handleWANEvent prior to obtaining a lock, which allowe

[VOTE] Apache Geode release - v1.2.1 RC1

2017-08-25 Thread Anthony Baker
This is the first release candidate for Apache Geode, version 1.2.1. Thanks to all the community members for their contributions to this release! *** Please download, test and vote by Wednesday, August 25, 0800 hrs US Pacific. *** It fixes the following issues: https://issues.apache.org/jira/s

Re: [Discuss] Enable region set operations to start JTA transactions

2017-08-25 Thread Anilkumar Gingade
+1 Having a consistent behavior for set operation within the transaction (wherever its invoked). And having a gemfire property to allow old way. On Fri, Aug 25, 2017 at 2:49 PM, Eric Shu wrote: > Hi Team, > > Currently in GEODE JTA implementation, if GEODE transaction is not yet > bootstrapped (

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

2017-08-25 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #658 was successful. --- Scheduled 2029 tests in total. https://build.spring.io/browse/SGF-NAG-658/ -- This

[GitHub] geode pull request #738: GEODE-3506: improve validation/error checking for p...

2017-08-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/738 --- 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 is enabl

[GitHub] geode pull request #731: GEODE-2842: Removed redundant default annotation pa...

2017-08-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/731 --- 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 is enabl

[GitHub] geode issue #733: Keeping old versions around even during a clean

2017-08-25 Thread upthewaterspout
Github user upthewaterspout commented on the issue: https://github.com/apache/geode/pull/733 Closing this PR, it seems like this isn't the right change. I'm a little short on time to mess with this, so if someone else wants to tackle handling the download in a nicer way please do.

[GitHub] geode pull request #733: Keeping old versions around even during a clean

2017-08-25 Thread upthewaterspout
Github user upthewaterspout closed the pull request at: https://github.com/apache/geode/pull/733 --- 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] geode pull request #741: GEODE-3512 Doc removal of gfsh encrypt password

2017-08-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/741 --- 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 is enabl

Passed: apache/geode#3695 (rel/v1.2.1.RC1 - 1bb1cae)

2017-08-25 Thread Travis CI
Build Update for apache/geode - Build: #3695 Status: Passed Duration: 22 minutes and 19 seconds Commit: 1bb1cae (rel/v1.2.1.RC1) Author: Karen Miller Message: GEODE-3249 Document geode.allow-internal-messages-without-credentials This closes #735 (cherry p

[GitHub] geode pull request #744: GEODE-3522: create via load needs to add event to A...

2017-08-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/744 --- 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 is enabl

[GitHub] geode pull request #325: merge new version

2017-08-25 Thread zmyer
Github user zmyer closed the pull request at: https://github.com/apache/geode/pull/325 --- 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 is enable