Re: [GitHub] incubator-geode pull request: GEODE-1163 Fix branding in Swagger U...

2016-04-08 Thread Nitin Lamba
Hi Dan, I tried that first (without https) but got a permission denied (pubkey) error - will try your suggested URL again next time. Thanks @Anthony! Nitin On Friday, April 8, 2016, Dan Smith wrote: > Hi Nitin, > > Github is just a mirror, you need to push your changes to

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Nabarun Nag
As per Jason's advice I will open a new ticket for rewriting the test case design as tests are not suppose to wait for 4 min. Also, while running all the WAN tests on IntelliJ I was able to find more tests with timing issue which Jason and me solved earlier. (Running two IntelliJ instances

Re: [GitHub] incubator-geode pull request: GEODE-1163 Fix branding in Swagger U...

2016-04-08 Thread Anthony Baker
Done. Anthony > On Apr 8, 2016, at 9:07 PM, Dan Smith wrote: > > Hi Nitin, > > Github is just a mirror, you need to push your changes to the apache > repository. Use this url and your apache credentials: > > https://git-wip-us.apache.org/repos/asf/incubator-geode.git > >

[GitHub] incubator-geode pull request: GEODE-1163 Fix branding in Swagger U...

2016-04-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/122 --- 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

Re: [GitHub] incubator-geode pull request: GEODE-1163 Fix branding in Swagger U...

2016-04-08 Thread Dan Smith
Hi Nitin, Github is just a mirror, you need to push your changes to the apache repository. Use this url and your apache credentials: https://git-wip-us.apache.org/repos/asf/incubator-geode.git -Dan On Fri, Apr 8, 2016 at 7:35 PM, Nitin Lamba wrote: > I tried but get

Re: [GitHub] incubator-geode pull request: GEODE-1163 Fix branding in Swagger U...

2016-04-08 Thread Nitin Lamba
I tried but get the following error: Pushing to https://nitin-...@github.com/apache/incubator-geode.git remote: Permission to apache/incubator-geode.git denied to nitin-amp. Perhaps, it was not enabled last month OR I may be doing something wrong. Any help is appreciated. Thanks On Fri, Apr

Review Request 45971: GEODE-1204 modified conditionals and opacity to make logo visible on Releases page.

2016-04-08 Thread Dave Barnes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45971/ --- Review request for geode. Repository: geode Description --- GEODE-1204

Review Request 45947: Update lucene version to 6.0.0

2016-04-08 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45947/ --- Review request for geode, anilkumar gingade, Barry Oglesby, nabarun nag, Dan

Re: [GitHub] incubator-geode pull request: GEODE-1163 Fix branding in Swagger U...

2016-04-08 Thread Swapnil Bawaskar
I have merged-in my pull requests in the past. So, I think you can too. Just follow the instructions in "Accepting the Pull-request" section on https://cwiki.apache.org/confluence/display/GEODE/Code+contributions On Fri, Apr 8, 2016 at 3:34 PM, Nitin Lamba wrote: > Any takers

Re: Incubator status report on wiki?

2016-04-08 Thread Swapnil Bawaskar
I have added the April 2016 report: https://cwiki.apache.org/confluence/display/GEODE/Community+development On Fri, Apr 8, 2016 at 2:50 PM, Swapnil Bawaskar wrote: > Sure! I will add it to the geode wiki too. > > On Fri, Apr 8, 2016 at 2:29 PM, William Markito

Re: Incubator status report on wiki?

2016-04-08 Thread Swapnil Bawaskar
Sure! I will add it to the geode wiki too. On Fri, Apr 8, 2016 at 2:29 PM, William Markito wrote: > Yup, all the reports usually are stored in the Community Development page > as well as in the Incubator wiki - >

Re: Incubator status report on wiki?

2016-04-08 Thread William Markito
Yup, all the reports usually are stored in the Community Development page as well as in the Incubator wiki - https://cwiki.apache.org/confluence/display/GEODE/Community+development On Fri, Apr 8, 2016 at 1:07 PM Greg Chase wrote: > Yes please  > > This email encrypted by tiny

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

