Re: New Committer And PMC Member: Joey McAllister

2017-07-25 Thread Deepak Dixit
Welcome Joey! - Deepak On Wed, Jul 26, 2017 at 11:26 AM, Amey Barve wrote: > Welcome Joey and Congratulations! > > Regards, > Amey > > On Wed, Jul 26, 2017 at 1:08 AM, Joey McAllister > wrote: > > > Thanks, everyone! > > > > [drum solos; throws sticks to Fred] > > > > > > On Tue, Jul 25, 2017

Re: New Committer And PMC Member: Joey McAllister

2017-07-25 Thread Amey Barve
Welcome Joey and Congratulations! Regards, Amey On Wed, Jul 26, 2017 at 1:08 AM, Joey McAllister wrote: > Thanks, everyone! > > [drum solos; throws sticks to Fred] > > > On Tue, Jul 25, 2017 at 12:36 PM Nitin Lamba wrote: > > > Welcome Joey! > > > > On Tue, Jul 25, 2017 at 12:21 PM, Fred Krone

[GitHub] geode pull request #609: GEODE-2886 : sent IllegalStateException instead of ...

2017-07-25 Thread ameybarve15
Github user ameybarve15 commented on a diff in the pull request: https://github.com/apache/geode/pull/609#discussion_r129480518 --- Diff: geode-lucene/src/test/java/org/apache/geode/cache/lucene/LuceneQueriesIntegrationTest.java --- @@ -290,6 +295,24 @@ public void queryJsonObject

Re: Review Request 61114: GEODE-3296: Speed up Acceptance Tests

2017-07-25 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61114/#review181430 --- Ship it! Ship It! - Jens Deppe On July 25, 2017, 4:58 p.m.,

[GitHub] geode pull request #658: GEODE-3315: Replaced PreferBytes... with VMCachedDe...

2017-07-25 Thread upthewaterspout
GitHub user upthewaterspout opened a pull request: https://github.com/apache/geode/pull/658 GEODE-3315: Replaced PreferBytes... with VMCachedDeserializable When getting a HAEventWrapper as part of a GII, make sure that we store the wrapper in a VMCachedDeserializable. This object

[GitHub] geode pull request #515: GEODE-240: Remove deprecated methods on Transaction...

2017-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/515 --- 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 feature is enabl

[GitHub] geode pull request #654: GEODE-3291: add soundex analyzer into lucene query

2017-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/654 --- 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 feature is enabl

[GitHub] geode pull request #130: UDA functionality for OQL engine of Geode

2017-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/130 --- 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 feature is enabl

[GitHub] geode pull request #29: [GEODE-252] Remove deprecated PartitionAttributes me...

2017-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/29 --- 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 feature is enable

[GitHub] geode pull request #535: GEODE-2986: Remove redundant log message

2017-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/535 --- 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 feature is enabl

[GitHub] geode pull request #657: GEODE-3286: Failing to cleanup connections from Con...

2017-07-25 Thread WireBaron
GitHub user WireBaron opened a pull request: https://github.com/apache/geode/pull/657 GEODE-3286: Failing to cleanup connections from ConnectionTable recei… …ver table @kohlmu-pivotal @galen-pivotal @pivotal-amurmann @bschuchardt @hiteshk25 - prevent adding a c

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

2017-07-25 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #627 was successful. --- Scheduled 1989 tests in total. https://build.spring.io/browse/SGF-NAG-627/ -

[GitHub] geode pull request #656: GEODE-3312 Update doc gfsh list members output

2017-07-25 Thread karensmolermiller
GitHub user karensmolermiller opened a pull request: https://github.com/apache/geode/pull/656 GEODE-3312 Update doc gfsh list members output IP address is output, not host name. @joeymcallister @davebarnes97 Please review. You can merge this pull request into a Git reposit

[GitHub] geode issue #655: GEODE-3030: Set possibleDuplicate=true for all bucket even...

2017-07-25 Thread upthewaterspout
Github user upthewaterspout commented on the issue: https://github.com/apache/geode/pull/655 @boglesby @jhuynh1 --- 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 wishe

[GitHub] geode pull request #655: GEODE-3030: Set possibleDuplicate=true for all buck...

