Re: geode-3153 waiting on reviews

2017-07-03 Thread Mark Bretl
Hi Barry, What Geode versions are these related to? Please only use Geode versions when discussing issues on the community lists. Thanks, --Mark On Mon, Jul 3, 2017 at 6:26 PM Barry Oglesby wrote: > Bruce, > > Here is what I am seeing in my test (same one I sent you). > > 904 server -> 823 c

Fixed: apache/geode#3082 (release/1.2.0 - 00a066b)

2017-07-03 Thread Travis CI
Build Update for apache/geode - Build: #3082 Status: Fixed Duration: 11 minutes and 23 seconds Commit: 00a066b (release/1.2.0) Author: Dan Smith Message: GEODE-3139: Fixing compilation errors ProcessManager appears to have been screwed up in the merge from dev

Broken: apache/geode#3078 (release/1.2.0 - 30e2eb2)

2017-07-03 Thread Travis CI
Build Update for apache/geode - Build: #3078 Status: Broken Duration: 5 minutes and 32 seconds Commit: 30e2eb2 (release/1.2.0) Author: Bruce Schuchardt Message: GEODE-3139 remove artifacts from classpath of backward-compatibility tests reinstating this - pass

Broken: apache/geode#3077 (develop - fb4e96b)

2017-07-03 Thread Travis CI
Build Update for apache/geode - Build: #3077 Status: Broken Duration: 9 minutes and 25 seconds Commit: fb4e96b (develop) Author: Bruce Schuchardt Message: GEODE-3139 remove artifacts from classpath of backward-compatibility tests reinstating this - passes pre

Fixed: apache/geode#3079 (develop - 3433b3e)

2017-07-03 Thread Travis CI
Build Update for apache/geode - Build: #3079 Status: Fixed Duration: 21 minutes and 54 seconds Commit: 3433b3e (develop) Author: nabarun Message: GEODE-3139: Spotless error fix View the changeset: https://github.com/apache/geode/compare/fb4e96bfbad9...3433b3e

Re: geode-3153 waiting on reviews

2017-07-03 Thread Barry Oglesby
Bruce, Here is what I am seeing in my test (same one I sent you). 904 server -> 823 client: Before writing to stream: Client Message Dispatcher for 192.168.2.4(client:78763:loner):49721:b274160b:client(version:GFE 8.1): ClientUpdateMessageImpl.getGFE70Message membershipIdLength=39; membershipId=

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

2017-07-03 Thread WireBaron
GitHub user WireBaron opened a pull request: https://github.com/apache/geode/pull/617 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 ha

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

2017-07-03 Thread Brian Rowe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60629/ --- Review request for geode, Alexander Murmann, Bruce Schuchardt, Galen O'Sullivan,

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

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

Re: Build failed in Jenkins: Geode-release #72

2017-07-03 Thread Dan Smith
Looks like compilation of the release branch was broken by these changes. I went ahead and fixed the compilation error since we are trying to get this release out - I just made ProcessManager match the contents on develop, which I think was the intention. commit 30e2eb2080afff3af2f5226a412259c3a53

[GitHub] geode pull request #616: GEODE-2860: Refactor use of EventTracker

2017-07-03 Thread nreich
GitHub user nreich opened a pull request: https://github.com/apache/geode/pull/616 GEODE-2860: Refactor use of EventTracker * change EventTracker to an interface with two implementations * move as much logic out of LocalRegion down into subclasses that make use EventT

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

2017-07-03 Thread Apache Jenkins Server
See Changes: [bschuchardt] GEODE-3139 remove artifacts from classpath of backward-compatibility -- [...truncated 73.70 KB...] :geode-core:javadoc :extensions/geode-modul

Build failed in Jenkins: Geode-release #72

2017-07-03 Thread Apache Jenkins Server
See Changes: [bschuchardt] GEODE-3139 remove artifacts from classpath of backward-compatibility -- [...truncated 68.57 KB...] :geode-lucene:processResources :geode-lucene:clas

Re: geode-3153 waiting on reviews

2017-07-03 Thread Bruce Schuchardt
I've checked in the fix for geode-3153, though Barry said he may still be seeing trouble with it. I plan to get together with him on-line wednesday morning, if possible, to step through his test & verify if the problem still exists. I've also pushed the commit to release/1.2.0 On 7/1/17 8:5

Re: Apache Geode + Narayana JTA = Global Transactions with Last-Resource Optimization

2017-07-03 Thread Swapnil Bawaskar
Great work Christian! Can you add a page about this on the wiki as well, so that your work is not lost. On Mon, Jul 3, 2017 at 10:58 AM Dan Smith wrote: > Awesome! > > -Dan > > On Mon, Jul 3, 2017 at 6:08 AM, Christian Tzolov > wrote: >

Apache Nightly build machine disk space usage

