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

2016-08-05 Thread Anthony Baker
+1 * Verified she’s * Verified signatures * Verified tag signature * Build from tag * Build and run from src distro * Checked src distro for binaries * Run from binary distro * Run some examples from mvn repo Anthony > On Aug 5, 2016, at 10:34 AM, Jason Huynh wrote: > > +1

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

2016-08-05 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #392 was successful. --- Scheduled 1425 tests in total. https://build.spring.io/browse/SGF-NAG-392/ -- This

[GitHub] incubator-geode pull request #228: GEODE-1673: Use security.json as default

2016-08-05 Thread gracemeilen
GitHub user gracemeilen opened a pull request: https://github.com/apache/incubator-geode/pull/228 GEODE-1673: Use security.json as default You can merge this pull request into a Git repository by running: $ git pull https://github.com/gracemeilen/incubator-geode

Re: Review Request 50856: GEODE-225 excessive CPU utilization and garbage collection strain for JSON processing

2016-08-05 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50856/#review144963 --- Fix it, then Ship it! This is just a suggestion. I didn't see

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

2016-08-05 Thread Jason Huynh
+1 - built from source distribution - started locator, server, listed members and created regions in gfsh from binary - started locator and server from source built gfsh On Fri, Aug 5, 2016 at 9:58 AM Dan Smith wrote: > +1 > > Verified > * Successful precheckin run of this

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

2016-08-05 Thread Dan Smith
+1 Verified * Successful precheckin run of this release - https://builds.apache.org/job/ Geode-release/24/ * Signatures * Basic gfsh commands with binary dist * Built from source dist * Basic CRUD test with maven artifacts * No jars in source dist -Dan On Thu, Aug 4, 2016 at 6:02 PM, William

Re: Review Request 50855: Geode-1569: require principal to be Serializable so that post process can happen in a remote server

2016-08-05 Thread Jinmei Liao
> On Aug. 5, 2016, 4:51 p.m., Kirk Lund wrote: > > geode-core/src/main/java/org/apache/geode/security/PostProcessor.java, line > > 52 > > > > > > Don't we want this to be a new > >

Re: Review Request 50855: Geode-1569: require principal to be Serializable so that post process can happen in a remote server

2016-08-05 Thread Jinmei Liao
On Aug. 5, 2016, 4:32 p.m., Jinmei Liao wrote: > > In your comment you said that you wanted to require that the principal was > > Serializable. But did you also intend to not require it to be an instanceof > > Principal? Yes, It does not need to be an instance of Principal. It could be a

Re: Review Request 50855: Geode-1569: require principal to be Serializable so that post process can happen in a remote server

2016-08-05 Thread Kirk Lund
> On Aug. 5, 2016, 4:51 p.m., Kirk Lund wrote: > > geode-core/src/main/java/org/apache/geode/security/PostProcessor.java, line > > 24 > > > > > > This should be reworded to not use the term "customer". Maybe use >

Re: Review Request 49962: Removed extra fields from distributedmember while serialization/de

2016-08-05 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49962/ --- (Updated Aug. 5, 2016, 4:52 p.m.) Review request for geode, Bruce Schuchardt

Re: Review Request 50855: Geode-1569: require principal to be Serializable so that post process can happen in a remote server

2016-08-05 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50855/#review144948 ---

Review Request 50858: GEODE-1675: Modifying tc server templates and adding the tomcat 8 jars to the tc server distributions

2016-08-05 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50858/ --- Review request for geode, Jens Deppe and William Markito. Repository: geode

Re: Review Request 50855: Geode-1569: require principal to be Serializable so that post process can happen in a remote server

2016-08-05 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50855/#review144943 ---

Re: Review Request 50856: GEODE-225 excessive CPU utilization and garbage collection strain for JSON processing

2016-08-05 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50856/#review144941 --- how about keeping ObjectMapper in threadLocal? - Hitesh Khamesra

Review Request 50856: GEODE-225 excessive CPU utilization and garbage collection strain for JSON processing

2016-08-05 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50856/ --- Review request for geode, Hitesh Khamesra, Jianxia Chen, and Udo Kohlmeyer.

[GitHub] incubator-geode pull request #227: Rename servlet URLs from gemfire to geode

2016-08-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/227 --- 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 50855: Geode-1569: require principal to be Serializable so that post process can happen in a remote server

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