Re: Internal statistics classes package

2016-08-04 Thread Kirk Lund
Note: com.gemstone.gemfire.internal.statistics already exists. I would like to eventually start on a new and improved Statistics API users in org.apache.geode.statistics and then once again move the internal classes to org.apache.geode.statistics.internal. -Kirk On Thursday, August 4, 2016, Kirk

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

2016-08-04 Thread William Markito
All, This is the seventh release candidate Apache Geode, version 1.0.0-incubating.M3. We're including the feedback received in RC6 including a fix (83f97ceef52febf92ef7737726548aa0865c1a59) to run REST API tests. Thanks to all the community members to drive towards this milestone! It fixes the

Re: Internal statistics classes package

2016-08-04 Thread Darrel Schneider
+1 On Thu, Aug 4, 2016 at 4:35 PM, Dan Smith wrote: > +1 > > -Dan > > On Thu, Aug 4, 2016 at 3:00 PM, Kirk Lund wrote: > > > I'm working on a statistics related issue. While I'm working on this > code, > > I'd like to move all of the internal statistics classes that are in > > com.gemstone.gemf

Re: Internal statistics classes package

2016-08-04 Thread Dan Smith
+1 -Dan On Thu, Aug 4, 2016 at 3:00 PM, Kirk Lund wrote: > I'm working on a statistics related issue. While I'm working on this code, > I'd like to move all of the internal statistics classes that are in > com.gemstone.gemfire.internal into > the com.gemstone.gemfire.internal.statistics pkg. >

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

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

Internal statistics classes package

2016-08-04 Thread Kirk Lund
I'm working on a statistics related issue. While I'm working on this code, I'd like to move all of the internal statistics classes that are in com.gemstone.gemfire.internal into the com.gemstone.gemfire.internal.statistics pkg. I would leave component specific stats in their component pkgs. Exampl

Re: Review Request 50615: GEODE-1676: Multiple Not Equals in query with Compact Range Index returns incorrect results

2016-08-04 Thread nabarun nag
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50615/#review144832 --- Ship it! Ship It! - nabarun nag On July 29, 2016, 9:16 p.m.,

Re: Review Request 50818: GEODE-1728: Removing the dispatcher elements from the session filter

2016-08-04 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50818/#review144822 --- Fix it, then Ship it! extensions/geode-modules-session/src/tes

[GitHub] incubator-geode issue #224: GEODE-1648: Introduce security-enabled-component...

2016-08-04 Thread kirklund
Github user kirklund commented on the issue: https://github.com/apache/incubator-geode/pull/224 1) we need to fix the javadocs for security-enabled-components on DistributedSystem class 2) move SecurableComponent enum into internal.security pkg 3) consider collapsing SecurableC

Review Request 50818: GEODE-1728: Removing the dispatcher elements from the session filter

2016-08-04 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50818/ --- Review request for geode and Jason Huynh. Repository: geode Description -

Fixed: apache/incubator-geode#974 (develop - 91e234f)

2016-08-04 Thread Travis CI
Build Update for apache/incubator-geode - Build: #974 Status: Fixed Duration: 14 minutes and 43 seconds Commit: 91e234f (develop) Author: Darrel Schneider Message: GEODE-1718: fix replace on overflowed entry When replace checks the current value for equality w

Fixed: apache/incubator-geode#975 (develop - 8897b87)

2016-08-04 Thread Travis CI
Build Update for apache/incubator-geode - Build: #975 Status: Fixed Duration: 13 minutes and 20 seconds Commit: 8897b87 (develop) Author: Olivier Mallassi Message: Fix e.printStackTrace() in ClientPartitionAdvisor the exception was logged using e.printstacktra

Fixed: apache/incubator-geode#973 (develop - ccb4fd0)

2016-08-04 Thread Travis CI
Build Update for apache/incubator-geode - Build: #973 Status: Fixed Duration: 13 minutes and 12 seconds Commit: ccb4fd0 (develop) Author: Kirk Lund Message: Fix apache license headers View the changeset: https://github.com/apache/incubator-geode/compare/88247

