Build failed in Jenkins: Geode-nightly #878

2017-06-26 Thread Apache Jenkins Server
See Changes: [jiliao] Revert: GEODE-2924: fix lucene commands security test -- [...truncated 69.55 KB...] Note: Some input files use unchecked or unsafe operations. Note:

Failed: jinmeiliao/geode#1 (fix - 4d1f3bf)

2017-06-26 Thread Travis CI
Build Update for jinmeiliao/geode - Build: #1 Status: Failed Duration: 10 minutes and 48 seconds Commit: 4d1f3bf (fix) Author: Jinmei Liao Message: GEODE-2919: fix lucene security tests View the changeset:

Review Request 60451: GEODE-2996: adding Put handler

2017-06-26 Thread Brian Rowe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60451/ --- Review request for geode, Alexander Murmann, Bruce Schuchardt, Galen

[GitHub] geode pull request #604: GEODE-2919: fix lucene security tests

2017-06-26 Thread jinmeiliao
GitHub user jinmeiliao opened a pull request: https://github.com/apache/geode/pull/604 GEODE-2919: fix lucene security tests this is to fix the failing lucent tests. Created a PR to see how travis thinks about it. You can merge this pull request into a Git repository by running:

[GitHub] geode pull request #600: GEODE-1958: Rolling back changes to decrypt method

2017-06-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/600 --- 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

[GitHub] geode pull request #577: Feature/geode 3049

2017-06-26 Thread nreich
Github user nreich commented on a diff in the pull request: https://github.com/apache/geode/pull/577#discussion_r124145519 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/BucketRedundancyTracker.java --- @@ -0,0 +1,110 @@ +/* + * Licensed to the Apache

Re: Review Request 60446: Events do not get removed from the client queue for 1.0 clients

2017-06-26 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60446/#review178936 --- Ship it! Ship It! - Hitesh Khamesra On June 26, 2017, 10:24

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #597 has FAILED (2 tests failed)

2017-06-26 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #597 failed. --- Scheduled 2/1870 tests failed. https://build.spring.io/browse/SGF-NAG-597/

Re: Review Request 60446: Events do not get removed from the client queue for 1.0 clients

2017-06-26 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60446/ --- (Updated June 26, 2017, 10:24 p.m.) Review request for geode and Barry

Re: Review Request 60446: Events do not get removed from the client queue for 1.0 clients

2017-06-26 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60446/#review178932 --- Ship it! Ship It! - Barry Oglesby On June 26, 2017, 8:45

[GitHub] geode pull request #577: Feature/geode 3049

2017-06-26 Thread dschneider-pivotal
Github user dschneider-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/577#discussion_r124134874 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/BucketRedundancyTracker.java --- @@ -0,0 +1,110 @@ +/* + * Licensed to

Review Request 60447: GEODE-3132 EndBucketCreationMessage should not partitcipate in a transaction

2017-06-26 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60447/ --- Review request for geode, anilkumar gingade, Darrel Schneider, and Lynn

Re: Review Request 60446: Events do not get removed from the client queue for 1.0 clients

2017-06-26 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60446/#review178926 --- Fix it, then Ship it! Fix it (or drop it) and then ship it!

re: 1.2 release blocker

2017-06-26 Thread Bruce Schuchardt
I've posted a review for GEODE-3072 on reviews.apache.org. https://reviews.apache.org/r/60446/

Review Request 60446: Events do not get removed from the client queue for 1.0 clients

2017-06-26 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60446/ --- Review request for geode and Barry Oglesby. Bugs: GEODE-3072

[GitHub] geode issue #603: GEODE-3128: Changed the batch size to 1000 when creating A...

2017-06-26 Thread DivineEnder
Github user DivineEnder commented on the issue: https://github.com/apache/geode/pull/603 @ladyVader @nabarunnag @boglesby @jhuynh1 @upthewaterspout @gesterzhou --- 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

[GitHub] geode pull request #603: GEODE-3128: Changed the batch size to 1000 when cre...

2017-06-26 Thread DivineEnder
GitHub user DivineEnder opened a pull request: https://github.com/apache/geode/pull/603 GEODE-3128: Changed the batch size to 1000 when creating AEQFactory * Prevously the batch size was the default 100 * Also added Lucene index overhead tests as performance tests. Currently

[GitHub] geode pull request #602: GEODE-3130: Refactoring AcceptorImpl

2017-06-26 Thread WireBaron
GitHub user WireBaron opened a pull request: https://github.com/apache/geode/pull/602 GEODE-3130: Refactoring AcceptorImpl We extracted a switch statement in AcceptorImpl.handleNewClientConnection to a new method. Signed-off-by: Alexander Murmann

Re: Review Request 60442: GEODE-3130: Refactoring AcceptorImpl communication mode switch

2017-06-26 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60442/#review178916 --- Ship it! Ship It! - Hitesh Khamesra On June 26, 2017, 6:31

[GitHub] geode pull request #593: GEODE-2995: Handle stream of ProtoBuf encoded messa...

2017-06-26 Thread WireBaron
Github user WireBaron closed the pull request at: https://github.com/apache/geode/pull/593 --- 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

Review Request 60442: GEODE-3130: Refactoring AcceptorImpl communication mode switch

2017-06-26 Thread Brian Rowe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60442/ --- Review request for geode, Alexander Murmann, Bruce Schuchardt, Galen

[GitHub] geode issue #529: GEODE-2980: All the methods in an interface are implicitly...

2017-06-26 Thread kirklund
Github user kirklund commented on the issue: https://github.com/apache/geode/pull/529 @davinash please rebase on develop to fix the new conflicts and I'll merge to develop. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] geode pull request #601: GEODE-3117: fix Gateway authentication with legacy ...

2017-06-26 Thread kirklund
GitHub user kirklund opened a pull request: https://github.com/apache/geode/pull/601 GEODE-3117: fix Gateway authentication with legacy security * add GatewayLegacyAuthenticationRegressionTest to reproduce bug * fix authentication of Gateway sender/receiver with

[GitHub] geode pull request #597: Client protocol with develop, gated by a feature fl...

2017-06-26 Thread galen-pivotal
Github user galen-pivotal closed the pull request at: https://github.com/apache/geode/pull/597 --- 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 issue #597: Client protocol with develop, gated by a feature flag.

2017-06-26 Thread galen-pivotal
Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/597 merged in e1c6c3a . --- 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

[GitHub] geode pull request #599: GEODE-2819 Document http status code 414 for PUT

2017-06-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/599 --- 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

Re: Review Request 60394: GEODE-3075 and GEODE-2995: merge of new protobuf protocol work.

2017-06-26 Thread Alexander Murmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60394/#review178901 --- Ship it! Ship It! - Alexander Murmann On June 23, 2017,

[GitHub] geode pull request #600: GEODE-1958: Rolling back changes to decrypt method

2017-06-26 Thread YehEmily
GitHub user YehEmily opened a pull request: https://github.com/apache/geode/pull/600 GEODE-1958: Rolling back changes to decrypt method Earlier, I made changes to the `decrypt` method in `PasswordUtil` to allow it to decrypt any password, when it should only be decrypting passwords