Re: Review Request 46705: Integrated Security for JMX and commands

2016-04-28 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46705/#review131046 --- geode-core/src/main/java/com/gemstone/gemfire/management/internal

1 urgent upgrade

2016-04-28 Thread Raciti, Dominic
Dear customer, You have 1 urgent upgrade Message from PAYPAL. In order to update it please sign on www paypal.com to upgrade you account. Paypal-Upgrade Access Temporarily Limited Security

Please review "Integrated Security for JMX and commands"

2016-04-28 Thread Kirk Lund
Please review "Integrated Security for JMX and commands" at https://reviews.apache.org/r/46705/ We another review or two so we can get this merged tomorrow! Thanks, Kirk

Re: Review Request 46705: Integrated Security for JMX and commands

2016-04-28 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46705/#review131006 --- Fix it, then Ship it! Fix issues and then ship it! geode-juni

Re: Review Request 46402: GEODE-1236 GEODE-1248: Fix gfsh sutdown call

2016-04-28 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46402/#review130993 --- Ship it! Ship It! - Jinmei Liao On April 20, 2016, 1:56 p.m.

Review Request 46801: GEDOE-1321 GMSHealthMonitor: java.lang.IllegalStateException: Timer already cancelled

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

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #293 has FAILED

2016-04-28 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #293 failed. --- Scheduled No failed tests found, a possible compilation error. https://build.spring.io/browse/SG

Re: Upgrading Gradle to version 2.12

2016-04-28 Thread Mark Bretl
+1 Thanks Jason! --Mark On Wed, Apr 27, 2016 at 5:22 PM, William Markito wrote: > +1 > > On Wed, Apr 27, 2016 at 10:41 AM, Jason Huynh wrote: > > > Hello Geode dev list, > > > > As a heads up, I am about to check in a change to upgrade the Gradle > > version to 2.12. This will address GEODE-1

Re: Review Request 46705: Integrated Security for JMX and commands

2016-04-28 Thread Jinmei Liao
> On April 28, 2016, 4:45 p.m., Kirk Lund wrote: > > geode-core/src/main/java/com/gemstone/gemfire/management/DistributedSystemMXBean.java, > > line 78 > > > > > > Should this be deleted or uncommented? This is a te

Re: Review Request 46705: Integrated Security for JMX and commands

2016-04-28 Thread Jinmei Liao
> On April 28, 2016, 5:02 p.m., Kirk Lund wrote: > > geode-core/src/main/java/com/gemstone/gemfire/management/internal/security/MBeanServerWrapper.java, > > line 244 > > > > > > Should this include e as a cause? > >

Re: Review Request 46705: Integrated Security for JMX and commands

2016-04-28 Thread Jinmei Liao
> On April 28, 2016, 5:42 p.m., Kirk Lund wrote: > > geode-core/src/main/java/com/gemstone/gemfire/security/ShiroUtil.java, line > > 18 > > > > > > This is an external public API package. ShiroUtil needs to move to a

Re: Review Request 46705: Integrated Security for JMX and commands

2016-04-28 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46705/#review130956 --- geode-core/src/main/java/com/gemstone/gemfire/security/ShiroUtil.

Re: Review Request 46705: Integrated Security for JMX and commands

2016-04-28 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46705/#review130938 --- geode-core/src/main/java/com/gemstone/gemfire/management/internal

Re: Review Request 46705: Integrated Security for JMX and commands

2016-04-28 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46705/#review130937 --- geode-core/src/main/java/com/gemstone/gemfire/management/Distribu

Re: Review Request 46705: Integrated Security for JMX and commands

2016-04-28 Thread Jinmei Liao
> On April 28, 2016, 4:28 p.m., Kirk Lund wrote: > > geode-core/src/main/java/com/gemstone/gemfire/internal/security/AuthorizeRequest.java, > > line 112 > > > > > > Why do we need new method for the old logging? t

Re: Review Request 46705: Integrated Security for JMX and commands

2016-04-28 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46705/#review130933 --- geode-core/src/main/java/com/gemstone/gemfire/internal/security/A

Jenkins build is back to normal : Geode-spark-connector #19

2016-04-28 Thread Apache Jenkins Server
See