RE: DISCUSS : Monitor the neighbour JVM using neihbour's member-timeout (GEODE-3411)

2017-09-01 Thread Aravind Musigumpula
Hi Brian, This will help if the user has some member doing a heavy duty when compared to others, in this case we need to give such member some extra time to that member. Thanks, Aravind Musigumpula -Original Message- From: Brian Baynes [mailto:bbay...@pivotal.io] Sent: Friday, Septem

[VOTE] Apache Geode release - v1.2.1 RC2

2017-09-01 Thread Anthony Baker
This is the second release candidate for Apache Geode, version 1.2.1. Thanks to all the community members for their contributions to this release! *** Please download, test and vote by Thursday, September 7, 0800 hrs US Pacific. *** It fixes the following issues: https://issues.apache.org/jira

Re: Gitbox enables the full GitHub workflow

2017-09-01 Thread Anthony Baker
See https://issues.apache.org/jira/browse/INFRA-15001 . Anthony > On Aug 22, 2017, at 10:43 AM, Anthony Baker wrote: > > >> On Aug 7, 2017, at 6:09 PM, Roman Shaposhnik wrote: >> >> Hi! >> >> it has just come to my attention that Gitbox at

Re: Review Request 61978: GEODE-3059: LoadMonitor.connectionClosed incrementing statistics only for client-server connection

2017-09-01 Thread Bruce Schuchardt
> On Aug. 29, 2017, 3:48 p.m., Alexander Murmann wrote: > > geode-protobuf/src/test/java/org/apache/geode/protocol/AuthenticationIntegrationTest.java > > Lines 105 (patched) > > > > > > We are doing this for the Roun

Re: DISCUSS : Monitor the neighbour JVM using neihbour's member-timeout (GEODE-3411)

2017-09-01 Thread Udo Kohlmeyer
Hi there Aravind, I have a singular problem with this approach. If a some members are designated to do more work, and don't have time to respond to the cluster that they are alive using the current member timeout, then they are not available to accept data. Which means they are not effective

Re: [Discuss] Use of -1 as Infinite/All for retry related functions...

2017-09-01 Thread Mark Hanson
I actually don’t really have a strong opinion one way or the other. I get both approaches. If we want to tailor the code to use a timeout instead of retry attempts I guess that is fine. It seems kind of like we are perpetuating the same API problem, that the LCD approach alleviates, but ok. It

[VOTE] Use of -1 as Infinite/All for retry related functions...

2017-09-01 Thread Jacob Barrett
Remove retry attempts option and use existing timeouts as sole abort mechanism. If you -1 please include constructive reasoning. -Jake > On Sep 1, 2017, at 11:07 AM, Mark Hanson wrote: > > I actually don’t really have a strong opinion one way or the other. I get > both approaches. If we wa

Re: [VOTE] Use of -1 as Infinite/All for retry related functions...

