License / notice guide

Thanks to a lot of really helpful threads on general@ [1][2][3][4] I put together a guide on LICENSE / NOTICE for geode [5]. Let me know if something is unclear or incorrect. This should be helpful for the following M2 tasks: GEODE-906 (Cleanup source headers) GEODE-905 (Source headers)

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #224 was SUCCESSFUL (with 1245 tests). Change made by John Blum <jb...@pivotal.io> and Oliver Gierke.

--- Spring Data GemFire > Nightly-ApacheGeode > #224 was successful. --- Scheduled with changes by John Blum , Oliver Gierke. 1249 tests in

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

Github user dschneider-pivotal commented on the pull request: https://github.com/apache/incubator-geode/pull/97#issuecomment-185480218 I pulled it in with this commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/45693879 but I forgot to mark it closed in the

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

Github user dschneider-pivotal commented on the pull request: https://github.com/apache/incubator-geode/pull/97#issuecomment-185470092 This looks good. I'll pull it in. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

Re: Review Request 43681: GEODE-855: Create unit test class for Fragment class

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43681/#review119558 --- Ship it! Ship It! - Darrel Schneider On Feb. 17, 2016, 3:19

Review Request 43681: GEODE-855: Create unit test class for Fragment class

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

Review Request 43680: GEODE-831: FreeListManager unit test

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

Re: Review Request 43655: Not including gradle generated directories in the source distribution

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43655/#review119546 --- Ship it! Ship It! - Mark Bretl On Feb. 17, 2016, 10:32

[GitHub] incubator-geode pull request: GEODE-876: Add a README file to the ...

Github user sboorlagadda commented on the pull request: https://github.com/apache/incubator-geode/pull/96#issuecomment-185445347 @metatype Thanks for the review. I have removed RUNNING.txt from the distribution and removed instructions about docker form README. --- If your project

Re: Review Request 43677: GEODE-970: Remove duplicate HLL classes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43677/#review119542 --- Ship it! Ship It! - Sai Boorlagadda On Feb. 17, 2016, 10:29

Re: Review Request 43677: GEODE-970: Remove duplicate HLL classes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43677/#review119541 --- Ship it! Ship It! - Darrel Schneider On Feb. 17, 2016, 2:29

Re: GEODE-52 Remove @author tags from Java source

+1 On Wed, Feb 17, 2016 at 1:52 PM, William Markito wrote: > +1 > > Sent from my iPhone > > > On Feb 17, 2016, at 1:44 PM, Kenneth Howe wrote: > > > > +1 > > > > Ken > > > >> On Feb 17, 2016, at 1:35 PM, Anthony Baker wrote: > >> > >>

Re: GEODE-52 Remove @author tags from Java source

+1 *Vince Ford* GemFire Sustenance Engineering Beaverton, OR USA http://www.pivotal.io Open Source Project Geode https://geode.incubator.apache.org/ On Wed, Feb 17, 2016 at 1:52 PM, William Markito wrote: > +1 > > Sent

Re: GEODE-52 Remove @author tags from Java source

+1 Sent from my iPhone > On Feb 17, 2016, at 1:44 PM, Kenneth Howe wrote: > > +1 > > Ken > >> On Feb 17, 2016, at 1:35 PM, Anthony Baker wrote: >> >> +1 >> >> The ASF policy is well known [1] and I think @author information is no >> longer helpful in

Re: GEODE-52 Remove @author tags from Java source

+1 Ken > On Feb 17, 2016, at 1:35 PM, Anthony Baker wrote: > > +1 > > The ASF policy is well known [1] and I think @author information is no longer > helpful in any case. > > Anthony > > [1] >

Re: Review Request 43674: GEODE-969: Decreases WAN dunit log file size

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43674/#review119533 --- Ship it! Ship It! - xiaojian zhou On Feb. 17, 2016, 8:44

Re: GEODE-52 Remove @author tags from Java source