2016-04-08 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #273 was successful. --- Scheduled 1383 tests in total. https://build.spring.io/browse/SGF-NAG-273/ -- This

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45938/#review127879 --- Ship it! Ship It! - Barry Oglesby On April 8, 2016, 8:26

Re: Bundled jar dependencies

2016-04-08 Thread Dan Smith
I think these gfsh failures were caused by my changes for GEODE-1025. By removing the rest of the spring jars from the geode-core depencies, we reverted to using the version of spring-core that spring-shell depends on. I have a fix to force the version of spring core to be 4.2.4:

Re: Review Request 45945: GEODE-1025: Force the version of spring core we use

2016-04-08 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45945/#review127878 --- Ship it! Ship It! - Anthony Baker On April 8, 2016, 8:42

Re: [discuss] Install Google Analytics for tracking activity on geode.incubator.apache.org

2016-04-08 Thread Gregory Chase
Amazing! We've had the largest spike of traffic on geode.incubator.apache.org that I've seen in 3 months. Or, our analytics are working again! Thanks Dave! If anyone is interested in having access, please email your request to priv...@geode.incubator.apache.org, or apachege...@gmail.com.

Review Request 45945: GEODE-1025: Force the version of spring core we use

2016-04-08 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45945/ --- Review request for geode and Anthony Baker. Repository: geode Description

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45938/ --- (Updated April 8, 2016, 8:26 p.m.) Review request for geode, anilkumar

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Barry Oglesby
> On April 8, 2016, 8:02 p.m., Barry Oglesby wrote: > > Maybe that timeout should be configurable? Most times 4 minutes might as > > well be forever. > > Jason Huynh wrote: > Yes, that would be a good idea. I'm trying to revert this one change in > the big refactoring I did earlier for

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Jason Huynh
> On April 8, 2016, 8:02 p.m., Barry Oglesby wrote: > > Maybe that timeout should be configurable? Most times 4 minutes might as > > well be forever. > > Jason Huynh wrote: > Yes, that would be a good idea. I'm trying to revert this one change in > the big refactoring I did earlier for

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Barry Oglesby
> On April 8, 2016, 8:02 p.m., Barry Oglesby wrote: > > Maybe that timeout should be configurable? Most times 4 minutes might as > > well be forever. > > Jason Huynh wrote: > Yes, that would be a good idea. I'm trying to revert this one change in > the big refactoring I did earlier for

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45938/#review127870 --- Ship it! Ship It! - Barry Oglesby On April 8, 2016, 6:37

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Jason Huynh
> On April 8, 2016, 8:02 p.m., Barry Oglesby wrote: > > Maybe that timeout should be configurable? Most times 4 minutes might as > > well be forever. Yes, that would be a good idea. I'm trying to revert this one change in the big refactoring I did earlier for GEODE-1062. I didn't think

Re: Incubator status report on wiki?

2016-04-08 Thread Greg Chase
Yes please  This email encrypted by tiny buttons & fat thumbs, beta voice recognition, and autocorrect on my iPhone. > On Apr 8, 2016, at 1:05 PM, Michael Stolz wrote: > > Should the geode incubator status report be posted on the geode wiki under > "Incubation Status

Incubator status report on wiki?

2016-04-08 Thread Michael Stolz
Should the geode incubator status report be posted on the geode wiki under "Incubation Status Reports"? -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: 631-835-4771

Re: Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45938/#review127865 --- Maybe that timeout should be configurable? Most times 4 minutes

[GitHub] incubator-geode pull request: Feature/geode 904

2016-04-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/126 --- 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

[VIDEO] Posted! Geode Clubhouse 4/6/16 - Security in Geode

2016-04-08 Thread Gregory Chase
Greetings Geode community, We've just posted the recording for last Wednesday's Geode Clubhouse. Please enjoy: https://www.youtube.com/watch?v=so7AJWgHnvA=youtu.be -Greg On Tue, Apr 5, 2016 at 8:39 AM, Gregory Chase wrote: > It's time for the next Apache Geode Clubhouse

Re: Review Request 45827: GEODE-902: Remove incorrect src header; update LICENSE and NOTICE

2016-04-08 Thread Joey McAllister
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45827/#review127851 --- Ship it! Ship It! - Joey McAllister On April 6, 2016, 6:07