2017-07-03 Thread Nabarun Nag
Hello all, Build 54 has a failed test where the machine has reached 92% disk usage *Failed Test:* testStartServerAndExecuteCommands – org.apache.geode.management.internal.configuratio

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

2017-07-03 Thread Apache Jenkins Server
See -- [...truncated 98.99 KB...] Download https://repo1.maven.org/maven2/org/springframework/plugin/spring-plugin-core/1.2.0.RELEASE/spring-plugin-core-1.2.0.RELEASE.pom Download http

Re: Apache Geode + Narayana JTA = Global Transactions with Last-Resource Optimization

2017-07-03 Thread Dan Smith
Awesome! -Dan On Mon, Jul 3, 2017 at 6:08 AM, Christian Tzolov wrote: > Recent experience of setting up Geode as a last-resource inside WebSphere > JTA, has motivated me to find a simpler way to integrate with JTA. > > The https://github.com/tzolov/narayana-jta-geode-support provides > utilitie

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

2017-07-03 Thread Dan Smith
> On July 3, 2017, 4:02 p.m., Dan Smith wrote: > > geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/membership/GMSJoinLeave.java > > Lines 129 (patched) > > > > > > I don't see this prope

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

2017-07-03 Thread Bruce Schuchardt
> On July 3, 2017, 8:29 a.m., Alexander Murmann wrote: > > geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/ClientServerMiscBCDUnitTest.java > > Lines 131 (patched) > > > > > > Nit pick: Do our t

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

2017-07-03 Thread Bruce Schuchardt
> On July 3, 2017, 9:02 a.m., Dan Smith wrote: > > It looks like this code should work. I'm still a bit worried that we are > > relying on code that tries to deserialize the member id with the wrong > > version and then ignores errors that occur. I'd like to see us revisit > > propegating the

[GitHub] geode pull request #615: GEODE-2991 Add man page for gfsh destroy gateway-se...

2017-07-03 Thread karensmolermiller
GitHub user karensmolermiller opened a pull request: https://github.com/apache/geode/pull/615 GEODE-2991 Add man page for gfsh destroy gateway-sender @boglesby @joeymcallister @davebarnes97 @upthewaterspout Please review. You can merge this pull request into a Git repository b

Build failed in Jenkins: Geode-nightly #885

2017-07-03 Thread Apache Jenkins Server
See -- [...truncated 115.00 KB...] :geode-json:compileTestJava UP-TO-DATE :geode-json:processTestResources :geode-json:testClasses :geode-json:checkMissedTests UP-TO-DATE :geode-json:spotless

Re: Question regarding node failure scenario

2017-07-03 Thread Dan Smith
How are you performing the updates? Is N1 directly making updates or are they coming from some other source? What is the subscription-redundancy of your client that has registered continuous queries? -Dan On Mon, Jul 3, 2017 at 8:19 AM, Roi Apelker wrote: > Hi, > > I have the following scenari

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

2017-07-03 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60570/#review179513 --- Fix it, then Ship it! It looks like this code should work. I'm

WFH Today (7/3)

2017-07-03 Thread Dan Smith
I'll be working from home today. -Dan

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

2017-07-03 Thread Alexander Murmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60570/#review179512 --- Fix it, then Ship it! Ship It! geode-core/src/test/java/org/a

Question regarding node failure scenario

2017-07-03 Thread Roi Apelker
Hi, I have the following scenario, I wonder if anyone can comment on it - is it a known issue, maybe it was solved already in later version, etc. (I am using version 1.0.0) Bear with me I am a bit new here :) I have a replicated Node, say N1 and its replicated N2 (N2 gets activated when N1 is

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

2017-07-03 Thread pivotal-jbarrett
Github user pivotal-jbarrett commented on the issue: https://github.com/apache/geode-native/pull/105 > I suggest to fix the issue by defining handshake pseudo message within the range probably defined for such pseudo messages by original design i.e.: > typedef enum { >/* S

Apache Geode + Narayana JTA = Global Transactions with Last-Resource Optimization

2017-07-03 Thread Christian Tzolov
Recent experience of setting up Geode as a last-resource inside WebSphere JTA, has motivated me to find a simpler way to integrate with JTA. The https://github.com/tzolov/narayana-jta-geode-support provides utilities to integrate Geode with Narayana JTA. This enables global transactions using the

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

2017-07-03 Thread dineshpune2006
Github user dineshpune2006 commented on the issue: https://github.com/apache/geode/pull/613 i have added etc/eclipseFormatterProfile.xml in eclipse profile & did the code style set. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

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

2017-07-03 Thread gregt5259
Github user gregt5259 commented on the issue: https://github.com/apache/geode-native/pull/105 I suggest to fix the issue by defining handshake pseudo message within the range probably defined for such pseudo messages by original design i.e.: typedef enum { /* Server