[GitHub] incubator-geode pull request: GEODE-854: refactor gemfire-pulse no...

Github user metatype commented on the pull request: https://github.com/apache/incubator-geode/pull/78#issuecomment-174831927 LGTM! +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

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

--- Spring Data GemFire > Nightly-ApacheGeode > #201 was successful. --- Scheduled 1226 tests in total. https://build.spring.io/browse/SGF-NAG-201/ -- This

Re: Review Request 42736: GEODE-854: Put pulseversion.properties in generated-resources

> On Jan. 25, 2016, 8:46 p.m., Jinmei Liao wrote: > > Anthony filed a related bug: > > https://issues.apache.org/jira/browse/GEODE-852. I think we should re-use > > the GemfireVersion.properties that is already created in gemfire-core and > > not use this pulseVersion.properties at all. I can

[GitHub] incubator-geode pull request: GEODE-854: refactor gemfire-pulse no...

GitHub user jinmeiliao opened a pull request: https://github.com/apache/incubator-geode/pull/78 GEODE-854: refactor gemfire-pulse not to generate a pulseverion.prope… …rties file You can merge this pull request into a Git repository by running: $ git pull https://github.com

Re: Review Request 42736: GEODE-854: Put pulseversion.properties in generated-resources

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42736/#review116262 --- Ship it! Ship It! - Jinmei Liao On Jan. 25, 2016, 8:24 p.m.,

Re: Review Request 42736: GEODE-854: Put pulseversion.properties in generated-resources

> On Jan. 25, 2016, 8:46 p.m., Jinmei Liao wrote: > > Anthony filed a related bug: > > https://issues.apache.org/jira/browse/GEODE-852. I think we should re-use > > the GemfireVersion.properties that is already created in gemfire-core and > > not use this pulseVersion.properties at all. I can

Re: Review Request 42768: GEODE-857: Fixing synchronization in SystemFailure.stopWatchDog

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42768/#review116255 --- looks good but it would be great if this code has a unit test that

Review Request 42769: GEODE-858: Remove 1 second sleep from InternalLocator.startTcpServer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42769/ --- Review request for geode and Bruce Schuchardt. Repository: geode Description

Re: Review Request 42736: GEODE-854: Put pulseversion.properties in generated-resources

> On Jan. 25, 2016, 8:46 p.m., Jinmei Liao wrote: > > Anthony filed a related bug: > > https://issues.apache.org/jira/browse/GEODE-852. I think we should re-use > > the GemfireVersion.properties that is already created in gemfire-core and > > not use this pulseVersion.properties at all. I can

Review Request 42768: GEODE-857: Fixing synchronization in SystemFailure.stopWatchDog

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42768/ --- Review request for geode, Darrel Schneider and Kirk Lund. Repository: geode D

Proposal: Move Events to a non-Repo website

RFF (Request for Feedback) Apache Geode website design suggestion: Replace the volatile Events section of the Geode website with a link to a community events site that's not part of the Apache Geode source code repo. That way, the events can be updated quickly by community members without the dela

Re: Review Request 42763: GEODE-836: multicast flush is broken

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42763/#review116230 --- Ship it! Ship It! - Hitesh Khamesra On Jan. 26, 2016, 12:16

Review Request 42763: GEODE-836: multicast flush is broken

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42763/ --- Review request for geode and Hitesh Khamesra. Repository: geode Description -

[GitHub] incubator-geode pull request: Verified preceding content merges, f...

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

[GitHub] incubator-geode pull request: Update index.html

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

Re: Review Request 42729: GEODE-851: Cleanup tests in management.internal.security package

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42729/#review116221 --- Ship it! Ship It! - Jens Deppe On Jan. 25, 2016, 10:51 p.m.,

Re: Review Request 42729: GEODE-851: Cleanup tests in management.internal.security package

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42729/ --- (Updated Jan. 25, 2016, 10:51 p.m.) Review request for geode, Jens Deppe and Ji

Re: Review Request 42729: GEODE-851: Cleanup tests in management.internal.security package

> On Jan. 25, 2016, 10:09 p.m., Jens Deppe wrote: > > I'm not sure it's a good idea to add `ManagementInternalSecurityTestSuite`. > > If it's just for adhoc testing then I'd suggest rather using the > > `customTest` task which lets you do the same thing (of course not from the > > IDE). Yeah

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

On Mon, Jan 25, 2016 at 11:28 AM, Anthony Baker wrote: > Reviewing the feedback received so far on RC1 we have the following issues to > consider: > > 1) The binary NOTICE files needs additions for Spring and Netty [1]. > 2) The source distributions instructions specify use of gradlew which is no

Obsolete Geode website - EOL?

There's an out-of-date Geode site at http://pivotalsoftware.github.io/geode-site/. I'm not sure who owns this site, but I think it's time to end-of-life (EOL) it - can it come down now? Thanks!

Re: Obsolete Geode website - EOL?

Thanks! On Mon, Jan 25, 2016 at 2:14 PM, Gregory Chase wrote: > Its not the Apache project's responsibility as the "Project Geode" site is > owned by Pivotal. I'll get this to refer to the Apache project. > > -Greg > > On Mon, Jan 25, 2016 at 2:10 PM, Dave Barnes wrote: > > > There's an out-of

Re: Obsolete Geode website - EOL?

Its not the Apache project's responsibility as the "Project Geode" site is owned by Pivotal. I'll get this to refer to the Apache project. -Greg On Mon, Jan 25, 2016 at 2:10 PM, Dave Barnes wrote: > There's an out-of-date Geode site at > http://pivotalsoftware.github.io/geode-site/. > I'm not

