Re: Question regarding node failure scenario

2017-07-27 Thread Akihiro Kitada
Hello Roi, >1. At first, it did seem that adding the parameter subscription-redundancy="1" to the client cache xml was doing the trick. >(I did not understand however what is the meaning of "1" as the value - should it be the number of servers here - meaning if there are >2 servers the value shoul

[GitHub] geode pull request #663: GEODE-3314: Fix DLockService token leak.

2017-07-27 Thread galen-pivotal
GitHub user galen-pivotal opened a pull request: https://github.com/apache/geode/pull/663 GEODE-3314: Fix DLockService token leak. And add a test so that we don't do that again. We had been unlocking the DLock on the remote server before freeing the local `DLockToken`. T

Re: Tomcat session tests are failing in nightly build

2017-07-27 Thread David Anuta
I took a look at the failures and I think it may be an environment problem with Jenkins. We haven't made any changes to the module involved with these tests in the past couple weeks and the ones in question write some stuff to the tmp dir that may have somehow gotten corrupted. I've merged a change

[GitHub] geode pull request #507: GEODE-231 : Remove deprecated AttributesMutator.set...

2017-07-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/507 --- 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 #662: GEODE-3301: Installations now cleanup previously le...

2017-07-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/662 --- 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: GMSJoinLeaveJUnitTest failing on develop

2017-07-27 Thread Patrick Rhomberg
I don't see a JIRA for it, but it's been failing sporadically for me on my precheckins. On Tue, Jul 25, 2017 at 10:41 AM, Kirk Lund wrote: > Is someone working on this test? It fails consistently for me on develop... > > ./gradlew -DintegrationTest.single=GMSJoinLeaveJUnitTest > geode-core:integ

Re: Review Request 61166: GEODE-3313: Test utility supports building jar files with multiple classes

2017-07-27 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61166/ --- (Updated July 27, 2017, 11:29 p.m.) Review request for geode, Emily Yeh, Jared

Re: Review Request 61166: GEODE-3313: Test utility supports building jar files with multiple classes

2017-07-27 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61166/ --- (Updated July 27, 2017, 11:15 p.m.) Review request for geode, Emily Yeh, Jared

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

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

[GitHub] geode issue #662: GEODE-3301: Installations now cleanup previously left over...

2017-07-27 Thread DivineEnder
Github user DivineEnder commented on the issue: https://github.com/apache/geode/pull/662 @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 proj

[GitHub] geode pull request #662: GEODE-3301: Installations now cleanup previously le...

2017-07-27 Thread DivineEnder
GitHub user DivineEnder opened a pull request: https://github.com/apache/geode/pull/662 GEODE-3301: Installations now cleanup previously left over directories… This should hopefully fix Jenkins build issues Thank you for submitting a contribution to Apache Geode.

Review Request 61196: GEODE-3326: Fix intermittent ConcurrentDeployDUnitTest failure

2017-07-27 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61196/ --- Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and Pat

Re: Apache Geode Native Client 32 bit build and OQL question

2017-07-27 Thread Jacob Barrett
On Thu, Jul 27, 2017 at 2:00 PM Rupert St John Webster < rup...@impress-solutions.com> wrote: > I have a requirement for a 32bit version of a strongly named Geode native > dll targeted at .Net framework 3.5 so it can be used by an application > built in VB6 😊 > As I answered on the stackoverflow

Re: Apache Geode Native Client 32 bit build and OQL question

2017-07-27 Thread Anthony Baker
One approach is to use Continuous Queries (CQ) to receive push notifications for all changes matching a predicate. > On Jul 27, 2017, at 9:34 AM, Rupert St John Webster > wrote: > > I also have a quick question about OQL -> how to get the most recent update > into a region having many updates

Re: Apache Geode Native Client 32 bit build and OQL question

2017-07-27 Thread Anilkumar Gingade
>> I also have a quick question about OQL -> how to get the most recent update into a region having many updates? Is it something like: You need to have timestamp associated with your entry/value; and use that in your query to get the last update. -Anil. On Thu, Jul 27, 2017 at 9:34 AM, Rupert S

Re: Review Request 61185: GEODE-3231: use tempWorkingFolder to avoid test log file contamination between tests.

2017-07-27 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61185/#review181613 --- Ship it! Ship It! - Jared Stewart On July 27, 2017, 4:57 p.m

[GitHub] geode issue #657: GEODE-3286: Failing to cleanup connections from Connection...

2017-07-27 Thread WireBaron
Github user WireBaron commented on the issue: https://github.com/apache/geode/pull/657 Pushed a new revision tightening the check before adding a connection to the connectionTable's receivers. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] geode pull request #657: GEODE-3286: Failing to cleanup connections from Con...

2017-07-27 Thread WireBaron
Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/657#discussion_r129961462 --- Diff: geode-core/src/main/java/org/apache/geode/internal/tcp/Connection.java --- @@ -1322,6 +1328,14 @@ private void createBatchSendBuffer() {

[GitHub] geode pull request #657: GEODE-3286: Failing to cleanup connections from Con...

2017-07-27 Thread WireBaron
Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/657#discussion_r129961225 --- Diff: geode-core/src/main/java/org/apache/geode/internal/tcp/ConnectionTable.java --- @@ -279,26 +280,29 @@ protected void acceptConnection(Socket sock)

Apache Geode Native Client 32 bit build and OQL question

2017-07-27 Thread Rupert St John Webster
Hi, I have a requirement for a 32bit version of a strongly named Geode native dll targeted at .Net framework 3.5 so it can be used by an application built in VB6 😊 At the moment I’m trying to build the Geode native client any way at all and come across errors detailed in https://stackoverflow

[GitHub] geode pull request #661: GEODE-3319 - refactor to use protobuf encoding for ...

2017-07-27 Thread WireBaron
GitHub user WireBaron opened a pull request: https://github.com/apache/geode/pull/661 GEODE-3319 - refactor to use protobuf encoding for primitive types @pivotal-amurmann @galen-pivotal @hiteshk25 @kohlmu-pivotal @bschuchardt This changes protobuf EncodedValue messages to s

[GitHub] geode issue #649: GEODE-2997: Change new protocol GetAllResponse.

2017-07-27 Thread galen-pivotal
Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/649 This isn't code complete; I'm planning to work on this when I have the time (unless anyone else wants to). --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] geode pull request #660: GEODE-2924 Revise authorization permissions