2017-07-25 Thread upthewaterspout
GitHub user upthewaterspout opened a pull request: https://github.com/apache/geode/pull/655 GEODE-3030: Set possibleDuplicate=true for all bucket events after failover ### For all changes: - [ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit messa

Re: Review Request 61119: GEODE-3097: GFSH works over HTTP with SSL

2017-07-25 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61119/#review181398 --- Ship it! Ship It! - Jared Stewart On July 25, 2017, 10:11 p.

Review Request 61119: GEODE-3097: GFSH works over HTTP with SSL

2017-07-25 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61119/ --- Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and Pat

Re: Revert gradle upgrade

2017-07-25 Thread Mark Bretl
Now that we have 1.2.0 out the door, would I be able to get someone to verify my fixes from those who were having trouble with Gradle 3.4.1 in their IDE? Thanks, --Mark On Wed, Jun 14, 2017 at 5:08 PM, Mark Bretl wrote: > I think I figured it out. > > Here is what I did: > Fix was to declare t

[GitHub] geode-native pull request #113: (no ticket) Capitalize C# client member func...

2017-07-25 Thread davebarnes97
GitHub user davebarnes97 opened a pull request: https://github.com/apache/geode-native/pull/113 (no ticket) Capitalize C# client member functions From Kyle Dunn on Slack: Looks like a minor typo here : http://gemfire-native-90.docs.pivotal.io/native/continuous-querying/5b-writin

Re: Review Request 60875: GEODE-2654: Backups can capture different members from different points in time

2017-07-25 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60875/#review181388 --- Ship it! Ship It! - Darrel Schneider On July 25, 2017, 12:03

Re: Review Request 60924: GEODE-2920: added security tests for create diskstore and create persistent region.

2017-07-25 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60924/#review181380 --- geode-core/src/test/java/org/apache/geode/management/internal/sec

[GitHub] geode pull request #609: GEODE-2886 : sent IllegalStateException instead of ...

2017-07-25 Thread jhuynh1
Github user jhuynh1 commented on a diff in the pull request: https://github.com/apache/geode/pull/609#discussion_r129418949 --- Diff: geode-lucene/src/test/java/org/apache/geode/cache/lucene/LuceneQueriesIntegrationTest.java --- @@ -290,6 +295,24 @@ public void queryJsonObject() t

Re: New Committer And PMC Member: Joey McAllister

2017-07-25 Thread Joey McAllister
Thanks, everyone! [drum solos; throws sticks to Fred] On Tue, Jul 25, 2017 at 12:36 PM Nitin Lamba wrote: > Welcome Joey! > > On Tue, Jul 25, 2017 at 12:21 PM, Fred Krone wrote: > > > Drum solo! > > > > On Tue, Jul 25, 2017 at 12:05 PM, Anthony Baker > wrote: > > > > > Welcome Joey! > > > >

Re: New Committer And PMC Member: Joey McAllister

2017-07-25 Thread Nitin Lamba
Welcome Joey! On Tue, Jul 25, 2017 at 12:21 PM, Fred Krone wrote: > Drum solo! > > On Tue, Jul 25, 2017 at 12:05 PM, Anthony Baker wrote: > > > Welcome Joey! > > > > > On Jul 25, 2017, at 11:58 AM, Mark Bretl wrote: > > > > > > The Apache Geode Project Management Committee has invited Joey > >

Re: New Committer And PMC Member: Joey McAllister

2017-07-25 Thread Fred Krone
Drum solo! On Tue, Jul 25, 2017 at 12:05 PM, Anthony Baker wrote: > Welcome Joey! > > > On Jul 25, 2017, at 11:58 AM, Mark Bretl wrote: > > > > The Apache Geode Project Management Committee has invited Joey > McAllister to > > be committer on the project and join the Geode PMC. We are pleased t

[GitHub] geode pull request #580: GEODE-2936: Refactoring OrderByComparator

2017-07-25 Thread YehEmily
Github user YehEmily commented on a diff in the pull request: https://github.com/apache/geode/pull/580#discussion_r129399451 --- Diff: geode-core/src/test/java/org/apache/geode/cache/query/internal/OrderByComparatorJUnitTest.java --- @@ -173,36 +157,58 @@ public void testUnsupport