Re: Review Request 45919: Updates to NOTICE

2016-04-08 Thread Joey McAllister
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45919/#review127849 --- Ship it! Ship It! - Joey McAllister On April 8, 2016, 12:21

Re: Review Request 45931: GEODE-914: Added specific NOTICE files for war projects

2016-04-08 Thread Joey McAllister
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45931/#review127850 --- Ship it! Ship It! - Joey McAllister On April 8, 2016, 3:26

[GitHub] incubator-geode pull request: Feature/geode 904

2016-04-08 Thread metatype
Github user metatype commented on the pull request: https://github.com/apache/incubator-geode/pull/126#issuecomment-207566042 +1 --- 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

Re: Geode Nightly

2016-04-08 Thread William Markito
I've just added a timeout (24h) to our builds to avoid it getting stuck for such long periods and we're now going to archive progress test files as well. Hope that helps On Fri, Apr 8, 2016 at 8:37 AM Anthony Baker wrote: > The last task was :geode-wan:distributedTest.

[GitHub] incubator-geode pull request: Feature/geode 904

2016-04-08 Thread metatype
Github user metatype commented on the pull request: https://github.com/apache/incubator-geode/pull/126#issuecomment-207558046 This looks awesome!! Only change I would make is to add this to the CDDL license for the javax* dependencies: ``` NOTICE PURSUANT TO SECTION 9

Re: Bundled jar dependencies

2016-04-08 Thread Swapnil Bawaskar
With the latest revision: dd9e5af4b4324e5511d170238f6532217384e15b I can't even start gfsh. I get this message: $ ./bin/gfsh Required (Spring Core) libraries not found in the classpath. gfsh can't start. On Fri, Apr 8, 2016 at 11:00 AM, Jinmei Liao wrote: > Looks like the

Review Request 45938: GEODE-1062: Undoing reduction in wait time

2016-04-08 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45938/ --- Review request for geode, anilkumar gingade, Barry Oglesby, nabarun nag, Dan

Re: Bundled jar dependencies

2016-04-08 Thread Jinmei Liao
Looks like the error is due to the fact that geode-dependencies.jar and gfsh-dependencies.jar includes both spring-core versions in it's classpath, and it fails to find ResolvableTypeProvider.java which only exists in the later version. I tried turning off the transitive dependency of

Re: Bundled jar dependencies

2016-04-08 Thread Jinmei Liao
On a similar note, I just did a pull from develop and ran a clean installDist. When I tried to start a locator in GFSH, I got the following error in the log file: [error 2016/04/08 09:53:12.979 PDT locator1 tid=0x27] Jmx manager could not be started because HTTP service failed to start

Re: Geode Nightly

2016-04-08 Thread Anthony Baker
The last task was :geode-wan:distributedTest. Without the full contents of the build dirs it’s hard to tell what test hung. We don’t have a way to detect a hung DUnit, dump stacks, and move on right? Anthony > On Apr 8, 2016, at 7:55 AM, Jinmei Liao wrote: > > FYI: The

Review Request 45931: GEODE-914: Added specific NOTICE files for war projects

2016-04-08 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45931/ --- Review request for geode, Jens Deppe, Joey McAllister, and Dan Smith.

Geode Nightly

2016-04-08 Thread Jinmei Liao
FYI: The last geode nightly has been running for more than 3 days. William killed it this morning. The log doesn't say much (at least not to me and William :)). https://builds.apache.org/job/Geode-nightly/431/ -- Cheers Jinmei

Re: Bundled jar dependencies

2016-04-08 Thread Anthony Baker
After an update I’m seeing a conflict on spring-core. Here’s the geode-dependencies classpath with both versions of spring-core: Class-Path: geode-common-1.0.0-incubating.M2-SNAPSHOT.jar geode-json-1 .0.0-incubating.M2-SNAPSHOT.jar geode-joptsimple-1.0.0-incubating.M2- SNAPSHOT.jar

Re: Review Request 45919: Updates to NOTICE

2016-04-08 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45919/ --- (Updated April 8, 2016, 12:21 p.m.) Review request for geode, Joey McAllister,