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

See

Build failed in Jenkins: Geode-nightly #889

See Changes: [klund] GEODE-3117: fix Gateway authentication with legacy security -- [...truncated 71.95 KB...] :geode-cq:processResources :geode-cq:classes

Fixed: apache/geode#3111 (develop - 3b90f9f)

Build Update for apache/geode - Build: #3111 Status: Fixed Duration: 8 minutes and 59 seconds Commit: 3b90f9f (develop) Author: Anil Message: GEODE-3105: Fixing spotlessJavaCheck with PersistentPartitionedRegionTestBase.java View the changeset:

Fixed: apache/geode#3109 (develop - 1537847)

Build Update for apache/geode - Build: #3109 Status: Fixed Duration: 10 minutes and 22 seconds Commit: 1537847 (develop) Author: Hitesh Khamesra Message: GEODE-3153 applied spotless View the changeset:

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60718/#review179967 ---

Failed: jinmeiliao/geode#9 (refactor-week2-friday - 9d204a9)

Build Update for jinmeiliao/geode - Build: #9 Status: Failed Duration: 11 minutes and 44 seconds Commit: 9d204a9 (refactor-week2-friday) Author: Jinmei Liao Message: have SimpleHttpOperationInvoker handle file download View the changeset:

Re: Review Request 60710: GEODE-3015: fix a flaky unit test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60710/#review179966 --- Ship it! Ship It! - Darrel Schneider On July 7, 2017, 4:08

[GitHub] geode-native issue #102: GEODE-2741: Adding a warning for 64bit Windows Tool...

Github user mhansonp commented on the issue: https://github.com/apache/geode-native/pull/102 Sorry, I will fix that shortly. I thought that went to my fork only…. > On Jul 7, 2017, at 3:58 PM, Jacob Barrett wrote: > > @mhansonp

[GitHub] geode-native pull request #110: GEODE-3135 Update Geode Client docs OpenSSL ...

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

Re: Review Request 60710: GEODE-3015: fix a flaky unit test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60710/ --- (Updated July 7, 2017, 11:08 p.m.) Review request for geode, anilkumar

[GitHub] geode-native issue #102: GEODE-2741: Adding a warning for 64bit Windows Tool...

Github user pivotal-jbarrett commented on the issue: https://github.com/apache/geode-native/pull/102 @mhansonp you have conflicts, can you rebase and force push. --- 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

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

--- Spring Data GemFire > Nightly-ApacheGeode > #609 was successful. --- Scheduled 1910 tests in total. https://build.spring.io/browse/SGF-NAG-609/ -- This

Broken: apache/geode#3106 (rollback/1.2.0_memberID - 57fdb34)

Build Update for apache/geode - Build: #3106 Status: Broken Duration: 9 minutes and 12 seconds Commit: 57fdb34 (rollback/1.2.0_memberID) Author: Bruce Schuchardt Message: Revert serialized form of InternalDistributedMember to 1.0.0-incubating This reopens

[GitHub] geode issue #621: GEODE-3129 - Added error messages to protobuf protocol

Github user scmbuildguy commented on the issue: https://github.com/apache/geode/pull/621 Thanks! It was not clear from the history of the pull request. --Mark --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

Re: Review Request 60526: GEODE-3121: Verify SSL works with new protobuf protocol

> On July 7, 2017, 8:58 p.m., Udo Kohlmeyer wrote: > > geode-core/src/main/java/org/apache/geode/internal/net/SocketCreator.java > > Line 267 (original), 267 (patched) > > > > > > You should maybe use the

Re: Review Request 60526: GEODE-3121: Verify SSL works with new protobuf protocol

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60526/ --- (Updated July 7, 2017, 10:16 p.m.) Review request for geode, Alexander

[GitHub] geode-native pull request #110: GEODE-3135 Update Geode Client docs OpenSSL ...

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/110#discussion_r126254300 --- Diff: docs/geode-native-docs/introduction/client-configurations.html.md.erb --- @@ -157,7 +157,7 @@ The following libraries are external

[GitHub] geode-native pull request #110: GEODE-3135 Update Geode Client docs OpenSSL ...

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/110#discussion_r126254216 --- Diff: docs/geode-native-docs/introduction/client-configurations.html.md.erb --- @@ -157,7 +157,7 @@ The following libraries are external

[GitHub] geode issue #621: GEODE-3129 - Added error messages to protobuf protocol