[GitHub] geode pull request #580: GEODE-2936: Refactoring OrderByComparator

2017-07-25 Thread YehEmily
Github user YehEmily commented on a diff in the pull request: https://github.com/apache/geode/pull/580#discussion_r129399374 --- Diff: geode-core/src/test/java/org/apache/geode/cache/query/internal/OrderByComparatorJUnitTest.java --- @@ -173,36 +157,58 @@ public void testUnsupport

[GitHub] geode pull request #580: GEODE-2936: Refactoring OrderByComparator

2017-07-25 Thread pdxrunner
Github user pdxrunner commented on a diff in the pull request: https://github.com/apache/geode/pull/580#discussion_r129397800 --- Diff: geode-core/src/test/java/org/apache/geode/cache/query/internal/OrderByComparatorJUnitTest.java --- @@ -173,36 +157,58 @@ public void testUnsuppor

Re: New Committer And PMC Member: Joey McAllister

2017-07-25 Thread Anthony Baker
Welcome Joey! > On Jul 25, 2017, at 11:58 AM, Mark Bretl wrote: > > The Apache Geode Project Management Committee has invited Joey McAllister to > be committer on the project and join the Geode PMC. We are pleased to > announce he has accepted. > > Please join me in welcoming Joey! > > Best re

Re: Review Request 60875: GEODE-2654: Backups can capture different members from different points in time

2017-07-25 Thread Lynn Gallinat
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60875/ --- (Updated July 25, 2017, 7:03 p.m.) Review request for geode, anilkumar gingade,

[GitHub] geode pull request #580: GEODE-2936: Refactoring OrderByComparator

2017-07-25 Thread pdxrunner
Github user pdxrunner commented on a diff in the pull request: https://github.com/apache/geode/pull/580#discussion_r129395826 --- Diff: geode-core/src/test/java/org/apache/geode/cache/query/internal/OrderByComparatorJUnitTest.java --- @@ -173,36 +157,58 @@ public void testUnsuppor

Re: New Committer And PMC Member: Joey McAllister

2017-07-25 Thread Dave Barnes
Welcome, Joey, to our very exclusive club. You'll especially enjoy the monthly picnics! On Tue, Jul 25, 2017 at 11:58 AM, Michael William Dodge wrote: > Welcome, Joey! > > Sarge > > > On 25 Jul, 2017, at 11:58, Mark Bretl wrote: > > > > The Apache Geode Project Management Committee has invited

Re: New Committer And PMC Member: Joey McAllister

2017-07-25 Thread Michael William Dodge
Welcome, Joey! Sarge > On 25 Jul, 2017, at 11:58, Mark Bretl wrote: > > The Apache Geode Project Management Committee has invited Joey McAllister to > be committer on the project and join the Geode PMC. We are pleased to > announce he has accepted. > > Please join me in welcoming Joey! > > Be

New Committer And PMC Member: Joey McAllister

2017-07-25 Thread Mark Bretl
The Apache Geode Project Management Committee has invited Joey McAllister to be committer on the project and join the Geode PMC. We are pleased to announce he has accepted. Please join me in welcoming Joey! Best regards, Mark On behalf of the Apache Geode PMC

[GitHub] geode pull request #642: GEODE-3031: Extracting startLocator and startServer...

2017-07-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/642 --- 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 feature is enabl

Re: Review Request 61114: GEODE-3296: Speed up Acceptance Tests

2017-07-25 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61114/#review181365 --- Ship it! Ship It! - Patrick Rhomberg On July 25, 2017, 4:58

Re: Review Request 61114: GEODE-3296: Speed up Acceptance Tests

2017-07-25 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61114/#review181360 --- Ship it! +1 - Kirk Lund On July 25, 2017, 4:58 p.m., Jared S

GMSJoinLeaveJUnitTest failing on develop

2017-07-25 Thread Kirk Lund
Is someone working on this test? It fails consistently for me on develop... ./gradlew -DintegrationTest.single=GMSJoinLeaveJUnitTest geode-core:integrationTest org.apache.geode.distributed.internal.membership.gms.membership.GMSJoinLeaveJUnitTest > testViewNotSentWhenShuttingDown FAILED java.l

