Re: Review Request 44242: GEODE-1025: Replacing direct dependency on spring data gemfire with an SPI

2016-03-01 Thread John Blum
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44242/#review121586 --- geode-core/src/main/java/com/gemstone/gemfire/distributed/ServerL

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #237 was SUCCESSFUL (with 1263 tests). Change made by Dan Smith and John Blum .

2016-03-01 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #237 was successful. --- Scheduled with changes by Dan Smith , John Blum . 1267 tests in total. https://build.sp

Review Request 44242: GEODE-1025: Replacing direct dependency on spring data gemfire with an SPI

2016-03-01 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44242/ --- Review request for geode, Jason Huynh, Jens Deppe, and John Blum. Repository: g

Fixed: apache/incubator-geode#43 (feature/GEODE-949-2 - f9b621d)

2016-03-01 Thread Travis CI
Build Update for apache/incubator-geode - Build: #43 Status: Fixed Duration: 15 minutes and 6 seconds Commit: f9b621d (feature/GEODE-949-2) Author: Kirk Lund Message: Fix up serialization of exceptions View the changeset: https://github.com/apache/incubator-g

Re: Review Request 44233: GEODE-1029 fixed test issue, where system is not working properly.

2016-03-01 Thread Jianxia Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44233/#review121545 --- I am just curious. Why the test didn't fail previously? Only until

Re: Scope for M2 release

2016-03-01 Thread Anthony Baker
(thumbs up emoji) > On Mar 1, 2016, at 3:12 PM, Nitin Lamba wrote: > > Please try the link again... > > > From: Nitin Lamba > Sent: Tuesday, March 1, 2016 3:07 PM > To: dev@geode.incubator.apache.org > Subject: Re: Scope for M2 release > > My bad - let

Re: Scope for M2 release

2016-03-01 Thread Nitin Lamba
Please try the link again... From: Nitin Lamba Sent: Tuesday, March 1, 2016 3:07 PM To: dev@geode.incubator.apache.org Subject: Re: Scope for M2 release My bad - let me fix the filter and make it public. From: Ant

Re: Review Request 44193: GEODe-562 and changes to clean up some gradle dependencies

2016-03-01 Thread Mark Bretl
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44193/#review121536 --- Ship it! I am in line with making dependencies more definitive

Re: Scope for M2 release

2016-03-01 Thread Nitin Lamba
My bad - let me fix the filter and make it public. From: Anthony Baker Sent: Tuesday, March 1, 2016 2:34 PM To: dev@geode.incubator.apache.org Subject: Re: Scope for M2 release Great idea Nitin! I don’t think we have permissions to see it though:

Re: Review Request 44035: GEODE-1014: PdxSerializationException Error when putall an object from native client withSecurity enabled

2016-03-01 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44035/#review121534 --- Ship it! Ship It! - Udo Kohlmeyer On March 1, 2016, 12:01 a.

Failed: apache/incubator-geode#42 (feature/GEODE-17-2 - 406a80f)

2016-03-01 Thread Travis CI
Build Update for apache/incubator-geode - Build: #42 Status: Failed Duration: 19 minutes and 18 seconds Commit: 406a80f (feature/GEODE-17-2) Author: Jens Deppe Message: GEODE-17: Initial integration work - Fix failing tests View the changeset: https://github

Re: Valid characters in Region names

2016-03-01 Thread William Markito
That would be changed from now on as it's too much open ended and troublesome to support in other subsystems such as JMX beans... Note that "-" is still in the list, other symbols would not be allowed like "+" or "@" - Which are very odd and should be very rare. On Tue, Mar 1, 2016 at 2:52 PM, Ani

Re: Review Request 44233: GEODE-1029 fixed test issue, where system is not working properly.

2016-03-01 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44233/#review121529 --- Ship it! Ship It! - Udo Kohlmeyer On March 1, 2016, 10:29 p.

Re: Valid characters in Region names