2017-07-27 Thread karensmolermiller
Github user karensmolermiller commented on a diff in the pull request: https://github.com/apache/geode/pull/660#discussion_r129917470 --- Diff: geode-docs/managing/security/implementing_authorization.html.md.erb --- @@ -56,13 +56,23 @@ which classifies whether the operation as

[GitHub] geode issue #660: GEODE-2924 Revise authorization permissions

2017-07-27 Thread joeymcallister
Github user joeymcallister commented on the issue: https://github.com/apache/geode/pull/660 +1 with JinMei's changes. --- 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 #660: GEODE-2924 Revise authorization permissions

2017-07-27 Thread jinmeiliao
Github user jinmeiliao commented on a diff in the pull request: https://github.com/apache/geode/pull/660#discussion_r129910320 --- Diff: geode-docs/managing/security/implementing_authorization.html.md.erb --- @@ -56,13 +56,23 @@ which classifies whether the operation as The op

[GitHub] geode pull request #660: GEODE-2924 Revise authorization permissions

2017-07-27 Thread jinmeiliao
Github user jinmeiliao commented on a diff in the pull request: https://github.com/apache/geode/pull/660#discussion_r129908926 --- Diff: geode-docs/managing/security/implementing_authorization.html.md.erb --- @@ -56,13 +56,23 @@ which classifies whether the operation as The op

[GitHub] geode pull request #660: GEODE-2924 Revise authorization permissions

2017-07-27 Thread karensmolermiller
GitHub user karensmolermiller opened a pull request: https://github.com/apache/geode/pull/660 GEODE-2924 Revise authorization permissions For finer-grained security, identify new targets for CLUSTER operations (DISK, GATEWAY, QUERY, JAR). And, revise 4 operations listed in th

Re: Review Request 61143: GEODE-3310 Set target node in TXStateProxy during TXFailover if necessary

2017-07-27 Thread Nick Reich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61143/#review181588 --- Ship it! Ship It! - Nick Reich On July 27, 2017, 5:32 p.m.,