Re: Review Request 42729: GEODE-851: Cleanup tests in management.internal.security package

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42729/#review116191 --- I'm not sure it's a good idea to add `ManagementInternalSecurityTe

Re: Feature branch cleanup - branches to delete

These branches are now deleted. -Dan On Tue, Jan 19, 2016 at 1:56 PM, Dan Smith wrote: > Now that we can delete branches again, we should clean up a bunch of > feature branches that were left around due to the freeze. > > Here is a list of branches that are associated with closed tickets. If >

Re: Review Request 42736: GEODE-854: Put pulseversion.properties in generated-resources

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42736/#review116161 --- Anthony filed a related bug: https://issues.apache.org/jira/browse

Re: Review Request 42734: GEODE-781: Check for existence of settings.xml

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42734/#review116158 --- Ship it! Ship It! - Dan Smith On Jan. 25, 2016, 7:44 p.m., A

Review Request 42736: GEODE-854: Put pulseversion.properties in generated-resources

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42736/ --- Review request for geode, Jens Deppe and Jinmei Liao. Repository: geode Descr

[GitHub] incubator-geode pull request: Update Events section of Geode Commu...

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

Review Request 42729: GEODE-851: Cleanup tests in management.internal.security package

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42729/ --- Review request for geode, Jens Deppe and Jinmei Liao. Bugs: GEODE-851 https

java 1.7 support was([jira] [Commented] (GEODE-479) Bump Java Language level to 1.8)

Sending to the dev list (to escape mail filters). -- Forwarded message -- From: Christian Tzolov (JIRA) Date: Sat, Jan 23, 2016 at 12:40 PM Subject: [jira] [Commented] (GEODE-479) Bump Java Language level to 1.8 To: iss...@geode.incubator.apache.org [ https://issues.apache

Review Request 42734: GEODE-781: Check for existence of settings.xml

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42734/ --- Review request for geode, Darrel Schneider and Dan Smith. Repository: geode D

Re: Pulse UI not starting

On second thought, since we are close to a release and will be updating the master soon after, I think it will be good to leave the default branch as master. So that when somebody clones the repo they are on a stable released code. On Mon, Jan 25, 2016 at 10:51 AM, William Markito wrote: > If w

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

Reviewing the feedback received so far on RC1 we have the following issues to consider: 1) The binary NOTICE files needs additions for Spring and Netty [1]. 2) The source distributions instructions specify use of gradlew which is not included and also specify use of jdk 1.7 instead of jdk1.8. I

Re: Pulse UI not starting

If we want to change that someone needs to open a ticket with ASF infra. Sent from my iPhone > On Jan 25, 2016, at 10:41 AM, Swapnil Bawaskar wrote: > > Yes, the default branch can be changed by following instructions here: > https://help.github.com/articles/setting-the-default-branch/ > I

Re: Pulse UI not starting

I think this has to be done by filing an INFRA ticket: https://issues.apache.org/jira/browse/INFRA-9208 https://issues.apache.org/jira/browse/INFRA-7659 Anthony > On Jan 25, 2016, at 10:41 AM, Swapnil Bawaskar wrote: > > Yes, the default branch can be changed by following instructions here:

Re: Review Request 42679: GEODE-832: Modifying the dunit framework to allow lambda expressions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42679/#review116142 --- Ship it! Ship It! - Kirk Lund On Jan. 23, 2016, 1:41 a.m., D

Re: Pulse UI not starting

Yes, the default branch can be changed by following instructions here: https://help.github.com/articles/setting-the-default-branch/ I think it is enough to change this only on github, since we do not point users to ASF. I don't have permissions to change this, Anthony/Roman can either of you pleas

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

GitHub user pdxrunner opened a pull request: https://github.com/apache/incubator-geode/pull/77 Feature/geode 735 Added OffHeapHelperJUnitTest class You can merge this pull request into a Git repository by running: $ git pull https://github.com/pdxrunner/incubator-geode feature/

Re: Review Request 41836: GEODE-719: Add error logs while cache.xml processing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41836/#review116129 --- Ship it! Ship It! - Darrel Schneider On Jan. 22, 2016, 11:22

Re: Review Request 42725: Called shutdownOutput and flush while closing the client socket.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42725/#review116125 --- Ship it! Ship It! - Bruce Schuchardt On Jan. 25, 2016, 5:25

Re: Review Request 42725: Called shutdownOutput and flush while closing the client socket.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42725/#review116124 --- Ship it! Ship It! - anilkumar gingade On Jan. 25, 2016, 5:25

Review Request 42725: Called shutdownOutput and flush while closing the client socket.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42725/ --- Review request for geode and Bruce Schuchardt. Repository: geode Description

Re: Pulse UI not starting

Is it possible to set the default branch to develop? That will save a lot of head-scratching, though once we have a regular release cadence going it won’t be as much of an issue. Anthony > On Jan 25, 2016, at 8:38 AM, Joseph Winston > wrote: > > Jinmei — > > It appears that the issue that

Re: Review Request 42665: Move dunit to com.gemstone.gemfire.test.dunit

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42665/#review116098 --- Ship it! Ship It! - Jinmei Liao On Jan. 22, 2016, 6:58 p.m.,

Re: Review Request 42665: Move dunit to com.gemstone.gemfire.test.dunit

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42665/#review116096 --- Ship it! Ship It! - Jens Deppe On Jan. 22, 2016, 6:58 p.m.,

Build failed in Jenkins: Geode-nightly #357

See -- [...truncated 427 lines...] :gemfire-web-api:sourcesJar :gemfire-web-api:war :gemfire-web-api:signArchives SKIPPED :gemfire-assembly:distTar :gemfire-assembly:distZip :gemfire-assembly:srcDistTar :gem