Re: Review Request 61114: GEODE-3296: Speed up Acceptance Tests

2017-07-25 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61114/#review181359 --- Ship it! Ship It! - Anthony Baker On July 25, 2017, 4:58 p.m

Review Request 61114: GEODE-3296: Speed up Acceptance Tests

2017-07-25 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61114/ --- Review request for geode, Anthony Baker, Emily Yeh, Jens Deppe, Jared Stewart, K

Re: Need to know complete execution time for geode-core Junit tests

2017-07-25 Thread Dan Smith
What target are you running? It's a bit confusing because there are actually 3 different sets of tests. These times a really rough because I don't have a good run in front of me, but this should give you an idea. "True" unit tests, runs in 1-2 minutes like Jens Said: ./gradlew geode-core:test Sin

[GitHub] geode-native pull request #112: GEODE-3173: Upgrade to gtest 1.8

2017-07-25 Thread dgkimura
Github user dgkimura closed the pull request at: https://github.com/apache/geode-native/pull/112 --- 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 feature

Re: Need to know complete execution time for geode-core Junit tests

2017-07-25 Thread Anthony Baker
You might want to get a few thread dumps to see if there is a hung test. Also, make sure you have sufficient RAM. Anthony > On Jul 25, 2017, at 1:13 AM, Dinesh Akhand wrote: > > Hi Team, > > I trigger the geode-core Junit tests . it keeping running from last 3 hour. > Can you please let me k

[GitHub] geode-native issue #109: GEODE-3173: Upgrade to gtest 1.8

2017-07-25 Thread dgkimura
Github user dgkimura commented on the issue: https://github.com/apache/geode-native/pull/109 echobravopapa requested this PR to come off a feature branch. There is a duplicate PR from a feature branch here: https://github.com/apache/geode-native/pull/112 --- If your project is set

[GitHub] geode-native pull request #109: GEODE-3173: Upgrade to gtest 1.8

2017-07-25 Thread dgkimura
Github user dgkimura closed the pull request at: https://github.com/apache/geode-native/pull/109 --- 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 feature

[GitHub] geode-native pull request #112: GEODE-3173: Upgrade to gtest 1.8

2017-07-25 Thread dgkimura
GitHub user dgkimura opened a pull request: https://github.com/apache/geode-native/pull/112 GEODE-3173: Upgrade to gtest 1.8 Upgrade to gtest 1.8 which also gives us gmock. You can merge this pull request into a Git repository by running: $ git pull https://github.com/dgkimura/

Re: Need to know complete execution time for geode-core Junit tests

2017-07-25 Thread Jens Deppe
Hi Dinesh, When I run '*./gradlew clean geode-core:test*', it takes about 1:30m on an iMac. How are you invoking the build? You can see the progress by tailing the file: geode-core/build/test/test-progress.test. --Jens On Tue, Jul 25, 2017 at 1:13 AM, Dinesh Akhand wrote: > Hi Team, > > I tr

[GitHub] geode pull request #609: GEODE-2886 : sent IllegalStateException instead of ...

2017-07-25 Thread ameybarve15
Github user ameybarve15 commented on a diff in the pull request: https://github.com/apache/geode/pull/609#discussion_r129300137 --- Diff: geode-lucene/src/test/java/org/apache/geode/cache/lucene/LuceneQueriesIntegrationTest.java --- @@ -290,6 +295,24 @@ public void queryJsonObject

Build failed in Jenkins: Geode-nightly-flaky #75

2017-07-25 Thread Apache Jenkins Server
See Changes: [jstewart] GEODE-3214: Remove support for multistep gfsh commands. [dbarnes] (no ticket) Update OSX to macOS [dschneider] GEODE-3049: Add warning on loss of all buckets in region

Need to know complete execution time for geode-core Junit tests

2017-07-25 Thread Dinesh Akhand
Hi Team, I trigger the geode-core Junit tests . it keeping running from last 3 hour. Can you please let me know how much time it take completion of Junit test in Geode-core. I am using geode 1.1.1 Thanks, Dinesh Akhand This message and the information contained herein is proprietary and confid