+1 -Dan On Wed, Feb 17, 2016 at 1:35 PM, Anthony Baker wrote: > +1 > > The ASF policy is well known [1] and I think @author information is no > longer helpful in any case. > > Anthony > > [1] >

Re: Review Request 43620: Added stat to know time spent in handover udp message to other executor pool

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43620/#review119527 --- Please change the spelling of every string and variable name that

Review Request 43674: GEODE-969: Decreases WAN dunit log file size

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

[GitHub] incubator-geode pull request: GEODE-876: Add a README file to the ...

GitHub user sboorlagadda opened a pull request: https://github.com/apache/incubator-geode/pull/96 GEODE-876: Add a README file to the binary distribution * included both README.md and RUNNING.txt into the binary distribution You can merge this pull request into a Git repository by

Re: Review Request 43655: Not including gradle generated directories in the source distribution

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43655/#review119513 --- Ship it! Ship It! - anilkumar gingade On Feb. 17, 2016,

Re: Review Request 43654: GEODE-967: Added xml support for GatewayEventSubstitutionFilter

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43654/#review119505 --- Ship it! Ship It! - Dan Smith On Feb. 17, 2016, 6:32 p.m.,

Review Request 43655: Not including gradle generated directories in the source distribution

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43655/ --- Review request for geode, Anthony Baker and Mark Bretl. Repository: geode

Review Request 43654: GEODE-967: Added xml support for GatewayEventSubstitutionFilter

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

Re: Review Request 43656: GEODE-511: rebranding pulse

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43656/ --- (Updated Feb. 17, 2016, 6:32 p.m.) Review request for geode. Repository:

Review Request 43647: GEODE-636: Removing DNS resolution for test

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

Review Request 43656: GEODE-511: rebranding pulse

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43656/ --- Review request for geode. Repository: geode Description --- GEODE-511:

Build failed in Jenkins: Geode-nightly #381

See Changes: [jdeppe] Bring version format in line [sbawaskar] Fixed broken links in README.md file. [jiliao] GEODE-954: display HostnameForClient in the member list view [bschuchardt] GEODE-899

Re: Review Request 43624: GEODE-323: Primary Key indexes are not being properly used for Partitioned Region Queries

> On Feb. 17, 2016, 1:24 a.m., anilkumar gingade wrote: > > gemfire-core/src/main/java/com/gemstone/gemfire/cache/query/internal/index/PrimaryKeyIndexCreationHelper.java, > > line 80 > > > > > > How did this use to

Re: Review Request 43620: Added stat to know time spent in handover udp message to other executor pool

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43620/#review119496 --- Ship it! Ship It! - Udo Kohlmeyer On Feb. 16, 2016, 8:21

[GitHub] incubator-geode pull request: GEODE-342: Make changes to Geode gra...

Github user jujoramos commented on the pull request: https://github.com/apache/incubator-geode/pull/71#issuecomment-185093557 Hey Vincent, I have the option to close the pull request from my GitHub account, do you want me to close it?. Or doing so implies some type of history

Re: Review Request 43647: GEODE-636: Removing DNS resolution for test

> On Feb. 17, 2016, 4:36 p.m., Bruce Schuchardt wrote: > > I don't think this is the correct fix for GEODE-636. The failure was > > > > expected:<...o6(16540:loner):[0]:67c8047a> but > > was:<...o6(16540:loner):[48516]:67c8047a> > > > > showing that the port number in the cached

Re: Review Request 43606: Fixing a suspect NPE string

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43606/#review119495 --- Ship it! Ship It! - Jianxia Chen On Feb. 16, 2016, 5:15

Re: Review Request 43620: Added stat to know time spent in handover udp message to other executor pool

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43620/#review119494 --- Ship it! Ship It! - Jianxia Chen On Feb. 16, 2016, 8:21

Re: Review Request 43647: GEODE-636: Removing DNS resolution for test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43647/#review119474 --- I don't think this is the correct fix for GEODE-636. The failure