Github user kohlmu-pivotal commented on the issue: https://github.com/apache/geode/pull/621 Yes, there was a PR related to this, but it was deleted and then this one created. In addition, there is/was a review in apache reviewboard since Jul,3 --- If your project is set up for it,

[GitHub] geode issue #621: GEODE-3129 - Added error messages to protobuf protocol

Github user scmbuildguy commented on the issue: https://github.com/apache/geode/pull/621 This was opened for about 90 minutes before it was merged into develop. Was there a review done by a committer on this code? --- If your project is set up for it, you can reply to this email

[GitHub] geode pull request #620: GEODE-3172: Fix serialization errors with client qu...

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

Passed: apache/geode#3101 (rollback/1.2.0_memberID - 40fdc5d)

Build Update for apache/geode - Build: #3101 Status: Passed Duration: 10 minutes and 7 seconds Commit: 40fdc5d (rollback/1.2.0_memberID) Author: Bruce Schuchardt Message: Revert "Revert "GEODE-3139 remove current artifacts from classpath of

Review Request 60718: GEODE-2997: New flow getAll/putAll

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60718/ --- Review request for geode, Alexander Murmann, Bruce Schuchardt, Galen

Re: Review Request 60710: GEODE-3015: fix a flaky unit test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60710/#review179944 ---

Re: Review Request 60526: GEODE-3121: Verify SSL works with new protobuf protocol

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60526/#review179943 ---

Re: Review Request 60526: GEODE-3121: Verify SSL works with new protobuf protocol

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60526/ --- (Updated July 7, 2017, 8:54 p.m.) Review request for geode, Alexander Murmann,

[GitHub] geode pull request #621: GEODE-3129 - Added error messages to protobuf proto...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/621 --- 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 60486: GEODE-3105: Adding handler for GetRegions

> On June 28, 2017, 3:53 a.m., Udo Kohlmeyer wrote: > > geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/ProtobufOpsProcessor.java > > Line 50 (original), 50 (patched) > > > > > > Feels like this is

Re: Review Request 60451: GEODE-2996: adding Put handler

> On June 28, 2017, 4:24 a.m., Udo Kohlmeyer wrote: > > geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/ProtobufOpsProcessor.java > > Lines 68 (patched) > > > > > > This has nothing to do with the

Re: Review Request 60629: GEODE-3129 - Added error messages to protobuf protocol

> On July 6, 2017, 8:45 p.m., Udo Kohlmeyer wrote: > > geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/ProtobufStreamProcessor.java > > Line 67 (original), 68 (patched) > > > > > > Why do we have

Re: Review Request 60526: GEODE-3121: Verify SSL works with new protobuf protocol

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60526/#review179916 --- Ship it! I'd prefer this SSL test to be really part of the

[GitHub] geode pull request #621: GEODE-3129 - Added error messages to protobuf proto...

GitHub user WireBaron opened a pull request: https://github.com/apache/geode/pull/621 GEODE-3129 - Added error messages to protobuf protocol added a new ErrorResponse type to ClientProtocol removed success field from several RegionAPI response objects and refactored operation

[GitHub] geode issue #620: GEODE-3172: Fix serialization errors with client queues fr...

Github user upthewaterspout commented on the issue: https://github.com/apache/geode/pull/620 After reviewing these changes with @jhuynh1 - it looks like the HAEventWrapper will never get serialized - we don't configure eviction on the regions that hold HAEventWrapper. So these

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

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

[GitHub] geode pull request #617: GEODE-3129 - Added error messages to protobuf proto...

Github user WireBaron closed the pull request at: https://github.com/apache/geode/pull/617 --- 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-native pull request #110: GEODE-3135 Update Geode Client docs OpenSSL ...

GitHub user karensmolermiller opened a pull request: https://github.com/apache/geode-native/pull/110 GEODE-3135 Update Geode Client docs OpenSSL version @pivotal-jbarrett @joeymcallister @davebarnes97 Please review. I have reduced the locations which call out which version of

[GitHub] geode-native pull request #108: GEODE-3135 Update Geode Client docs OpenSSL ...

Github user karensmolermiller closed the pull request at: https://github.com/apache/geode-native/pull/108 --- 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

[GitHub] geode-native issue #108: GEODE-3135 Update Geode Client docs OpenSSL version

Github user karensmolermiller commented on the issue: https://github.com/apache/geode-native/pull/108 Closing this PR. Will address issues, and open a new PR. --- 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

Review Request 60710: GEODE-3015: fix a flaky unit test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60710/ --- Review request for geode, anilkumar gingade, Darrel Schneider, and Lynn