2017-09-01 Thread Bruce Schuchardt
+1 On 9/1/17 11:19 AM, Jacob Barrett wrote: Remove retry attempts option and use existing timeouts as sole abort mechanism. If you -1 please include constructive reasoning. -Jake On Sep 1, 2017, at 11:07 AM, Mark Hanson wrote: I actually don’t really have a strong opinion one way or th

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread jinmeiliao
Github user jinmeiliao commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136637196 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/i18n/CliStrings.java --- @@ -1351,6 +1351,12 @@ "File to which the

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread jinmeiliao
Github user jinmeiliao commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136638633 --- Diff: geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportDataIntegrationTest.java --- @@ -0,0 +1,127 @@ +/* + *

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread jinmeiliao
Github user jinmeiliao commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136638194 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExportDataCommand.java --- @@ -87,4 +80,45 @@ public Result exportDat

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread jinmeiliao
Github user jinmeiliao commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136638437 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ImportDataFunction.java --- @@ -40,9 +40,13 @@ public void execute(F

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread jinmeiliao
Github user jinmeiliao commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136639140 --- Diff: geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ImportDataIntegrationTest.java --- @@ -0,0 +1,158 @@ +/* + *

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread jinmeiliao
Github user jinmeiliao commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136638047 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExportDataCommand.java --- @@ -41,44 +44,34 @@ public Result exportDa

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread agingade
Github user agingade commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136640863 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/snapshot/RegionSnapshotServiceImpl.java --- @@ -342,6 +342,7 @@ private void exportOnMemb

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread nreich
Github user nreich commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136641632 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExportDataCommand.java --- @@ -87,4 +80,45 @@ public Result exportData(

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread nreich
Github user nreich commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136642718 --- Diff: geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ImportDataIntegrationTest.java --- @@ -0,0 +1,158 @@ +/* + * Lice

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread nreich
Github user nreich commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136643154 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/snapshot/RegionSnapshotServiceImpl.java --- @@ -342,6 +342,7 @@ private void exportOnMember

Re: Review Request 61978: GEODE-3059: LoadMonitor.connectionClosed incrementing statistics only for client-server connection

2017-09-01 Thread Galen O'Sullivan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61978/#review184385 --- Fix it, then Ship it! Looks good! Except for the one comparison

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread agingade
Github user agingade commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136644394 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ImportDataFunction.java --- @@ -40,9 +40,13 @@ public void execute(Fun

[GitHub] geode pull request #753: GEODE-3283: Expose parallel import and export in gf...

2017-09-01 Thread agingade
Github user agingade commented on a diff in the pull request: https://github.com/apache/geode/pull/753#discussion_r136644610 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ImportDataFunction.java --- @@ -40,9 +40,13 @@ public void execute(Fun

Re: Review Request 61978: GEODE-3059: LoadMonitor.connectionClosed incrementing statistics only for client-server connection

2017-09-01 Thread Alexander Murmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61978/#review184389 --- Ship it! Ship It! - Alexander Murmann On Aug. 30, 2017, 8:48

[GitHub] geode pull request #754: GEODE-3550: Improve snapshot filter testing

2017-09-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/754 --- 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: Review Request 61978: GEODE-3059: LoadMonitor.connectionClosed incrementing statistics only for client-server connection

2017-09-01 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61978/#review184391 --- Ship it! Ship It! - Hitesh Khamesra On Aug. 30, 2017, 8:48 p

git repo is gone

2017-09-01 Thread Bruce Schuchardt
I'm not able to access the Apache repo anymore git clone https://git-wip-us.apache.org/repos/asf/geode.git Cloning into 'geode'... fatal: repository 'https://git-wip-us.apache.org/repos/asf/geode.git/' not found Is this part of the gitbox migration?

Re: git repo is gone

2017-09-01 Thread Jens Deppe
I believe it is. Just looking to see how/what we're supposed to use now. --Jens On Fri, Sep 1, 2017 at 2:30 PM, Bruce Schuchardt wrote: > I'm not able to access the Apache repo anymore > > git clone https://git-wip-us.apache.org/repos/asf/geode.git > Cloning into 'geode'... > fatal: repository

Re: git repo is gone

2017-09-01 Thread Galen O'Sullivan
It looks like it's moved. If you go to the URL in a browser it will give you a hint. This command should work to update your remote: git remote set-url https://gitbox.apache.org/repos/asf/geode.git running `git remote -v` after should confirm the change. On Fri, Sep 1, 2017 at 2:30 PM, Bruce S

Re: git repo is gone

2017-09-01 Thread Jens Deppe
TL;DR - Remove references to https://git-wip-us.apache.org/repos/asf/geode.git and replace/add ssh:// g...@github.com/apache/geode.git In my environment this is what I had for remotes: $ git remote -v apache https://git-wip-us.apache.org/repos/asf/geode.git (fetch) apache https://git-wip-us.apach

Re: git repo is gone

2017-09-01 Thread Michael William Dodge
Can non-committers use the ssh protocol, too? Sarge > On 1 Sep, 2017, at 14:37, Jens Deppe wrote: > > TL;DR - Remove references to > https://git-wip-us.apache.org/repos/asf/geode.git and replace/add ssh:// > g...@github.com/apache/geode.git > > In my environment this is what I had for remotes:

Re: Geode Clubhouse meeting on the integration of Lucene text search indexes in Geode

2017-09-01 Thread Jagdish Mirani
Hello everybody: This is a quick reminder that we're hosting a Geode Clubhouse meeting on Tuesday, Sept 5th, at 10AM PST. We'll be covering the text search feature in Geode 1.2.0. Here's the meeting link: https://pivotal.zoom.us/j/341598449

Re: [Discuss] Use of -1 as Infinite/All for retry related functions...

2017-09-01 Thread Michael Stolz
We would certainly rather have the time-out set correctly, but one of the things I've noticed is, sometimes there is just one query or one function that takes a really long time, and because we keep retrying it with the same timeout, it keeps timing out each retry. It would probably be much smarter

Re: git repo is gone

2017-09-01 Thread Bruce Schuchardt
So we go directly to github now?  Not gitbox.apache.org? I'm able to clone this: https://gitbox.apache.org/repos/asf/geode.git On 9/1/17 2:37 PM, Jens Deppe wrote: TL;DR - Remove references to https://git-wip-us.apache.org/repos/asf/geode.git and replace/add ssh:// g...@github.com/apache/geode

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

2017-09-01 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #665 was successful. --- Scheduled 2029 tests in total. https://build.spring.io/browse/SGF-NAG-665/ -- This

Re: git repo is gone

2017-09-01 Thread Jacob Barrett
Gitbox is the mirror now, github is the authoritative now. > On Sep 1, 2017, at 2:48 PM, Bruce Schuchardt wrote: > > So we go directly to github now? Not gitbox.apache.org? > > I'm able to clone this: https://gitbox.apache.org/repos/asf/geode.git > > >> On 9/1/17 2:37 PM, Jens Deppe wrote: >

Re: git repo is gone

2017-09-01 Thread Jens Deppe
I'm going to update our Jenkins nightly builds to point at the github repo. --Jens On Fri, Sep 1, 2017 at 3:51 PM, Jacob Barrett wrote: > Gitbox is the mirror now, github is the authoritative now. > > > On Sep 1, 2017, at 2:48 PM, Bruce Schuchardt > wrote: > > > > So we go directly to github n

Re: git repo is gone

2017-09-01 Thread Bruce Schuchardt
I am unable to push to the github repo > git push origin develop ERROR: Permission to apache/geode.git denied to bschuchardt. fatal: Could not read from remote repository. Please make sure you have the correct access rights and the repository exists. > git remote -v origin    ssh://g...@github

Re: git repo is gone

2017-09-01 Thread Jacob Barrett
Have you registered your github account with Apache’s directory? Log into the Apache ID portal and add your github. You’ll have access after the next sync. > On Sep 1, 2017, at 4:16 PM, Bruce Schuchardt wrote: > > I am unable to push to the github repo > > > git push origin develop > ERROR: P

Build failed in Jenkins: Geode-nightly #941

2017-09-01 Thread Apache Jenkins Server
See Changes: [jstewart] GEODE-3525: Dockerize AcceptanceTests [jiliao] GEODE-3549: fix the constantly failing flaky tests in -- [...truncated 90.07 KB...] org.apache.geode.m

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

2017-09-01 Thread Apache Jenkins Server
See -- Started by upstream project "Geode-nightly" build number 941 originally caused by: Started by timer [EnvInject] - Loading node environment variables. Building remotely on H26 (c