[GitHub] incubator-geode pull request: GEODE-511 Pulse: Residual 'Gemfire' ...

Github user nitin-amp commented on the pull request: https://github.com/apache/incubator-geode/pull/111#issuecomment-199145501 Thanks for +1's. Is anything pending to merge this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

JVSD status

Just wanted to know where this is at. Don't see any JIRA activity [1] since mid-Feb though all the subtasks are resolved. Is JVSD working but currently parked on a feature branch [2]? Thanks, Nitin [1] https://issues.apache.org/jira/browse/GEODE-78 [2]

Review Request 45129: fixed the DUnit failure due to previous fix

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45129/ --- Review request for geode and Swapnil Bawaskar. Bugs: GEODE-1109

Re: Review Request 45129: fixed the DUnit failure due to previous fix

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45129/#review124671 --- Ship it! Ship It! - Darrel Schneider On March 21, 2016,

Re: Replacement of System environment variable GEMFIRE

Any idea why we would rely on a GEODE_HOME at all. Would we not include the relevant libraries on the classpath? Maybe I'm missing a use case here.. On 22/03/2016 9:59 am, Jens Deppe wrote: +1 for GEODE_HOME Just a note on this. I'm not sure where all the GEMFIRE env variable is currently

Re: Review Request 45114: GEODE-27: Adding spring-webmvc to pulse

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45114/#review124657 --- Ship it! Ship It! - Jason Huynh On March 21, 2016, 5:46

Re: Review Request 45114: GEODE-27: Adding spring-webmvc to pulse

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45114/#review124676 --- Ship it! Ship It! - Jens Deppe On March 21, 2016, 5:46

Replacement of System environment variable GEMFIRE

Hi there. There seems to be some reliance of the GEMFIRE system environment variable. Would we not want to change this to GEODE or maybe GEODE_HOME? --Udo

Re: Replacement of System environment variable GEMFIRE

+1 for GEODE_HOME Le 3/21/2016 3:53 PM, Anthony Baker a écrit : Switching to GEODE_HOME makes sense to me (that is, if we HAVE to use an env var at all). What code is affected? Anthony On Mar 21, 2016, at 3:50 PM, Udo Kohlmeyer wrote: Hi there. There seems to be

Re: Replacement of System environment variable GEMFIRE

+1 for GEODE_HOME Just a note on this. I'm not sure where all the GEMFIRE env variable is currently used, but I do know it gets used to find the various war files we have. We've recently added code to also discover these wars on the classpath. This allows, for example, an embedded locator to find

Re: Review Request 45116: GEODE-27: Adding spring-expression back in to geode-web

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45116/#review124640 --- Ship it! Ship It! - Mark Bretl On March 21, 2016, 11:07

Review Request 45131: GEODE-923 Validation for idle Timeout set properly.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45131/ --- Review request for geode, Bruce Schuchardt, Jianxia Chen, Udo Kohlmeyer, and

Re: Review Request 45114: GEODE-27: Adding spring-webmvc to pulse

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45114/#review124662 --- Ship it! Ship It! - Mark Bretl On March 21, 2016, 10:46

Re: Review Request 45129: fixed the DUnit failure due to previous fix

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45129/#review124669 --- Ship it! Ship It! - Swapnil Bawaskar On March 21, 2016,

Re: Replacement of System environment variable GEMFIRE

Switching to GEODE_HOME makes sense to me (that is, if we HAVE to use an env var at all). What code is affected? Anthony > On Mar 21, 2016, at 3:50 PM, Udo Kohlmeyer wrote: > > Hi there. > > There seems to be some reliance of the GEMFIRE system environment variable. >

Passed: apache/incubator-geode#218 (develop - 4d166a4)

Build Update for apache/incubator-geode - Build: #218 Status: Passed Duration: 16 minutes and 5 seconds Commit: 4d166a4 (develop) Author: Bruce Schuchardt Message: GEODE-956 Product use log shows duplicate entries fixing compilation error caused by a rebase

Errored: apache/incubator-geode#217 (develop - 68dab4a)

Build Update for apache/incubator-geode - Build: #217 Status: Errored Duration: 6 minutes and 33 seconds Commit: 68dab4a (develop) Author: Bruce Schuchardt Message: GEODE-956 Product use log shows duplicate entries The product use logger was adding up the

Re: Replacement of System environment variable GEMFIRE