[GitHub] geode issue #620: GEODE-3172: Fix serialization errors with client queues fr...

Github user upthewaterspout commented on the issue: https://github.com/apache/geode/pull/620 Head's up - I'm not sure my changes to fix 1.2->1.0 copying of the queue are going to completely work. If the HAEventWrapper gets serialized for some reason (maybe evicted to disk?) Then I

[GitHub] geode-native pull request #107: GEODE-3019: Refactor Struct class

Github user dgkimura commented on a diff in the pull request: https://github.com/apache/geode-native/pull/107#discussion_r126173633 --- Diff: src/cppcache/include/geode/Struct.hpp --- @@ -138,7 +138,7 @@ class CPPCACHE_EXPORT Struct : public Serializable { * Returns the

RE: Query for JMX "Async Event Queue properties"

Hi Team, I tried to execute select sql on Internal region associated the Async Queue using command “gfsh>query --query=’select * from /AsyncEventQueue_XYZ_PARALLEL_GATEWAY_SENDER_QUEUE’ But in geode 1.1.1 There is a validation check in below method . we always try to find out the

Build failed in Jenkins: Geode-release-flaky #24

See -- [...truncated 2.47 KB...] at hudson.remoting.Channel.attachCallSiteStackTrace(Channel.java:1545) at hudson.remoting.UserResponse.retrieve(UserRequest.java:253)

Build failed in Jenkins: Geode-release #75

See -- [...truncated 178.66 KB...] at org.apache.geode.cache.CacheFactory.create(CacheFactory.java:166) at

[GitHub] geode-native pull request #102: GEODE-2741: Adding a warning for 64bit Windo...

Github user vjr commented on a diff in the pull request: https://github.com/apache/geode-native/pull/102#discussion_r126127440 --- Diff: src/CMakeLists.txt --- @@ -17,6 +17,15 @@ project(nativeclient) list(APPEND CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/../cmake)

[GitHub] geode-native issue #102: GEODE-2741: Adding a warning for 64bit Windows Tool...

Github user pivotal-jbarrett commented on the issue: https://github.com/apache/geode-native/pull/102 @mhansonp please follow up or close this pull request. --- 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

[GitHub] geode-native pull request #107: GEODE-3019: Refactor Struct class

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/107#discussion_r126116586 --- Diff: src/cppcache/integration-test/testThinClientRemoteQuerySS.cpp --- @@ -56,59 +57,63 @@ const wchar_t* checkNullString(const wchar_t*

[GitHub] geode-native issue #105: GEODE-2891 connect-timeout violation in C++ Native ...

Github user gregt5259 commented on the issue: https://github.com/apache/geode-native/pull/105 Closed by Ernie Burghardt request as duplicated #106 --- 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

[GitHub] geode-native pull request #105: GEODE-2891 connect-timeout violation in C++ ...

Github user gregt5259 closed the pull request at: https://github.com/apache/geode-native/pull/105 --- 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

[GitHub] geode-native pull request #107: GEODE-3019: Refactor Struct class

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/107#discussion_r126115701 --- Diff: src/cppcache/include/geode/Struct.hpp --- @@ -138,7 +138,7 @@ class CPPCACHE_EXPORT Struct : public Serializable { * Returns

[GitHub] geode-native pull request #108: GEODE-3135 Update Geode Client docs OpenSSL ...

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/108#discussion_r126114762 --- Diff: docs/geode-native-docs/security/sslclientserver.html.md.erb --- @@ -42,17 +42,17 @@ To install OpenSSL: 2. Extract the archive

[GitHub] geode-native pull request #108: GEODE-3135 Update Geode Client docs OpenSSL ...

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/108#discussion_r126113664 --- Diff: docs/geode-native-docs/introduction/client-configurations.html.md.erb --- @@ -157,7 +157,7 @@ The following libraries are external

[GitHub] geode-native pull request #108: GEODE-3135 Update Geode Client docs OpenSSL ...

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/108#discussion_r126114295 --- Diff: docs/geode-native-docs/introduction/quickstart.html.md.erb --- @@ -53,7 +53,7 @@ for the latest Java version for your operating

[GitHub] geode-native pull request #108: GEODE-3135 Update Geode Client docs OpenSSL ...

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/108#discussion_r126114552 --- Diff: docs/geode-native-docs/security/sslclientserver.html.md.erb --- @@ -27,7 +27,7 @@ The open-source OpenSSL toolkit provides a