Re: Review Request 37209: GEODE-17 : Integrated Security Code Merge

2015-08-11 Thread Tushar Khairnar
* If i would have adopted spring security for REST APIs, REST APIs security might not work with customer's existing security plugins. * - Not entirely true Neelkanth. But yeah we might have to change interface a bit little. For eg. HMAC will require User Secret which we dont really expose through

Re: Review Request 37209: GEODE-17 : Integrated Security Code Merge

2015-08-11 Thread Nilkanth Patel
Implementation is designed in the context of integrated security approach where user/customer needs to install/configure single Authentication and Authorization plugin and that should work for all channels that communicate with server. FYI, we have not changed anything to existing geode C/S securit

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #32 was SUCCESSFUL (with 1020 tests). Change made by John Blum , Oliver Gierke and Oliver Gierke.

2015-08-11 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #32 was successful. --- Scheduled with changes by John Blum , Oliver Gierke, Oliver Gierke. 1024 tests in total.

Re: Incubator PMC/Board report for Aug 2015 ([ppmc])

2015-08-11 Thread Niall Pemberton
On Wed, Aug 12, 2015 at 1:27 AM, Kirk Lund wrote: > We actually plan to continue with monthly reports because we think it's > valuable to continue monthly instead of quarterly, so stay tuned! > OK, thats good. But I don't recall seeing that discussed here. Please remember to bring any off-list d

Re: Incubator PMC/Board report for Aug 2015 ([ppmc])

2015-08-11 Thread Kirk Lund
We actually plan to continue with monthly reports because we think it's valuable to continue monthly instead of quarterly, so stay tuned! -Kirk On Tuesday, August 11, 2015, Niall Pemberton wrote: > On Mon, Jul 27, 2015 at 4:24 PM, Marvin Humphrey > wrote: > > > Greets, > > > > This report remi

Build failed in Jenkins: Geode-nightly #181

2015-08-11 Thread Apache Jenkins Server
See Changes: [sbawaskar] GEODE-191 fix for transactional region.size() [dschneider] GEODE-190: Fix ReflectionBasedAutoSerializer leak [dschneider] GEODE-179: fix race in testEntryTtlDestroyEvent [dschneider] GEODE-138: remove race condi

Re: Incubator PMC/Board report for Aug 2015 ([ppmc])

2015-08-11 Thread Niall Pemberton
On Mon, Jul 27, 2015 at 4:24 PM, Marvin Humphrey wrote: > Greets, > > This report reminder was sent in error and there is no need to report this > month. Geode has finished its three months of initial monthly reporting > and > is now on a quarterly schedule. > Only just saw this after seeing a

Review Request 37371: GEODE-195: Remove debug logging from CloseCacheAuthorization

2015-08-11 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37371/ --- Review request for geode, anilkumar gingade, Bruce Schuchardt, Jianxia Chen, Lyn

Review Request 37372: GEODE-196: Remove unneeded system out in RuntimeIterator

2015-08-11 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37372/ --- Review request for geode, anilkumar gingade, Bruce Schuchardt, Jianxia Chen, Lyn

Re: Review Request 36484: GEODE-105: Null value in Map causes NPE with Map Indexes

2015-08-11 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36484/#review94993 --- gemfire-core/src/test/java/com/gemstone/gemfire/cache/query/interna

Re: Review Request 36484: GEODE-105: Null value in Map causes NPE with Map Indexes

2015-08-11 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36484/#review94994 --- Ship it! Ship It! - anilkumar gingade On July 24, 2015, 7:16 p.m

Re: Review Request 37316: GEODE-116:CopyOnReadIndexDUnitTest.testPRQueryOnLocalNode fails assertion

2015-08-11 Thread Jason Huynh
> On Aug. 10, 2015, 10:28 p.m., Jianxia Chen wrote: > > Better put the link of the review on JIRA, or put the explanation on JIRA. > > So that it is easier to track and understand in the future. The commit message will include a similar description and the jira will automatically be updated wi