Re: Tomcat session tests are failing in nightly build

2017-07-27 Thread Lynn Hughes-Godfrey
There are two related open issues: https://issues.apache.org/jira/browse/GEODE-3301: Cargo Module tests failing in nightly build with Unable to edit XML file https://issues.apache.org/jira/browse/GEODE-3303: Cargo Module tests failing with IOException Unfortunately, this is specific to the Jenkins

Re: Review Request 61143: GEODE-3310 Set target node in TXStateProxy during TXFailover if necessary

2017-07-27 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61143/#review181585 --- geode-core/src/test/java/org/apache/geode/internal/cache/execute/

Re: Review Request 61143: GEODE-3310 Set target node in TXStateProxy during TXFailover if necessary

2017-07-27 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61143/ --- (Updated July 27, 2017, 5:32 p.m.) Review request for geode, anilkumar gingade,

Review Request 61185: GEODE-3231: use tempWorkingFolder to avoid test log file contamination between tests.

2017-07-27 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61185/ --- Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and Pat

Re: Tomcat session tests are failing in nightly build

2017-07-27 Thread Udo Kohlmeyer
Kirk, it seems this could be related to something that was checked in recently. The potentially offending committer is investigating it right now. On 7/27/17 09:29, Kirk Lund wrote: Anyone have any ideas why the Tomcat session tests are failing in every nightly build? classMethod – org.apache

[GitHub] geode pull request #657: GEODE-3286: Failing to cleanup connections from Con...

2017-07-27 Thread galen-pivotal
Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/657#discussion_r129892735 --- Diff: geode-core/src/main/java/org/apache/geode/internal/tcp/ConnectionTable.java --- @@ -279,26 +280,29 @@ protected void acceptConnection(Socket s

Tomcat session tests are failing in nightly build

2017-07-27 Thread Kirk Lund
Anyone have any ideas why the Tomcat session tests are failing in every nightly build? classMethod – org.apache.geode.session.tests.Tomcat6ClientServerTest a few seconds classMethod – org.apache.geode.session.tests.Tomcat6Test a few seconds classMethod – org.apache.geode.session.tests.Tomcat7Clien

Build failed in Jenkins: Geode-nightly #908

2017-07-27 Thread Apache Jenkins Server
See Changes: [gzhou] GEODE-3291: add soundex analyzer into lucene query This closes #654 [jstewart] GEODE-3296: Speed up Acceptance Tests [kmiller] GEODE-3312 Update doc gfsh list members output [lgallinat] GEODE-

RE: Question regarding node failure scenario

2017-07-27 Thread Roi Apelker
Hi, and thanks/Arigato, 1. At first, it did seem that adding the parameter subscription-redundancy="1" to the client cache xml was doing the trick. (I did not understand however what is the meaning of "1" as the value - should it be the number of servers here - meaning if there are 2 servers the

Re: JIRA Assignment Permissions Request

2017-07-27 Thread Juan José Ramos
Thanks! On Thu, Jul 27, 2017 at 4:42 PM, Anthony Baker wrote: > done > > > On Jul 26, 2017, at 12:07 PM, Juan José Ramos wrote: > > > > Hello Anthony, > > > > I forgot to include that, sorry. My username is jujoramos. > > Cheers. > > > > On Wed 26 Jul 2017 at 18:25, Anthony Baker wrote: > > >

[GitHub] geode pull request #657: GEODE-3286: Failing to cleanup connections from Con...

2017-07-27 Thread kohlmu-pivotal
Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/657#discussion_r129879726 --- Diff: geode-core/src/main/java/org/apache/geode/internal/tcp/Connection.java --- @@ -1322,6 +1328,14 @@ private void createBatchSendBuffer() {

Re: JIRA Assignment Permissions Request

2017-07-27 Thread Anthony Baker
done > On Jul 26, 2017, at 12:07 PM, Juan José Ramos wrote: > > Hello Anthony, > > I forgot to include that, sorry. My username is jujoramos. > Cheers. > > On Wed 26 Jul 2017 at 18:25, Anthony Baker wrote: > >> Welcome! What is your JIRA username? >> >> Anthony >> >>> On Jul 26, 2017, at

[GitHub] geode pull request #643: GEODE-3192,GEODE-3229: Change API and implementatio...

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