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...]

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

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   

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

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

[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 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://

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

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: 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

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

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

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

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: 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

[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

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,

[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

[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

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

[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

[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 @@ +/* + *

[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 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

[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

[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

[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_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_r136638194 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExportDataCommand.java --- @@ -87,4 +80,45 @@ public Result

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

[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

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

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: 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

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

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,