Build failed in Jenkins: Geode-nightly-flaky #51

2017-06-30 Thread Apache Jenkins Server
See Changes: [bschuchardt] GEODE-3139 remove current artifacts from classpath of [dschneider] GEODE-3098: fix flaky test failure [boglesby] GEODE-3152: Changed to create a region name appropriate to the

Re: Review Request 60570: GEODE-3153 Client receives duplicate events during rolling upgrade

2017-06-30 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60570/ --- (Updated June 30, 2017, 4:17 p.m.) Review request for geode, Alexander

Re: Review Request 60570: GEODE-3153 Client receives duplicate events during rolling upgrade

2017-06-30 Thread Bruce Schuchardt
> On June 30, 2017, 2:56 p.m., Galen O'Sullivan wrote: > > I've been looking at this with @WireBaron, and we're wondering whether a > > client membership ID can still get sent with a zeroed UUID if it's passed > > between two Gemfire 9.1 servers as a result of client queue replication > >

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

2017-06-30 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #602 was successful. --- Scheduled 1910 tests in total. https://build.spring.io/browse/SGF-NAG-602/ -- This

Re: Review Request 60570: GEODE-3153 Client receives duplicate events during rolling upgrade

2017-06-30 Thread Bruce Schuchardt
> On June 30, 2017, 2:56 p.m., Galen O'Sullivan wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/EventID.java > > Lines 301 (patched) > > > > > > Maybe declare `dis` and then set it based on the

Re: Review Request 60570: GEODE-3153 Client receives duplicate events during rolling upgrade

2017-06-30 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60570/ --- (Updated June 30, 2017, 3:02 p.m.) Review request for geode, Alexander

Re: Review Request 60570: GEODE-3153 Client receives duplicate events during rolling upgrade

2017-06-30 Thread Galen O'Sullivan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60570/#review179384 --- I've been looking at this with @WireBaron, and we're wondering

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

2017-06-30 Thread PurelyApplied
Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/601 If you are using IntelliJ as your IDE, please remember to update your style file to that located in `geode/etc/intellij-java-modified-google-style.xml` to be consistent with Geode's established

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

2017-06-30 Thread PurelyApplied
Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/603 If you are using IntelliJ as your IDE and have not already done so, please remember to update your style file to that located in `geode/etc/intellij-java-modified-google-style.xml` to be

[GitHub] geode issue #609: GEODE-2886 : sent IllegalStateException instead of throwin...

2017-06-30 Thread PurelyApplied
Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/609 If you are using IntelliJ as your IDE, please remember to update your style file to that located in `geode/etc/intellij-java-modified-google-style.xml` to be consistent with Geode's established

[GitHub] geode issue #612: GEODE-3121: ensure that the protobuf protocol works over S...

2017-06-30 Thread PurelyApplied
Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/612 If you are using IntelliJ as your IDE, please remember to update your style file to that located in `geode/etc/intellij-java-modified-google-style.xml` to be consistent with Geode's established

[GitHub] geode issue #613: GEODE-3151: Internal Region Registration in JMX as per con...

2017-06-30 Thread PurelyApplied
Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/613 If you are using IntelliJ as your IDE, please remember to update your style file to that located in `geode/etc/intellij-java-modified-google-style.xml` to be consistent with Geode's established

[GitHub] geode pull request #584: GEODE-2901: Adding integration tests of session rep...

2017-06-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/584 --- 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 #603: GEODE-3128: Changed the batch size to 1000 when cre...

2017-06-30 Thread upthewaterspout
Github user upthewaterspout commented on a diff in the pull request: https://github.com/apache/geode/pull/603#discussion_r125094490 --- Diff: geode-lucene/src/test/java/org/apache/geode/cache/lucene/LuceneQueriesIntegrationTest.java --- @@ -59,7 +64,7 @@ @Rule

Re: Review Request 60570: GEODE-3153 Client receives duplicate events during rolling upgrade

2017-06-30 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60570/#review179390 ---

Re: Review Request 60312: GEODE-2804 Cache InetAddress if configure host as ip string.

2017-06-30 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60312/#review179382 --- Ship it! Ship It! - Bruce Schuchardt On June 29, 2017, 3:27

Re: Review Request 60570: GEODE-3153 Client receives duplicate events during rolling upgrade

2017-06-30 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60570/ --- (Updated June 30, 2017, 9:33 a.m.) Review request for geode, Alexander

Build failed in Jenkins: Geode-nightly #882

2017-06-30 Thread Apache Jenkins Server
See Changes: [upthewaterspout] GEODE-393: Providing cache for FunctionContext [upthewaterspout] GEODE-2018: Added instantiators to SessionCachingFilter init method [upthewaterspout] GEODE-3140: Removed

Review Request 60570: GEODE-3072 Events do not get removed from the client queue when 1.0 clients connect to 1.2 servers

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

Re: Review Request 60550: GEODE-3154: add geode-protobuf to expected_jars.txt

2017-06-30 Thread Brian Rowe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60550/#review179374 --- Ship it! Ship It! - Brian Rowe On June 29, 2017, 9:39 p.m.,