Build failed in Jenkins: Geode-release #31

2016-10-07 Thread Apache Jenkins Server
See Changes: [jiliao] GEODE-1570: upgrade spring libraries -- [...truncated 634 lines...] :geode-assembly:processTestResources :geode-assembly:testClasses :geode-assembly:checkMissedTests

Re: Review Request 52655: GEODE-1973: add more tests to cover GMSAuthenticator and SimpleSecurityManager

2016-10-07 Thread Jinmei Liao
> On Oct. 7, 2016, 11:21 p.m., Kirk Lund wrote: > > geode-core/src/main/java/org/apache/geode/security/templates/SimpleSecurityManager.java, > > line 9 > > > > > > This class in the external packages of src/main. If

Review Request 52655: GEODE-1973: add more tests to cover GMSAuthenticator and SimpleSecurityManager

2016-10-07 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52655/ --- Review request for geode, Jared Stewart, Kevin Duling, and Kirk Lund.

Re: Review Request 52650: GEODE-1973: having GMSAuthenticator work on a locator with no cache.

2016-10-07 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52650/ --- (Updated Oct. 7, 2016, 10:09 p.m.) Review request for geode, Jared Stewart,

Re: Review Request 52650: GEODE-1973: having GMSAuthenticator work on a locator with no cache.

2016-10-07 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52650/#review151888 --- Ship it! Ship It! - Kirk Lund On Oct. 7, 2016, 10:09 p.m.,

Re: Review Request 52650: GEM-1032: having GMSAuthenticator work on a locator with no cache.

2016-10-07 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52650/ --- (Updated Oct. 7, 2016, 10:07 p.m.) Review request for geode, Jared Stewart,

Re: Review Request 52650: GEM-1032: having GMSAuthenticator work on a locator with no cache.

2016-10-07 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52650/#review151887 --- Ship it! Ship It! - Kirk Lund On Oct. 7, 2016, 10:02 p.m.,

Review Request 52650: GEM-1032: having GMSAuthenticator work on a locator with no cache.

2016-10-07 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52650/ --- Review request for geode. Repository: geode Description --- GEM-1032:

Re: Review Request 52619: GEODE-1967 Old key are not removed from the removedKeyValuesEntries after it was removed from Compact Range Indexes

2016-10-07 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52619/#review151855 ---

Fixed: apache/incubator-geode#1218 (develop - fbbdb82)

2016-10-07 Thread Travis CI
Build Update for apache/incubator-geode - Build: #1218 Status: Fixed Duration: 13 minutes and 19 seconds Commit: fbbdb82 (develop) Author: nabarun Message: Revert "GEODE-1926" * Precehckin completion pending This reverts commit

Passed: apache/incubator-geode#1216 (develop2 - e78aa68)

2016-10-07 Thread Travis CI
Build Update for apache/incubator-geode - Build: #1216 Status: Passed Duration: 12 minutes and 1 second Commit: e78aa68 (develop2) Author: nabarun Message: GEODE-136: Adding the Apache License header to the file. * This resolves the rat issue in

Re: Review Request 52641: GEODE-1914 removed old versions of dtds

2016-10-07 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52641/ --- (Updated Oct. 7, 2016, 7:13 p.m.) Review request for geode, Bruce Schuchardt

Re: Adding Apache License header to GetRegionsFunctionJUnitTest

2016-10-07 Thread Nabarun Nag
Hi Mark, Thank you, The branch is deleted. And the code was check into the develop branch On Fri, Oct 7, 2016 at 11:55 AM Mark Bretl wrote: > Hi Nabarun, > > It looks like you added to the 'develop2' branch, did you mean 'develop' > instead? > > --Mark > > On Fri, Oct 7,

Re: Review Request 52619: GEODE-1967 Old key are not removed from the removedKeyValuesEntries after it was removed from Compact Range Indexes

2016-10-07 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52619/#review151843 --- Ship it! Ship It! - Dan Smith On Oct. 7, 2016, 5:48 p.m.,

Re: Adding Apache License header to GetRegionsFunctionJUnitTest

2016-10-07 Thread Mark Bretl
Hi Nabarun, It looks like you added to the 'develop2' branch, did you mean 'develop' instead? --Mark On Fri, Oct 7, 2016 at 11:50 AM, Nabarun Nag wrote: > Hi, > > I am adding the license header to this file. Resolving rat issue. > > Regards > Naba >

Adding Apache License header to GetRegionsFunctionJUnitTest

2016-10-07 Thread Nabarun Nag
Hi, I am adding the license header to this file. Resolving rat issue. Regards Naba

Re: Hibernate module and Geode 1.0 ?

2016-10-07 Thread Jason Huynh
I've created a ticket (feel free to modify/reword/rename it). It's GEODE-1972: Move Geode Hibernate module to a feature branch On Thu, Oct 6, 2016 at 10:17 AM Dave Barnes wrote: So, what form should the JIRA ticket assume? (a) Move HIbernate from develop to a feature

Re: Review Request 52641: GEODE-1914 removed old versions of dtds

2016-10-07 Thread Jacob Barrett
If were are in there changing them all why wouldn't we just change them all to the latest xsd? On Fri, Oct 7, 2016 at 10:56 AM Hitesh Khamesra wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 52641: GEODE-1914 removed old versions of dtds

2016-10-07 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52641/#review151830 ---

[GitHub] incubator-geode pull request #255: Feature/geode 999

2016-10-07 Thread jaredjstewart
GitHub user jaredjstewart opened a pull request: https://github.com/apache/incubator-geode/pull/255 Feature/geode 999 Converted uiTests to run using PhantomJS instead of the Firefox web driver. You can merge this pull request into a Git repository by running: $ git pull

Review Request 52619: GEODE-1967 Old key are not removed from the removedKeyValuesEntries after it was removed from Compact Range Indexes

2016-10-07 Thread nabarun nag
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52619/ --- Review request for geode, Barry Oglesby, Jason Huynh, Dan Smith, and xiaojian

Review Request 52641: GEODE-1914 removed old versions of dtds

2016-10-07 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52641/ --- Review request for geode, Bruce Schuchardt and Udo Kohlmeyer. Repository:

Build failed in Jenkins: Geode-nightly #615

2016-10-07 Thread Apache Jenkins Server
See Changes: [bschuchardt] GEODE-1927 backward compatibility support [bschuchardt] GEODE-1931 CI Failure: LocatorUDPSecurityDUnitTest.testStartTwoLocators [bschuchardt] GEODE-1931 CI Failure:

[GitHub] incubator-geode pull request #253: GEODE-136: Fix possible NullPointerExcept...

2016-10-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/253 --- 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

[GitHub] incubator-geode issue #254: Feature/geode 999

2016-10-07 Thread jinmeiliao
Github user jinmeiliao commented on the issue: https://github.com/apache/incubator-geode/pull/254 Have we decided whether we want to put uiTest in precheckin or start a new pipeline? --- If your project is set up for it, you can reply to this email and have your reply appear on