2016-03-01 Thread Anilkumar Gingade
>From the existing javadoc it looks like, application can have any chars in region name except the "/". Changing this one may have implication on existing GemFire customers In the past we had to make changes in OQL to support chars like "+", "-", "@"...(since customers used this in their regi

Re: Valid characters in Region names

2016-03-01 Thread Udo Kohlmeyer
+1 On 2/03/2016 9:35 am, William Markito wrote: Folks, it doesn't look like we have actually finished this thread... What do you guys think about the following pattern: "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz123456-_" ? I'm not specifying a regexp to avoid problems with unicode

Review Request 44233: GEODE-1029 fixed test issue, where system is not working properly.

2016-03-01 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44233/ --- Review request for geode, Bruce Schuchardt, Darrel Schneider, Jianxia Chen, and

Re: Valid characters in Region names

2016-03-01 Thread William Markito
Folks, it doesn't look like we have actually finished this thread... What do you guys think about the following pattern: "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz123456-_" ? I'm not specifying a regexp to avoid problems with unicode and to keep it only ASCII-only... Stackoveflow has

Re: Scope for M2 release

2016-03-01 Thread Anthony Baker
Great idea Nitin! I don’t think we have permissions to see it though: JIRA Issues Macro: JIRA project does not exist or you do not have permission to view it. Anthony > On Mar 1, 2016, at 1:51 PM, Nitin Lamba wrote: > > OK, I've added a couple of JIRA filters to extract all issues l

Re: [DISCUSS] JIRA components

2016-03-01 Thread William Markito
I've just updated the list of components on JIRA to reflect the list proposed here which seems to be a good start. The list can and should evolve of course if something needs to be removed or added. Thanks, On Mon, Feb 29, 2016 at 3:23 PM, Anilkumar Gingade wrote: > Any conclusion on this one.

Re: Review Request 44193: GEODe-562 and changes to clean up some gradle dependencies

2016-03-01 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44193/#review121510 --- Ship it! Seems like a good incremental improvement! - Dan Smit

Re: Review Request 44125: GEODE-996: OffheapStats - fragmentation percentage calculation has to be revisited.

2016-03-01 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44125/#review121511 --- Ship it! Ship It! - Darrel Schneider On March 1, 2016, 10:26

Re: Scope for M2 release

2016-03-01 Thread Nitin Lamba
OK, I've added a couple of JIRA filters to extract all issues loaded in our M2 sprint. This helps create a data-driven table of issues on the wiki page [1] instead of manually sync-ing it. Ideally, we should start this (scope-defined) sprint and close it when M2 is released. Thoughts? Please ch

Re: Scope for M2 release

2016-03-01 Thread Nitin Lamba
+1 I had recently updated the wiki page for completed JIRAs. Ideally, we should just have one place to update. Let me quickly research if there's a way to integrate the lists. Nitin From: Kirk Lund Sent: Tuesday, March 1, 2016 11:33 AM To: dev@geode.inc

Re: Scope for M2 release

2016-03-01 Thread Kirk Lund
+1 On Tuesday, March 1, 2016, Anthony Baker wrote: > I was discussing the M2 release scope and noticed that the wiki page [1] > differs slightly from the JIRA Geode-Release filter [2] and roadmap [3], > > Should we update the wiki page to match JIRA based on the prior email > thread [4]? > > Ant

Re: Conributors.md

2016-03-01 Thread Kenneth Howe
+1 to provide a link on the Geode Community web page to the cwiki How to Contribute, with the cwiki being the one place to keep the info current. Ken > On Mar 1, 2016, at 10:57 AM, Dave Barnes wrote: > > +1 to providing a more obvious path to the "how to contribute" info. This > thread suggest

Re: Conributors.md

2016-03-01 Thread Dave Barnes
+1 to providing a more obvious path to the "how to contribute" info. This thread suggests two possibilities: - add a CONTRIBUTING.md (or CONTRIBUTORS.md) file to the sources, or - Add a link to the cwiki page ( https://cwiki.apache.org/confluence/display/GEODE/How+to+Contribute). I like

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

2016-03-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/108 --- 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 featur

Re: Review Request 44107: GEDOE-856: Unit tests for TinyMemoryBlock

2016-03-01 Thread Sai Boorlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44107/#review121461 --- Ship it! Ship It! - Sai Boorlagadda On Feb. 26, 2016, 11:40

Re: Review Request 44125: GEODE-996: OffheapStats - fragmentation percentage calculation has to be revisited.

2016-03-01 Thread Sai Boorlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44125/ --- (Updated March 1, 2016, 6:26 p.m.) Review request for geode and Darrel Schneide

Scope for M2 release

2016-03-01 Thread Anthony Baker
I was discussing the M2 release scope and noticed that the wiki page [1] differs slightly from the JIRA Geode-Release filter [2] and roadmap [3], Should we update the wiki page to match JIRA based on the prior email thread [4]? Anthony [1] https://cwiki.apache.org/confluence/display/GEODE/1.0

Re: Review Request 44125: GEODE-996: OffheapStats - fragmentation percentage calculation has to be revisited.

2016-03-01 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44125/#review121451 --- geode-core/src/main/java/com/gemstone/gemfire/internal/offheap/Fr

Re: Conributors.md

2016-03-01 Thread Nitin Lamba
Hi Dor, >From the e-mail subject, is this question about capturing the list of >contributors on the Geode project? The website has a list of committers on the >community page [1]; perhaps a link to contribution guidelines (wiki) can be >created from there. Nitin [1] http://geode.incubator.apac

Build failed in Jenkins: Geode-nightly #395

2016-03-01 Thread Apache Jenkins Server
See Changes: [ukohlmeyer] GEODE-870: Rejecting of old view [ukohlmeyer] GEODE-870: Handling multiple concurrent locator restarts. Elder locator [ukohlmeyer] GEODE-870: Handling multiple concurrent locator restarts. Elder locator [ukoh