Re: Review Request 37066: Fix for GEODE-110 (DistributedTransactionDUnitTest.testPutAllWithTransactions failed with CommitIncompleteException)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37066/#review94000 --- Ship it! Ship It! - vivek bhaskar On Aug. 4, 2015, 6:18 a.m., Sh

Review Request 37066: Fix for GEODE-110 (DistributedTransactionDUnitTest.testPutAllWithTransactions failed with CommitIncompleteException)

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

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #24 was SUCCESSFUL (with 1000 tests). Change made by John Blum .

--- Spring Data GemFire > Nightly-ApacheGeode > #24 was successful. --- Scheduled with changes by John Blum . 1004 tests in total. https://build.spring.io/browse

Re: Review Request 36957: GEODE-181: Add new FastLogger tests and fix isDebug/TraceEnabled checks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36957/#review93977 --- Ship it! Ship It! - Darrel Schneider On Aug. 3, 2015, 1:34 p.m.,

Re: Geode Clubhouse August 4th

Hello Community, Just a reminder to join us for tomorrows Clubhouse at 9AM PDT at : http://bit.ly/1JSmjeN On Fri, Jul 31, 2015 at 12:33 PM, Marie Nedich wrote: > Dear Apache Geode Community, > > The next meeting of the Apache Geode (incubating) community is next > Tuesday. > > Apache Geode & Do

Re: Review Request 36957: GEODE-181: Add new FastLogger tests and fix isDebug/TraceEnabled checks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36957/ --- (Updated Aug. 3, 2015, 8:34 p.m.) Review request for geode and Darrel Schneider

Re: Review Request 36957: GEODE-181: Add new FastLogger tests and fix isDebug/TraceEnabled checks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36957/ --- (Updated Aug. 3, 2015, 8:32 p.m.) Review request for geode and Darrel Schneider

Re: Review Request 36957: GEODE-181: Add new FastLogger tests and fix isDebug/TraceEnabled checks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36957/#review93956 --- gemfire-core/src/main/java/com/gemstone/gemfire/internal/logging/Lo

Re: Review Request 36957: GEODE-181: Add new FastLogger tests and fix isDebug/TraceEnabled checks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36957/ --- (Updated Aug. 3, 2015, 7:58 p.m.) Review request for geode and Darrel Schneider

Re: Review Request 36950: [GEODE-177] Adding Dockerfile to Geode codebase

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36950/#review93916 --- Ship it! Ship It! - Kirk Lund On July 31, 2015, 12:13 a.m., Will

Re: Functional spec review

Minor nit: the spec should be referenced in the JIRA, Thanks, Bob On 8/3/15, 5:35 AM, "Tushar Khairnar" wrote: >This is to address https://issues.apache.org/jira/browse/GEODE-17 > >The functional spec is here: >https://cwiki.apache.org/confluence/display/GEODE/Integrated+Security > >regards,

Is REST APIs test coverage missing in geode codebase?

Hello, With the Geode code base, i can see REST APIs code base but do not find any test coverage for it (Junit/Dunits). Have we missed to move REST APIs test coverage to geode open source or am i missing something here. I prefer to add this test coverage so one can make sure their change are as ex

Functional spec review

This is to address https://issues.apache.org/jira/browse/GEODE-17 The functional spec is here: https://cwiki.apache.org/confluence/display/GEODE/Integrated+Security regards, Tushar

Re: Review Request 36936: GEODE-172 DistTXPersistentDebugDUnitTest.testTXPRRR2_putall fail intermittently because cache already exists

> On July 31, 2015, 4:47 p.m., Darrel Schneider wrote: > > gemfire-core/src/test/java/com/gemstone/gemfire/disttx/DistTXDebugDUnitTest.java, > > line 65 > > > > > > It is not clear to me why: > > 1. you are closi

Re: Review Request 36936: GEODE-172 DistTXPersistentDebugDUnitTest.testTXPRRR2_putall fail intermittently because cache already exists

> On July 31, 2015, 4:35 p.m., Darrel Schneider wrote: > > gemfire-core/src/test/java/com/gemstone/gemfire/disttx/DistributedTransactionDUnitTest.java, > > line 75 > > > > > > I know you are commenting this out but m

Re: Review Request 36936: GEODE-172 DistTXPersistentDebugDUnitTest.testTXPRRR2_putall fail intermittently because cache already exists

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36936/ --- (Updated Aug. 3, 2015, 12:19 p.m.) Review request for geode, Darrel Schneider a

Build failed in Jenkins: Geode-nightly #176

See -- [...truncated 213 lines...] :gemfire-core:processJcaResources UP-TO-DATE :gemfire-core:jcaClasses :gemfire-core:jar :gemfire-assembly:compileJava UP-TO-DATE :gemfire-assembly:processResources UP-TO-DA