Quickly searching through the code base it seems to be mainly used in: * Pulse * REST * GFSH But I imagine that it might pop up in other places as well. --Udo On 22/03/2016 9:53 am, Anthony Baker wrote: Switching to GEODE_HOME makes sense to me (that is, if we HAVE to use an env var at

Re: Review Request 45131: GEODE-923 Validation for idle Timeout set properly.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45131/#review124680 --- Ship it! Ship It! - Bruce Schuchardt On March 21, 2016,

Re: Review Request 45132: GEODE-1115 membership-port-range is not being established properly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45132/#review124682 ---

Re: Review Request 45077: GEODE-1113 - Redis Adapter region creation performance problems

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45077/#review124686 ---

Review Request 45132: GEODE-1115 membership-port-range is not being established properly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45132/ --- Review request for geode, Hitesh Khamesra, Jianxia Chen, and Udo Kohlmeyer.

Re: [DISCUSS] M2 issues - what's included and how we're tracking them

I changed the wiki page queries to look at fix version, based on the replies on this thread. I also removed the M2 fix version from the issues suggested. Our new list of M2 issues is now down to these: GEODE-52 Remove @author tags from Java source GEODE-386 Change xsd namespace to apache

Re: Review Request 45132: GEODE-1115 membership-port-range is not being established properly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45132/#review124689 ---

Re: Replacement of System environment variable GEMFIRE

GEMFIRE_HOME was used primarily to avoid having to require that the user set a CLASSPATH. Some of the code may have relied on walking backwards from a shell script to discover the GEMFIRE_HOME and its jar files. It's also used in documentation to facilitate talking about the root of a GEMFIRE

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

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

M2 or M3

Did M2 already release? I see an 1.0.0-incubating.M3 release already in Jira. When we mark a ticket resolved, should we be marking it with M2 or M3? -Kirk

Re: Review Request 45053: GEODE-936 test is seeing DistributedSystemDisconnectedException exception

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45053/#review124574 --- Ship it! Ship It! - Jianxia Chen On March 18, 2016, 9:45

Build failed in Jenkins: Geode-nightly #415

See -- [...truncated 866 lines...] com.gemstone.gemfire.management.internal.cli.commands.DurableClientCommandsDUnitTest > testCloseDurableCQ FAILED junit.framework.AssertionFailedError at

Re: M2 or M3

You should mark it as M2. M2 has not been released yet. -Dan On Mon, Mar 21, 2016 at 9:49 AM, Kirk Lund wrote: > Did M2 already release? I see an 1.0.0-incubating.M3 > > release > already in Jira. When we

Re: Build failed in Jenkins: Geode-nightly #415

I think my change for GEODE-27 broke these three tests. I'm looking into it: com.gemstone.gemfire.distributed.LocatorLauncherAssemblyJUnitTest.testLocatorStopsWhenJmxPortIsZero com.gemstone.gemfire.distributed.LocatorLauncherAssemblyJUnitTest.testLocatorStopsWhenJmxPortIsNonZero

Review Request 45114: GEODE-27: Adding spring-webmvc to pulse

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

Review Request 45116: GEODE-27: Adding spring-expression back in to geode-web

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

Re: M2 or M3

When we cut a release branch for M2, at that point we’ll update the develop version to M3-SNAPSHOT (or whatever the next version is). When you fix a bug on develop, use the version in gradle.properties to set the ‘Fix Version’ in JIRA. Anthony > On Mar 21, 2016, at 10:03 AM, Dan Smith

Re: JVSD status

As I know; initial work was done to donate the code to apache...Its one of the project community can pick-up and contribute without knowing much of the Geode... -Anil. On Sun, Mar 20, 2016 at 11:04 PM, Nitin Lamba wrote: > Just wanted to know where this is at. > > Don't see

Re: Review Request 45058: GEODE-1107 CI failure: RestAPIsWithSSLDUnitTest.testSimpleSSL

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45058/#review124602 ---

Re: Review Request 45116: GEODE-27: Adding spring-expression back in to geode-web

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45116/#review124621 --- Ship it! Ship It! - Jason Huynh On March 21, 2016, 6:07

Re: JVSD status

There is still significant work to make jvsd usable as a tool for review of the geode metrics. The initial work was to make sure it could be built, and generally worked ( started, open stats files, basic graphs). But there is still significant work needed to get to point where it can be part

Re: [DISCUSS] Remove setCustomEvictionAttributes that came in with HDFS changes from develop?

It will be nice to start a google doc or wiki to capture requirements, proposals and feedbacks I think the overflow should be treated similar to other data store; as its not different than storing the data in any persistence type...User may want to use one storage type (Disk, HDFS, database,

Review Request 45142: GEODE-722 CI failure: Bug42039JUnitTest.testBug42039

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45142/ --- Review request for geode, Bruce Schuchardt, Darrel Schneider, Hitesh Khamesra,