Fixed: apache/incubator-geode#977 (develop - c7a0a45)

2016-08-04 Thread Travis CI
Build Update for apache/incubator-geode - Build: #977 Status: Fixed Duration: 13 minutes and 57 seconds Commit: c7a0a45 (develop) Author: Bruce Schuchardt Message: GEODE-1727 NPE in JGroups during shutdown Moved the shutdown of the JGroups timer to before supe

Fixed: apache/incubator-geode#976 (develop - 64c3c62)

2016-08-04 Thread Travis CI
Build Update for apache/incubator-geode - Build: #976 Status: Fixed Duration: 13 minutes and 36 seconds Commit: 64c3c62 (develop) Author: Kirk Lund Message: GEODE-1712: fix AnalyzeSerializablesJUnitTest for GEODE-1712 changes View the changeset: https://githu

Re: Review Request 50811: GEODE-1241: Fixing the misspelt words in the geode-wan

2016-08-04 Thread Dave Barnes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50811/#review144814 --- I found a few possible problems, but need a knowledgable developer

Re: Flaky tests failing with BindException

2016-08-04 Thread Bruce Schuchardt
I've pushed the port-range changes that I described in my last email on this subject. Le 8/1/2016 à 5:33 PM, Kirk Lund a écrit : I think that the changes mentioned by Jens and Bruce obviate the need to do what I was proposing. -Kirk On Fri, Jul 29, 2016 at 3:41 PM, Bruce Schuchardt wrote:

Re: Review Request 50811: GEODE-1241: Fixing the misspelt words in the geode-wan

2016-08-04 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50811/#review144791 --- Ship it! Ship It! geode-wan/src/main/java/com/gemstone/gemfir

AnalyzeSerializablesJUnitTest

2016-08-04 Thread Kirk Lund
I broke AnalyzeSerializablesJUnitTest in my last checkin and I'm committing a fix now. -Kirk

Review Request 50811: GEODE-1241: Fixing the misspelt words in the geode-wan

2016-08-04 Thread nabarun nag
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50811/ --- Review request for geode, anilkumar gingade, Barry Oglesby, Dave Barnes, Jason H

[GitHub] incubator-geode pull request #225: Fix e.printStackTrace() in ClientPartitio...

2016-08-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/225 --- 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 featur

[GitHub] incubator-geode issue #225: Fix e.printStackTrace() in ClientPartitionAdviso...

2016-08-04 Thread upthewaterspout
Github user upthewaterspout commented on the issue: https://github.com/apache/incubator-geode/pull/225 +1 I'll merge this in. --- 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 enab

[GitHub] incubator-geode pull request #226: GEODE-1712: introduce SecurityService int...

2016-08-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/226 --- 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 featur

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

2016-08-04 Thread jinmeiliao
Github user jinmeiliao commented on the issue: https://github.com/apache/incubator-geode/pull/227 +1. I'll pull this in. --- 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 a

Re: Review Request 50798: GEODE-1727 NPE in JGroups during shutdown

2016-08-04 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50798/#review144774 --- Ship it! Ship It! - Hitesh Khamesra On Aug. 4, 2016, 3:44 p.

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

2016-08-04 Thread kjduling
GitHub user kjduling opened a pull request: https://github.com/apache/incubator-geode/pull/227 Rename servlet URLs from gemfire to geode Renamed gemfire-api/v1 to geode/v1 Renamed gemfire/v1 to geode-mgmt/v1 Verified old context paths still work, so both can be used for a tim

Review Request 50798: GEODE-1727 NPE in JGroups during shutdown

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

Jenkins build is back to normal : Geode-nightly #550

2016-08-04 Thread Apache Jenkins Server
See

Jenkins build is back to normal : Geode-release #24

2016-08-04 Thread Apache Jenkins Server
See