[GitHub] incubator-geode issue #207: GEODE-1647: Add Integrated Security to Peer Auth...

2016-07-14 Thread jinmeiliao
Github user jinmeiliao commented on the issue: https://github.com/apache/incubator-geode/pull/207 this seems to have some conflict with the current develop branch. Please rebase again. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-geode issue #201: GEODE-1617: Regions can be created with a variet...

2016-07-14 Thread jinmeiliao
Github user jinmeiliao commented on the issue: https://github.com/apache/incubator-geode/pull/201 I think it's probably safe to get the storage team involved to review this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Re: [VOTE] Release Apache Geode (incubating) 1.0.0-incubating.M3 - RC2

2016-07-14 Thread Anthony Baker
+1 + Verified tag signature, built from tag + Checked hashes and signature + Built from source distribution + Ran from binary distribution + Reviewed maven repo + Ran replication example (after modifying geode version / maven url) + Checked against a few sample apps [1] [2] + Reviewed changes in

Passed: apache/incubator-geode#861 (rel/v1.0.0-incubating.M3.RC2 - 1fdcdb2)

2016-07-14 Thread Travis CI
Build Update for apache/incubator-geode - Build: #861 Status: Passed Duration: 13 minutes and 59 seconds Commit: 1fdcdb2 (rel/v1.0.0-incubating.M3.RC2) Author: Anthony Baker Message: GEODE-1665: Remove bundled jar from source distribution Modified the

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

2016-07-14 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #370 was successful. --- Scheduled 1412 tests in total. https://build.spring.io/browse/SGF-NAG-370/ -- This

[GitHub] incubator-geode pull request #207: GEODE-1647: Add Integrated Security to Pe...

2016-07-14 Thread gracemeilen
GitHub user gracemeilen opened a pull request: https://github.com/apache/incubator-geode/pull/207 GEODE-1647: Add Integrated Security to Peer Authentication You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-geode pull request #206: GEODE-1647: Add Integrated Security to Pe...

2016-07-14 Thread gracemeilen
Github user gracemeilen closed the pull request at: https://github.com/apache/incubator-geode/pull/206 --- 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

Review Request 50051: GEODE-1571: have auth-init accept either a constructor or a static factory method.

2016-07-14 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50051/ --- Review request for geode, Grace Meilen, Kevin Duling, and Kirk Lund.

Re: [VOTE] Release Apache Geode (incubating) 1.0.0-incubating.M3

2016-07-14 Thread Anthony Baker
Fixed GEODE-1664 and GEODE-1665. Let's cancel this vote and move on to RC2. Anthony > On Jul 13, 2016, at 9:31 PM, Anthony Baker wrote: > > geode-assembly/src/main/dist/LICENSE changes needed: > > 1) Add aopalliance 1.0 (public domain) > 2) Update slf4j version to 1.7.21 >

Re: Review Request 50042: GEODE-1664: Update M3 LICENSE

2016-07-14 Thread Joey McAllister
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50042/#review142291 --- Ship it! Looks good! - Joey McAllister On July 14, 2016,

Re: Review Request 50043: GEODE-1665: Remove bundled jar from source distribution

2016-07-14 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50043/#review142273 --- Ship it! Ship It! - Dan Smith On July 14, 2016, 7:12 p.m.,

[GitHub] incubator-geode pull request #205: Feature/geode-11-Lucene gfsh list stats a...

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

Review Request 50043: GEODE-1665: Remove bundled jar from source distribution

2016-07-14 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50043/ --- Review request for geode, William Markito and Dan Smith. Repository: geode

Review Request 50042: GEODE-1664: Update M3 LICENSE

2016-07-14 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50042/ --- Review request for geode, Joey McAllister and William Markito. Repository:

[GitHub] incubator-geode issue #205: Feature/geode-11-Lucene gfsh list stats and desc...

2016-07-14 Thread gesterzhou
Github user gesterzhou commented on the issue: https://github.com/apache/incubator-geode/pull/205 +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 enabled and wishes so, or if

[GitHub] incubator-geode pull request #205: Feature/geode-11-Lucene gfsh list stats a...

2016-07-14 Thread aparnard
GitHub user aparnard opened a pull request: https://github.com/apache/incubator-geode/pull/205 Feature/geode-11-Lucene gfsh list stats and describe - Added an option to display lucene index stats in the list lucene index command. - Added describe lucene index gfsh command that

[GitHub] incubator-geode issue #203: Feature/geode 11 gfsh list lucene stats

2016-07-14 Thread aparnard
Github user aparnard commented on the issue: https://github.com/apache/incubator-geode/pull/203 Closing this pull request because most of the commits are repeated. Creating a new pull request only with the new commits --- If your project is set up for it, you can reply to this email

[GitHub] incubator-geode pull request #204: GEODE-1612: Gets a list of random ports a...

2016-07-14 Thread nabarunnag
GitHub user nabarunnag opened a pull request: https://github.com/apache/incubator-geode/pull/204 GEODE-1612: Gets a list of random ports at once * Getting a random port for each locator one by one led to different locators getting assigned the same port number * By

Re: Review Request 50008: GEODE-1571: fix permission needed for remove data command

2016-07-14 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50008/#review142253 --- Ship it! Ship It! - Kirk Lund On July 13, 2016, 10:05 p.m.,

Re: Review Request 49992: GEODE-1571: use constructor instead of static method to create security-manager and post-processor

2016-07-14 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49992/#review142252 --- Ship it! Ship It! - Kirk Lund On July 13, 2016, 4:33 p.m.,

Re: Review Request 49959: GEODE-1571: fix nightly failure

2016-07-14 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49959/#review142249 --- Ship it! Ship It! - Kirk Lund On July 12, 2016, 4:33 p.m.,

[GitHub] incubator-geode pull request #202: GEODE-1600: startLocatorAndDS takes 0 as ...

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

[GitHub] incubator-geode issue #201: GEODE-1617: Regions can be created with a variet...

2016-07-14 Thread kirklund
Github user kirklund commented on the issue: https://github.com/apache/incubator-geode/pull/201 I'm running this change through precheckin. --- 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

[GitHub] incubator-geode pull request #200: GEODE-1660: Fix Javadoc and Compilation W...

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