Re: New Committer And PMC Member: Joey McAllister

[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 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!
> > > > >
> > > > > > 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 regards,
> > > > > >
> > > > > > Mark
> > > > > > On behalf of the Apache Geode PMC
> > > > >
> > > > >
> > > >
> > >
> >
>



-- 
From:

Deepak D Dixit
deepakdixit2...@gmail.com
+919028507537

Re: New Committer And PMC Member: Joey McAllister

[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  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
> > > > 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 #609: GEODE-2886 : sent IllegalStateException instead of ...

[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() throws Exception {
   }
 
   @Test()
+  public void testWaitUntilFlushedForException() throws Exception {
+Map fields = new HashMap();
+fields.put("name", null);
+fields.put("lastName", null);
+fields.put("address", null);
+
luceneService.createIndexFactory().setFields(fields).create(INDEX_NAME, 
REGION_NAME);
+Region region = 
cache.createRegionFactory(RegionShortcut.PARTITION).create(REGION_NAME);
+final LuceneIndex index = luceneService.getIndex(INDEX_NAME, 
REGION_NAME);
+
+// This is to send IllegalStateException from WaitUntilFlushedFunction
+String nonCreatedIndex = "index2";
+
+boolean b =
+luceneService.waitUntilFlushed(nonCreatedIndex, REGION_NAME, 
6, TimeUnit.MILLISECONDS);
+assertFalse(b);
--- End diff --

@jhuynh1 
IllegalStateException thrown from the function executed on the server will 
not be propagated to the client, unless that exception is sent through 
resultSender.sendException() Api.
 IllegalStateException is only logged into server files in the case of 
throwing an exception.



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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., Jared Stewart wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61114/
> ---
> 
> (Updated July 25, 2017, 4:58 p.m.)
> 
> 
> Review request for geode, Anthony Baker, Emily Yeh, Jens Deppe, Jared 
> Stewart, Ken Howe, Kirk Lund, and Patrick Rhomberg.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-3296: Speed up Acceptance Tests
> 
> - Move the acceptanceTest task from the top-level module into the 
> :geode-assembly submodule.
> 
> 
> Diffs
> -
> 
>   geode-assembly/build.gradle 02c9e33a3a35c423ff3d6aa9b1e151a34aca0378 
>   gradle/test.gradle 741c64e8dc5d9965e03744b0929d7a701abb2456 
> 
> 
> Diff: https://reviews.apache.org/r/61114/diff/1/
> 
> 
> Testing
> ---
> 
> Precheckin running
> 
> 
> Thanks,
> 
> Jared Stewart
> 
>

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

[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 needs to have a
reference to the HAContainer. If PREFER_SERIALIZED is set to true, we
we using a PreferBytesSerializable which would always create new copy of
the HAEventWrapper.

Thank you for submitting a contribution to Apache Geode.

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

### For all changes:
- [ ] Is there a JIRA ticket associated with this PR? Is it referenced in 
the commit message?

- [ ] Has your PR been rebased against the latest commit within the target 
branch (typically `develop`)?

- [ ] Is your initial contribution a single, squashed commit?

- [ ] Does `gradlew build` run cleanly?

- [ ] Have you written or updated unit tests to verify your changes?

- [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?

### Note:
Please ensure that once the PR is submitted, you check travis-ci for build 
issues and
submit an update to your PR as soon as possible. If you need help, please 
send an
email to dev@geode.apache.org.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/upthewaterspout/incubator-geode 
feature/GEODE-3315

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/geode/pull/658.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #658


commit 68687f8f97bda1ba82ba1c192a6601b16ba7a67c
Author: Barry Oglesby 
Date:   2017-06-22T20:52:24Z

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 needs to have a
reference to the HAContainer. If PREFER_SERIALIZED is set to true, we
we using a PreferBytesSerializable which would always create new copy of
the HAEventWrapper.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 closed connection to the connection table's receivers
- add a new unit test for connection table
- adding connection factory object for creating receiving connections
- have the idle connection timeout ensure connections are removed from 
connection
  table receivers
- modify tcpConduit stat accesses to allow easier mocking

Signed-off-by: Hitesh Khamesra 

Thank you for submitting a contribution to Apache Geode.

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

### For all changes:
- [x] Is there a JIRA ticket associated with this PR? Is it referenced in 
the commit message?

- [x] Has your PR been rebased against the latest commit within the target 
branch (typically `develop`)?

- [x] Is your initial contribution a single, squashed commit?

- [x] Does `gradlew build` run cleanly?

- [x] Have you written or updated unit tests to verify your changes?

- [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?

### Note:
Please ensure that once the PR is submitted, you check travis-ci for build 
issues and
submit an update to your PR as soon as possible. If you need help, please 
send an
email to dev@geode.apache.org.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/WireBaron/geode feature/GEODE-3286

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/geode/pull/657.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #657


commit 8aed26846de6e9ff1c123acae98a7b5ce6d82a83
Author: Brian Rowe 
Date:   2017-07-25T22:43:35Z

GEODE-3286: Failing to cleanup connections from ConnectionTable receiver 
table

- prevent adding a closed connection to the connection table's receivers
- add a new unit test for connection table
- adding connection factory object for creating receiving connections
- have the idle connection timeout ensure connections are removed from 
connection
  table receivers
- modify tcpConduit stat accesses to allow easier mocking

Signed-off-by: Hitesh Khamesra 




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[Spring CI]

---
Spring Data GemFire > Nightly-ApacheGeode > #627 was successful.
---
Scheduled
1989 tests in total.

https://build.spring.io/browse/SGF-NAG-627/




--
Code Changes
--
Oliver Gierke (6dee0f6c6ea85d02d893a9afe7e4b062b89c97d5):

>DATAGEODE-28 - After release cleanups.

John Blum (b6f06b0a231c5cbc3b8dc5a124d78afe532b69d9):

>DATAGEODE-20 - Comment out reference to Apache (Build) Snapshots Repository.

John Blum (ca0776d9778d8390ed34389d8a522a1b7c13bc74):

>DATAGEODE-24 - Enable the 
>AutoConfiguredAuthenticationConfigurationIntegrationTests.clientAuthenticatesWithServer()
> test case.



--
This message is automatically generated by Atlassian Bamboo

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

[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 message?

- [ ] Has your PR been rebased against the latest commit within the target 
branch (typically `develop`)?

- [ ] Is your initial contribution a single, squashed commit?

- [ ] Does `gradlew build` run cleanly?

- [ ] Have you written or updated unit tests to verify your changes?

- [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?

### Note:
Please ensure that once the PR is submitted, you check travis-ci for build 
issues and
submit an update to your PR as soon as possible. If you need help, please 
send an
email to dev@geode.apache.org.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/upthewaterspout/incubator-geode 
feature/GEODE-3030

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/geode/pull/655.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #655


commit 2649d0faf3f64a1f25898f4b3f0ee512a7824f09
Author: Barry Oglesby 
Date:   2017-06-12T22:04:59Z

GEODE-3030: Set possibleDuplicate=true for all bucket events after failover




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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.m., Jinmei Liao wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61119/
> ---
> 
> (Updated July 25, 2017, 10:11 p.m.)
> 
> 
> Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and 
> Patrick Rhomberg.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-3097: GFSH works over HTTP with SSL
> 
> put the overhttp change in another test class and have it launhced in 
> geode-assembly, since only in that module the war file is created.
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ShellCommands.java
>  2da95a7c3f305aab4e615d4be7b14c19b9b31dbc 
>   
> geode-core/src/test/java/org/apache/geode/management/ConnectToLocatorSSLDUnitTest.java
>  844e0322eecdc93c4d46c546f4df2d278c5f15cd 
>   
> geode-core/src/test/java/org/apache/geode/management/ConnectToLocatorWithLegacySSLDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java
>  df371d28eac9d03cfb69ef462daf56f86bb95738 
>   
> geode-web/src/test/java/org/apache/geode/management/internal/cli/commands/ConnectToLocatorSSLOverHttpTest.java
>  PRE-CREATION 
> 
> 
> Diff: https://reviews.apache.org/r/61119/diff/1/
> 
> 
> Testing
> ---
> 
> precheckin successful
> 
> 
> Thanks,
> 
> Jinmei Liao
> 
>

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

[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 
Patrick Rhomberg.


Repository: geode


Description
---

GEODE-3097: GFSH works over HTTP with SSL

put the overhttp change in another test class and have it launhced in 
geode-assembly, since only in that module the war file is created.


Diffs
-

  
geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ShellCommands.java
 2da95a7c3f305aab4e615d4be7b14c19b9b31dbc 
  
geode-core/src/test/java/org/apache/geode/management/ConnectToLocatorSSLDUnitTest.java
 844e0322eecdc93c4d46c546f4df2d278c5f15cd 
  
geode-core/src/test/java/org/apache/geode/management/ConnectToLocatorWithLegacySSLDUnitTest.java
 PRE-CREATION 
  
geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java
 df371d28eac9d03cfb69ef462daf56f86bb95738 
  
geode-web/src/test/java/org/apache/geode/management/internal/cli/commands/ConnectToLocatorSSLOverHttpTest.java
 PRE-CREATION 


Diff: https://reviews.apache.org/r/61119/diff/1/


Testing
---

precheckin successful


Thanks,

Jinmei Liao

Re: Revert gradle upgrade

[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 the version of Antlr in the dependencies{} because if
> no version is set, the ANTLR plugin would default to 2.7.7. This was
> causing issues since the ANTLR plugin was trying to 'change' the version
> after dependency resolution for its configuration.
>
> I successfully tested imported into Eclipse, so I have checked in the
> changes on the feature/gradle-3.4.1 branch. Would others take some time to
> see if it works for them now?
>
> --Mark
>
> On Tue, May 9, 2017 at 10:51 AM, Udo Kohlmeyer 
> wrote:
>
>> I think we have to check the ANTLR plugin to understand it's
>> requirements..
>>
>> maybe there is a newer plugin version for ANTLR
>>
>>
>> On 5/9/17 10:00, Kirk Lund wrote:
>>
>>> I've reverted the upgrade to gradle 3.4.1. I also went ahead and created
>>> a
>>> branch that repeats the upgrade:
>>>
>>> feature/gradle-3.4.1
>>>
>>> Feel free to experiment with that branch.
>>>
>>> On Tue, May 9, 2017 at 9:53 AM, Kirk Lund  wrote:
>>>
>>> I'll go ahead and revert the upgrade. Hopefully we can eventually find
 the
 cause so we can repeat the upgrade.

 On Mon, May 8, 2017 at 4:19 PM, Bruce Schuchardt <
 bschucha...@pivotal.io>
 wrote:

 The "eclipse" target fails, too.
>
> Le 5/8/2017 à 3:32 PM, Kirk Lund a écrit :
>
> I want to revert the gradle upgrade because it breaks IntelliJ and the
>> idea
>> gradle task. Please let me know if you have a problem with this.
>>
>> PS: Please be prepared to help me if you vote against me reverting the
>> upgrade ;) I really want to be able to work again. Thanks!
>>
>>
>>
>>
>

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

[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-writing-cq-listener.html#cqlistener-implementation-(c#-.net)
C# CqListener should implement OnEvent(), OnError(), and Close() - notice 
the first letter capitalization.

NB(db): I also changed onCqDisconnected to OnCqDisconnected, onCqConnected 
to OnCqConnected, and clear to Clear.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/davebarnes97/geode-native feature/CapMembers

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/geode-native/pull/113.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #113


commit ec06436fad41a5f8cb6dd55f5f4a98b812c8127b
Author: Dave Barnes 
Date:   2017-07-25T21:41:05Z

(no ticket) Capitalize C# client member functions




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 p.m., Lynn Gallinat wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60875/
> ---
> 
> (Updated July 25, 2017, 12:03 p.m.)
> 
> 
> Review request for geode, anilkumar gingade, Darrel Schneider, and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> An online backup was not taking a snapshot of a single point in time. The 
> solution is for operations that change the disk files to acquire the backup 
> lock, causing them to wait until backup has rolled the op logs.
> 
> 
> Diffs
> -
> 
>   geode-core/src/main/java/org/apache/geode/internal/cache/BackupLock.java 
> 4b4fb10 
>   geode-core/src/main/java/org/apache/geode/internal/cache/DiskInitFile.java 
> 0925d28 
>   geode-core/src/main/java/org/apache/geode/internal/cache/DiskStoreImpl.java 
> 3e97d0e 
>   geode-core/src/main/java/org/apache/geode/internal/cache/Oplog.java 5399d5a 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/persistence/BackupPrepareAndFinishMsgDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/resources/org/apache/geode/codeAnalysis/excludedClasses.txt
>  db234e0 
>   
> geode-core/src/test/resources/org/apache/geode/codeAnalysis/sanctionedSerializables.txt
>  e5a111c 
> 
> 
> Diff: https://reviews.apache.org/r/60875/diff/2/
> 
> 
> Testing
> ---
> 
> Precheckin.
> 
> 
> Thanks,
> 
> Lynn Gallinat
> 
>

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

[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/security/GfshCommandsSecurityTest.java
Lines 135-152 (original), 129-145 (patched)


After the last few weeks, I agree that soft assertions were not a great 
choice when this was written.  On the other hand, we're still far from a 
single-assertion test here.

Maybe this would be a good test to try to use `@Parameterized`, to try to 
reach that one assertion per test ideal?


- Patrick Rhomberg


On July 17, 2017, 9:39 p.m., Jinmei Liao wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60924/
> ---
> 
> (Updated July 17, 2017, 9:39 p.m.)
> 
> 
> Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and 
> Patrick Rhomberg.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-2920: handle non-existent role in ExampleSecurityManager
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/org/apache/geode/examples/security/ExampleSecurityManager.java
>  c1d7ebfa017ccc627e7d4da6db951d3f61e2461b 
>   
> geode-core/src/test/java/org/apache/geode/management/internal/security/GfshCommandsSecurityTest.java
>  c2be173f8844bcf25361edc9e5d428f62aa8761f 
>   
> geode-core/src/test/java/org/apache/geode/security/ExampleSecurityManagerTest.java
>  9093a7bc70835ae50c72a80739a3f8ab4789cacd 
>   
> geode-core/src/test/resources/org/apache/geode/security/templates/security.json
>  c1ee9fc22bad58642d814c5930eb2d02810dad0e 
> 
> 
> Diff: https://reviews.apache.org/r/60924/diff/1/
> 
> 
> Testing
> ---
> 
> precheckin running
> 
> 
> Thanks,
> 
> Jinmei Liao
> 
>

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

[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() throws Exception {
   }
 
   @Test()
+  public void testWaitUntilFlushedForException() throws Exception {
+Map fields = new HashMap();
+fields.put("name", null);
+fields.put("lastName", null);
+fields.put("address", null);
+
luceneService.createIndexFactory().setFields(fields).create(INDEX_NAME, 
REGION_NAME);
+Region region = 
cache.createRegionFactory(RegionShortcut.PARTITION).create(REGION_NAME);
+final LuceneIndex index = luceneService.getIndex(INDEX_NAME, 
REGION_NAME);
+
+// This is to send IllegalStateException from WaitUntilFlushedFunction
+String nonCreatedIndex = "index2";
+
+boolean b =
+luceneService.waitUntilFlushed(nonCreatedIndex, REGION_NAME, 
6, TimeUnit.MILLISECONDS);
+assertFalse(b);
--- End diff --

@ameybarve15
I was under the assumption that the client currently is being passed the 
IllegalArgumentException and instead we would just throw the 
IllegalStateException and piggy back off of how FunctionExecution handles 
exceptions.  When a function fails during function execution, I would think an 
exception is passed back to the user?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: New Committer And PMC Member: Joey McAllister

[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!
> > >
> > > > 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 regards,
> > > >
> > > > Mark
> > > > On behalf of the Apache Geode PMC
> > >
> > >
> >
>

Re: New Committer And PMC Member: Joey McAllister

[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
> > 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
> >
> >
>

Re: New Committer And PMC Member: Joey McAllister

[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 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 #580: GEODE-2936: Refactoring OrderByComparator

[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 testUnsupportedOrderByForPR() throws 
Exception {
 
   @Test
   public void testSupportedOrderByForRR() throws Exception {
-
 String unsupportedQueries[] =
-{"select distinct p.status from /portfolio1 p order by p.status, 
p.ID",
-
-};
+{"select distinct p.status from /portfolio1 p order by p.status, 
p.ID"};
 Object r[][] = new Object[unsupportedQueries.length][2];
-QueryService qs;
-qs = CacheUtils.getQueryService();
 Position.resetCounter();
-// Create Regions
 
+// Create Regions
 Region r1 = CacheUtils.createRegion("portfolio1", Portfolio.class);
 
 for (int i = 0; i < 50; i++) {
   r1.put(new Portfolio(i), new Portfolio(i));
 }
 
 for (int i = 0; i < unsupportedQueries.length; i++) {
-  Query q = null;
-
+  Query q;
   CacheUtils.getLogger().info("Executing query: " + 
unsupportedQueries[i]);
   q = CacheUtils.getQueryService().newQuery(unsupportedQueries[i]);
   try {
 r[i][0] = q.execute();
-
   } catch (QueryInvalidException qe) {
 qe.printStackTrace();
 fail(qe.toString());
   }
 }
   }
 
+  /**
+   * Tests three cases that were not originally covered by the original 
tests in this class. (To
+   * confirm that these tests cover all cases in OrderByComparator, run 
them with coverage.)
+   */
+  @Test
+  public void testCompareMethodEdgeCases() throws Exception {
--- End diff --

Thanks for the feedback! I've updated the PR!


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 testUnsupportedOrderByForPR() throws 
Exception {
 
   @Test
   public void testSupportedOrderByForRR() throws Exception {
-
 String unsupportedQueries[] =
-{"select distinct p.status from /portfolio1 p order by p.status, 
p.ID",
-
-};
+{"select distinct p.status from /portfolio1 p order by p.status, 
p.ID"};
 Object r[][] = new Object[unsupportedQueries.length][2];
-QueryService qs;
-qs = CacheUtils.getQueryService();
 Position.resetCounter();
-// Create Regions
 
+// Create Regions
 Region r1 = CacheUtils.createRegion("portfolio1", Portfolio.class);
 
 for (int i = 0; i < 50; i++) {
   r1.put(new Portfolio(i), new Portfolio(i));
 }
 
 for (int i = 0; i < unsupportedQueries.length; i++) {
-  Query q = null;
-
+  Query q;
   CacheUtils.getLogger().info("Executing query: " + 
unsupportedQueries[i]);
   q = CacheUtils.getQueryService().newQuery(unsupportedQueries[i]);
   try {
 r[i][0] = q.execute();
-
   } catch (QueryInvalidException qe) {
 qe.printStackTrace();
 fail(qe.toString());
   }
 }
   }
 
+  /**
+   * Tests three cases that were not originally covered by the original 
tests in this class. (To
--- End diff --

All good points - I've fixed this and updated the PR!


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: New Committer And PMC Member: Joey McAllister

[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 regards,
> 
> Mark
> On behalf of the Apache Geode PMC

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

[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, Darrel Schneider, and Dan Smith.


Changes
---

This includes changes in response to review comments and a dunit test.


Repository: geode


Description
---

An online backup was not taking a snapshot of a single point in time. The 
solution is for operations that change the disk files to acquire the backup 
lock, causing them to wait until backup has rolled the op logs.


Diffs (updated)
-

  geode-core/src/main/java/org/apache/geode/internal/cache/BackupLock.java 
4b4fb10 
  geode-core/src/main/java/org/apache/geode/internal/cache/DiskInitFile.java 
0925d28 
  geode-core/src/main/java/org/apache/geode/internal/cache/DiskStoreImpl.java 
3e97d0e 
  geode-core/src/main/java/org/apache/geode/internal/cache/Oplog.java 5399d5a 
  
geode-core/src/test/java/org/apache/geode/internal/cache/persistence/BackupPrepareAndFinishMsgDUnitTest.java
 PRE-CREATION 
  
geode-core/src/test/resources/org/apache/geode/codeAnalysis/excludedClasses.txt 
db234e0 
  
geode-core/src/test/resources/org/apache/geode/codeAnalysis/sanctionedSerializables.txt
 e5a111c 


Diff: https://reviews.apache.org/r/60875/diff/2/

Changes: https://reviews.apache.org/r/60875/diff/1-2/


Testing
---

Precheckin.


Thanks,

Lynn Gallinat

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

[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 testUnsupportedOrderByForPR() throws 
Exception {
 
   @Test
   public void testSupportedOrderByForRR() throws Exception {
-
 String unsupportedQueries[] =
-{"select distinct p.status from /portfolio1 p order by p.status, 
p.ID",
-
-};
+{"select distinct p.status from /portfolio1 p order by p.status, 
p.ID"};
 Object r[][] = new Object[unsupportedQueries.length][2];
-QueryService qs;
-qs = CacheUtils.getQueryService();
 Position.resetCounter();
-// Create Regions
 
+// Create Regions
 Region r1 = CacheUtils.createRegion("portfolio1", Portfolio.class);
 
 for (int i = 0; i < 50; i++) {
   r1.put(new Portfolio(i), new Portfolio(i));
 }
 
 for (int i = 0; i < unsupportedQueries.length; i++) {
-  Query q = null;
-
+  Query q;
   CacheUtils.getLogger().info("Executing query: " + 
unsupportedQueries[i]);
   q = CacheUtils.getQueryService().newQuery(unsupportedQueries[i]);
   try {
 r[i][0] = q.execute();
-
   } catch (QueryInvalidException qe) {
 qe.printStackTrace();
 fail(qe.toString());
   }
 }
   }
 
+  /**
+   * Tests three cases that were not originally covered by the original 
tests in this class. (To
--- End diff --

This comment probably doesn't need to be a javadoc, and the note about 
coverage isn't needed. I'd make this comment say something like, "The following 
tests cover edge cases in OrderByComparator".


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: New Committer And PMC Member: Joey McAllister

[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 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
>
>

Re: New Committer And PMC Member: Joey McAllister

[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!
> 
> Best regards,
> 
> Mark
> On behalf of the Apache Geode PMC

New Committer And PMC Member: Joey McAllister

[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...

[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 p.m., Jared Stewart wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61114/
> ---
> 
> (Updated July 25, 2017, 4:58 p.m.)
> 
> 
> Review request for geode, Anthony Baker, Emily Yeh, Jens Deppe, Jared 
> Stewart, Ken Howe, Kirk Lund, and Patrick Rhomberg.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-3296: Speed up Acceptance Tests
> 
> - Move the acceptanceTest task from the top-level module into the 
> :geode-assembly submodule.
> 
> 
> Diffs
> -
> 
>   geode-assembly/build.gradle 02c9e33a3a35c423ff3d6aa9b1e151a34aca0378 
>   gradle/test.gradle 741c64e8dc5d9965e03744b0929d7a701abb2456 
> 
> 
> Diff: https://reviews.apache.org/r/61114/diff/1/
> 
> 
> Testing
> ---
> 
> Precheckin running
> 
> 
> Thanks,
> 
> Jared Stewart
> 
>

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

[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 Stewart wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61114/
> ---
> 
> (Updated July 25, 2017, 4:58 p.m.)
> 
> 
> Review request for geode, Anthony Baker, Emily Yeh, Jens Deppe, Jared 
> Stewart, Ken Howe, Kirk Lund, and Patrick Rhomberg.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-3296: Speed up Acceptance Tests
> 
> - Move the acceptanceTest task from the top-level module into the 
> :geode-assembly submodule.
> 
> 
> Diffs
> -
> 
>   geode-assembly/build.gradle 02c9e33a3a35c423ff3d6aa9b1e151a34aca0378 
>   gradle/test.gradle 741c64e8dc5d9965e03744b0929d7a701abb2456 
> 
> 
> Diff: https://reviews.apache.org/r/61114/diff/1/
> 
> 
> Testing
> ---
> 
> Precheckin running
> 
> 
> Thanks,
> 
> Jared Stewart
> 
>

GMSJoinLeaveJUnitTest failing on develop

[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.lang.AssertionError
at org.junit.Assert.fail(Assert.java:86)
at org.junit.Assert.assertTrue(Assert.java:41)
at org.junit.Assert.assertTrue(Assert.java:52)
at
org.apache.geode.distributed.internal.membership.gms.membership.GMSJoinLeaveJUnitTest.testViewNotSentWhenShuttingDown(GMSJoinLeaveJUnitTest.java:1157)

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

[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., Jared Stewart wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61114/
> ---
> 
> (Updated July 25, 2017, 4:58 p.m.)
> 
> 
> Review request for geode, Anthony Baker, Emily Yeh, Jens Deppe, Jared 
> Stewart, Ken Howe, Kirk Lund, and Patrick Rhomberg.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-3296: Speed up Acceptance Tests
> 
> - Move the acceptanceTest task from the top-level module into the 
> :geode-assembly submodule.
> 
> 
> Diffs
> -
> 
>   geode-assembly/build.gradle 02c9e33a3a35c423ff3d6aa9b1e151a34aca0378 
>   gradle/test.gradle 741c64e8dc5d9965e03744b0929d7a701abb2456 
> 
> 
> Diff: https://reviews.apache.org/r/61114/diff/1/
> 
> 
> Testing
> ---
> 
> Precheckin running
> 
> 
> Thanks,
> 
> Jared Stewart
> 
>

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

[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, 
Ken Howe, Kirk Lund, and Patrick Rhomberg.


Repository: geode


Description
---

GEODE-3296: Speed up Acceptance Tests

- Move the acceptanceTest task from the top-level module into the 
:geode-assembly submodule.


Diffs
-

  geode-assembly/build.gradle 02c9e33a3a35c423ff3d6aa9b1e151a34aca0378 
  gradle/test.gradle 741c64e8dc5d9965e03744b0929d7a701abb2456 


Diff: https://reviews.apache.org/r/61114/diff/1/


Testing
---

Precheckin running


Thanks,

Jared Stewart

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

[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

Single VM integration tests - runs in maybe 1.5 hours:
./gradlew geode-core:integrationTest

Multiple VM integration tests - runs in maybe 5 hours?
.gradlew geode-core:distributedTest

Run all of the tests
./gradlew geode-core:precheckin

-Dan

On Tue, Jul 25, 2017 at 8:34 AM, Anthony Baker  wrote:

> 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 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
> confidential and subject to the Amdocs policy statement,
> >
> > you may review at https://www.amdocs.com/about/email-disclaimer <
> https://www.amdocs.com/about/email-disclaimer>
>
>

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

[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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 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 
> confidential and subject to the Amdocs policy statement,
> 
> you may review at https://www.amdocs.com/about/email-disclaimer 
> 

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

[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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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/geode-native feature/GEODE-3173

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/geode-native/pull/112.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #112


commit 563bec192340f7f86fc2e6711033cbb08edc1826
Author: David Kimura 
Date:   2017-07-07T00:25:25Z

GEODE-3173: Upgrade to gtest 1.8




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 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
> confidential and subject to the Amdocs policy statement,
>
> you may review at https://www.amdocs.com/about/email-disclaimer <
> https://www.amdocs.com/about/email-disclaimer>
>

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

[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

--
[...truncated 98.27 KB...]
Download 
https://repo1.maven.org/maven2/io/springfox/springfox-spring-web/2.6.1/springfox-spring-web-2.6.1.pom
Download 
https://repo1.maven.org/maven2/com/fasterxml/classmate/1.3.1/classmate-1.3.1.pom
Download 
https://repo1.maven.org/maven2/org/springframework/plugin/spring-plugin-core/1.2.0.RELEASE/spring-plugin-core-1.2.0.RELEASE.pom
Download 
https://repo1.maven.org/maven2/org/springframework/plugin/spring-plugin/1.2.0.RELEASE/spring-plugin-1.2.0.RELEASE.pom
Download 
https://repo1.maven.org/maven2/org/springframework/plugin/spring-plugin-metadata/1.2.0.RELEASE/spring-plugin-metadata-1.2.0.RELEASE.pom
Download 
https://repo1.maven.org/maven2/org/mapstruct/mapstruct/1.0.0.Final/mapstruct-1.0.0.Final.pom
Download 
https://repo1.maven.org/maven2/org/mapstruct/mapstruct-parent/1.0.0.Final/mapstruct-parent-1.0.0.Final.pom
Download 
https://repo1.maven.org/maven2/io/springfox/springfox-core/2.6.1/springfox-core-2.6.1.pom
Download 
https://repo1.maven.org/maven2/com/fasterxml/jackson/module/jackson-module-scala_2.10/2.8.6/jackson-module-scala_2.10-2.8.6.jar
Download 
https://repo1.maven.org/maven2/io/springfox/springfox-swagger2/2.6.1/springfox-swagger2-2.6.1.jar
Download 
https://repo1.maven.org/maven2/io/springfox/springfox-swagger-ui/2.6.1/springfox-swagger-ui-2.6.1.jar
Download 
https://repo1.maven.org/maven2/org/springframework/hateoas/spring-hateoas/0.23.0.RELEASE/spring-hateoas-0.23.0.RELEASE.jar
Download 
https://repo1.maven.org/maven2/com/fasterxml/jackson/module/jackson-module-paranamer/2.8.6/jackson-module-paranamer-2.8.6.jar
Download 
https://repo1.maven.org/maven2/io/swagger/swagger-annotations/1.5.10/swagger-annotations-1.5.10.jar
Download 
https://repo1.maven.org/maven2/io/swagger/swagger-models/1.5.10/swagger-models-1.5.10.jar
Download 
https://repo1.maven.org/maven2/io/springfox/springfox-spi/2.6.1/springfox-spi-2.6.1.jar
Download 
https://repo1.maven.org/maven2/io/springfox/springfox-schema/2.6.1/springfox-schema-2.6.1.jar
Download 
https://repo1.maven.org/maven2/io/springfox/springfox-swagger-common/2.6.1/springfox-swagger-common-2.6.1.jar
Download 
https://repo1.maven.org/maven2/io/springfox/springfox-spring-web/2.6.1/springfox-spring-web-2.6.1.jar
Download 
https://repo1.maven.org/maven2/org/springframework/plugin/spring-plugin-core/1.2.0.RELEASE/spring-plugin-core-1.2.0.RELEASE.jar
Download 
https://repo1.maven.org/maven2/org/springframework/plugin/spring-plugin-metadata/1.2.0.RELEASE/spring-plugin-metadata-1.2.0.RELEASE.jar
Download 
https://repo1.maven.org/maven2/org/mapstruct/mapstruct/1.0.0.Final/mapstruct-1.0.0.Final.jar
Download 
https://repo1.maven.org/maven2/com/thoughtworks/paranamer/paranamer/2.8/paranamer-2.8.jar
Download 
https://repo1.maven.org/maven2/io/springfox/springfox-core/2.6.1/springfox-core-2.6.1.jar
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
:geode-web-api:processResources
:geode-web-api:classes
:geode-assembly:docs
:geode-assembly:gfshDepsJar
:geode-common:javadocJar
:geode-common:sourcesJar
:geode-common:signArchives SKIPPED
:geode-core:javadocJar
:geode-core:raJar
:geode-core:jcaJar
:geode-core:sourcesJar
:geode-core:signArchives SKIPPED
:geode-core:webJar
:geode-cq:jar
:geode-cq:javadoc
:geode-cq:javadocJar
:geode-cq:sourcesJar
:geode-cq:signArchives SKIPPED
:geode-json:javadocJar
:geode-json:sourcesJar
:geode-json:signArchives SKIPPED
:geode-lucene:jar
:geode-lucene:javadoc
:geode-lucene:javadocJar
:geode-lucene:sourcesJar
:geode-lucene:signArchives SKIPPED
:geode-old-client-support:jar
:geode-old-client-support:javadoc
:geode-old-client-support:javadocJar
:geode-old-client-support:sourcesJar
:geode-old-client-support:signArchives SKIPPED
:geode-protobuf:jar
:geode-protobuf:javadoc
:geode-protobuf:javadocJar
:geode-protobuf:sourcesJar
:geode-protobuf:signArchives SKIPPED
:geode-pulse:javadoc
:geode-pulse:javadocJar
:geode-pulse:sourcesJar
:geode-pulse:war
:geode-pulse:signArchives SKIPPED
:geode-rebalancer:jar
:geode-rebalancer:javadoc
:geode-rebalancer:javadocJar
:geode-rebalancer:sourcesJar
:geode-rebalancer:signArchives SKIPPED
:geode-wan:jar
:geode-wan:javadoc
:geode-wan:javadocJar
:geode-wan:sourcesJar
:geode-wan:signArchives SKIPPED
:geode-web:javadoc UP-TO-DATE
:geode-web:javadocJar
:geode-web:sourcesJar
:geode-web:war
:geode-web:signArchives SKIPPED
:geode-web-api:javadoc
:geode-web-api:javadocJar
:geode-web-api:sourcesJar
:geode-web-api:war
:geode-web-api:signArchives SKIPPED
:geode-assembly:installDist
:geode-pulse:jar
:geode-assembly:compileTestJava
Download 

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

[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 
confidential and subject to the Amdocs policy statement,

you may review at https://www.amdocs.com/about/email-disclaimer