[GitHub] geode pull request #630: GEODE-3141: GetRegion Operation implemented

GitHub user kohlmu-pivotal opened a pull request: https://github.com/apache/geode/pull/630 GEODE-3141: GetRegion Operation implemented Added OperationHandlerJUnitTest.java as parents class of all OperationHandler tests. General clean up of all `public static final` fields

[GitHub] geode pull request #629: GEODE-2997: New flow getAll/putAll

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/629 --- 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

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60718/#review180369 --- Tests could use a little thoughtful addition of whitespace.

Review Request 60834: GEODE-3051: Remove unreachable exception handling in AcceptorImpl.accept

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60834/ --- Review request for geode, Alexander Murmann, Bruce Schuchardt, Galen

[GitHub] geode pull request #631: GEODE-3051: Remove unreachable exception handling i...

GitHub user WireBaron opened a pull request: https://github.com/apache/geode/pull/631 GEODE-3051: Remove unreachable exception handling in AcceptorImpl.accept This removes handling of SSL exceptions from the AccepterImpl.accept call, as the SSL handling code is now all done in

Build failed in Jenkins: Geode-release-flaky #27

See -- [...truncated 90.00 KB...] Download https://repo1.maven.org/maven2/org/springframework/security/spring-security-config/4.2.1.RELEASE/spring-security-config-4.2.1.RELEASE.pom

Errored: jinmeiliao/geode#14 (refactor-rebased - 401f91b)

Build Update for jinmeiliao/geode - Build: #14 Status: Errored Duration: 8 minutes and 46 seconds Commit: 401f91b (refactor-rebased) Author: Jinmei Liao Message: Merge branch 'refactor-week2-day3' into refactor-week2-friday View the changeset:

Re: Requesting edit permissions for Geode tickets

Easy peasy. On Wed, Jul 12, 2017 at 3:28 PM, Dan Smith wrote: > I don't see that user. I think you probably need to create an account on > the wiki first. > > -Dan > > On Wed, Jul 12, 2017 at 12:09 PM, Fred Krone wrote: > > > I don't think I have access. I

[GitHub] geode-examples pull request #7: GEODE-3196: Fixing the format of the .md5 an...

GitHub user upthewaterspout opened a pull request: https://github.com/apache/geode-examples/pull/7 GEODE-3196: Fixing the format of the .md5 and .sha256 files You can merge this pull request into a Git repository by running: $ git pull

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60718/#review180349 ---

[GitHub] geode-examples pull request #8: GEODE-3195 Add querying example to the geode...

GitHub user karensmolermiller opened a pull request: https://github.com/apache/geode-examples/pull/8 GEODE-3195 Add querying example to the geode-examples Here is a new geode-example that presents very simple queries on a single region done 2 ways: API calls and issued via gfsh.

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

See

Re: Geode-Native Windows build

Hi Daniel, Would you please share your config step command line? EB On Wed, Jul 12, 2017 at 7:52 AM, Daniel Farcovich < daniel.farcov...@amdocs.com> wrote: > Hi, > I'm failing to build geode native cpp client on windows, although followed > the instructions on BUILDING.md > Looks like there

Re: Geode-Native Windows build

And output from the config phase. Sent from my iPhone > On Jul 12, 2017, at 7:01 AM, Ernest Burghardt wrote: > > Hi Daniel, > > Would you please share your config step command line? > > EB > > On Wed, Jul 12, 2017 at 7:52 AM, Daniel Farcovich < >

Re: Geode-Native Windows build

While we wait for your config output there is something else that might bite you if it isn't what is biting you now. Some of the third part libraries built as part of Geode native have deep directory structures. Your deep directory structure to your sources combine with that may result in paths

Geode-Native Windows build

Hi, I'm failing to build geode native cpp client on windows, although followed the instructions on BUILDING.md Looks like there are slash/backslash fixes should be done in the paths in cmake files. After overcoming cmake failures the build fails on stage "cmake --build . -- /m" with

Build failed in Jenkins: Geode-nightly #894

See Changes: [bschuchardt] GEODE-3175 backward-compatibility tests fail with bad classpath [hkhamesra] GEODE-2998: Add remove operation [ukohlmeyer] GEODE-3121: ensure that the protobuf protocol works over SSL

[GitHub] geode pull request #627: GEODE-3194: cleanup disk store on failed initial re...

GitHub user nreich opened a pull request: https://github.com/apache/geode/pull/627 GEODE-3194: cleanup disk store on failed initial recovery Thank you for submitting a contribution to Apache Geode. In order to streamline the review of the contribution we ask you to

[GitHub] geode issue #626: GEODE-3179: Fixing bundled jars test finding slf4j-jdk jar

Github user DivineEnder commented on the issue: https://github.com/apache/geode/pull/626 @ladyVader @nabarunnag @boglesby @jhuynh1 @upthewaterspout @gesterzhou --- 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

[GitHub] geode pull request #626: GEODE-3179: Fixing bundled jars test finding slf4j-...

GitHub user DivineEnder opened a pull request: https://github.com/apache/geode/pull/626 GEODE-3179: Fixing bundled jars test finding slf4j-jdk jar This fixes the failing verify bundled jars test, caused by an unexpected slf4j-jdk jar coming from the tomcat module folder when

[GitHub] geode issue #627: GEODE-3194: cleanup disk store on failed initial recovery

Github user dschneider-pivotal commented on the issue: https://github.com/apache/geode/pull/627 This looks good. I will pull it 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

Review Request 60808: GEODE-3178 Fix a flaky JTA test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60808/ --- Review request for geode, anilkumar gingade, Darrel Schneider, Lynn Gallinat,

Requesting wiki edit privileges

Will someone please give me edit privileges for Geode wiki pages? Thanks! Diane Hardman

Proposal: Lucene indexing/searching for nested objects

The Geode 1.2.0 release includes Lucene text search fully integrated and tested (no longer experimental). We are now proposing enhancements to improve Lucene usability in Geode. Some Geode users create data models that include nested and complex objects. The current Geode Lucene integration

Re: Requesting edit permissions for Geode tickets

Could someone give me karma on the wiki? Thanks, -Fred On Thu, Mar 30, 2017 at 1:20 PM, Kirk Lund wrote: > I gave you permissions for Jira. Someone else still needs to give you karma > on the Wiki. > > -Kirk > > On Thu, Mar 30, 2017 at 9:22 AM, Fred Krone

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

> On July 7, 2017, 11:51 p.m., Udo Kohlmeyer wrote: > > geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/PutAllRequestOperationHandler.java > > Lines 81 (patched) > > > > > > I'm sure that

[GitHub] geode pull request #626: GEODE-3179: Fixing bundled jars test finding slf4j-...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/626 --- 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

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60718/ --- (Updated July 12, 2017, 6:23 p.m.) Review request for geode, Alexander

Re: Review Request 60808: GEODE-3178 Fix a flaky JTA test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60808/#review180320 --- Ship it! Ship It! - Darrel Schneider On July 12, 2017,

[GitHub] geode issue #626: GEODE-3179: Fixing bundled jars test finding slf4j-jdk jar

Github user upthewaterspout commented on the issue: https://github.com/apache/geode/pull/626 +1 I'll merge this. --- 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

[GitHub] geode pull request #627: GEODE-3194: cleanup disk store on failed initial re...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/627 --- 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

Re: Review Request 60808: GEODE-3178 Fix a flaky JTA test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60808/#review180325 ---

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60718/ --- (Updated July 12, 2017, 6:27 p.m.) Review request for geode, Alexander

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60718/ --- (Updated July 12, 2017, 6:01 p.m.) Review request for geode, Alexander

Re: Requesting edit permissions for Geode tickets

I don't see that user. I think you probably need to create an account on the wiki first. -Dan On Wed, Jul 12, 2017 at 12:09 PM, Fred Krone wrote: > I don't think I have access. I thought I did. fkrone ? > > On Wed, Jul 12, 2017 at 2:36 PM, Dan Smith

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

We will still return ErrorResponses for any failures or invalid messages, we've just removed the serverInternal and retriable fields from this message in favor a errorCode integer. On Wed, Jul 12, 2017 at 11:45 AM, Michael Stolz wrote: > We removed error feedback? > So how is

[VOTE] Apache Geode release - v1.2.0 RC2

This is the second release candidate for Apache Geode, version 1.2.0. Thanks to all the community members for their contributions to this release! This release candidate fixes a number of backwards compatibility and rolling upgrade issues found in RC1. *** Please download, test and vote by

[GitHub] geode pull request #603: GEODE-3128: Changed the batch size to 1000 when cre...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/603 --- 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

[GitHub] geode pull request #628: GEODE-3131: Replaced Thread.sleep

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/628 --- 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

Re: [VOTE] Apache Geode release - v1.2.0 RC2

I created GEODE-3199 for the gpg prompting issue with geode-examples. -Dan On Wed, Jul 12, 2017 at 2:04 PM, Dan Smith wrote: > Travis was actually trying to tell us something was wrong with the > geode-examples on the release branch as well: > >

[GitHub] geode-examples pull request #9: GEODE-3199: Make signing with a gpg key opti...

GitHub user upthewaterspout opened a pull request: https://github.com/apache/geode-examples/pull/9 GEODE-3199: Make signing with a gpg key optional Make it optional to sign the archives with a gpg key, to avoid annoying users trying to build the examples. You can merge this

[GitHub] geode-examples pull request #7: GEODE-3196: Fixing the format of the .md5 an...

Github user asfgit closed the pull request at: https://github.com/apache/geode-examples/pull/7 --- 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-examples pull request #9: GEODE-3199: Make signing with a gpg key opti...

Github user metatype commented on a diff in the pull request: https://github.com/apache/geode-examples/pull/9#discussion_r127090456 --- Diff: gradle/release.gradle --- @@ -41,17 +41,19 @@ gradle.taskGraph.whenReady( { graph -> }) gradle.taskGraph.whenReady {

Re: [VOTE] Apache Geode release - v1.2.0 RC2

Travis was actually trying to tell us something was wrong with the geode-examples on the release branch as well: https://travis-ci.org/apache/geode-examples/builds/244002605 You must configure your signing.keyId and signing.secretKeyRingFile in ~/.gradle/gradle.properties in order to sign jars

[GitHub] geode-examples pull request #8: GEODE-3195 Add querying example to the geode...

Github user metatype commented on a diff in the pull request: https://github.com/apache/geode-examples/pull/8#discussion_r127073923 --- Diff: queries/src/main/java/org/apache/geode/examples/queries/EmployeeData.java --- @@ -0,0 +1,81 @@ +/* + * Licensed to the Apache

[GitHub] geode-examples pull request #8: GEODE-3195 Add querying example to the geode...

Github user metatype commented on a diff in the pull request: https://github.com/apache/geode-examples/pull/8#discussion_r127073986 --- Diff: queries/src/main/java/org/apache/geode/examples/queries/EmployeeData.java --- @@ -0,0 +1,81 @@ +/* + * Licensed to the Apache

[GitHub] geode issue #628: GEODE-3131: Replaced Thread.sleep

Github user nabarunnag commented on the issue: https://github.com/apache/geode/pull/628 Potential Reviewers: @ladyVader @boglesby @jhuynh1 @upthewaterspout @gesterzhou @DivineEnder --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] geode-examples pull request #8: GEODE-3195 Add querying example to the geode...

Github user metatype commented on a diff in the pull request: https://github.com/apache/geode-examples/pull/8#discussion_r127074360 --- Diff: queries/scripts/new_startAll.sh --- @@ -0,0 +1,48 @@ +#!/bin/bash --- End diff -- How is this file used? --- If your

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60718/#review180365 --- Ship it! Ship It! - Bruce Schuchardt On July 12, 2017,

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

> On July 11, 2017, 5:17 p.m., Bruce Schuchardt wrote: > > geode-protobuf/src/test/java/org/apache/geode/protocol/protobuf/operations/GetRequestOperationHandlerJUnitTest.java > > Lines 138 (patched) > > > > > > need

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

> On July 12, 2017, 8:07 p.m., Bruce Schuchardt wrote: > > geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/GetAllRequestOperationHandler.java > > Lines 55 (patched) > > > > > > I think we

[GitHub] geode-examples issue #7: GEODE-3196: Fixing the format of the .md5 and .sha2...

Github user metatype commented on the issue: https://github.com/apache/geode-examples/pull/7 LGTM --- 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

[GitHub] geode-examples pull request #8: GEODE-3195 Add querying example to the geode...

Github user metatype commented on a diff in the pull request: https://github.com/apache/geode-examples/pull/8#discussion_r127073565 --- Diff: queries/src/main/java/org/apache/geode/examples/queries/Example.java --- @@ -0,0 +1,129 @@ +/* + * Licensed to the Apache Software

[GitHub] geode issue #628: GEODE-3131: Replaced Thread.sleep

Github user nabarunnag commented on the issue: https://github.com/apache/geode/pull/628 Thank you @ladyVader . Fixed the typo --- 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

[GitHub] geode-examples issue #8: GEODE-3195 Add querying example to the geode-exampl...

Github user davebarnes97 commented on the issue: https://github.com/apache/geode-examples/pull/8 Encountered one problem: initial run failed rat check. I commented out the rat check in build.gradle "/* apply from: "gradle/rat.gradle"*/". After that, I followed instructions and

Re: [VOTE] Apache Geode release - v1.2.0 RC2

+0 I ran geode-release-check against this project. It all looks good except that the md5sum and sha256 sums tasks don't pass because the .md5 and .sha256 files are in the wrong format. I created GEODE-3196. I don't think that should hold up

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60718/#review180353 --- Ship it! Ship It! - Alexander Murmann On July 12, 2017,

[GitHub] geode pull request #628: GEODE-3131: Replaced Thread.sleep

GitHub user nabarunnag opened a pull request: https://github.com/apache/geode/pull/628 GEODE-3131: Replaced Thread.sleep * Replaced sleeps with an Awaitility clause which waits for the region sizes to be correct before starting validation. Thank you for submitting a

[GitHub] geode-examples pull request #8: GEODE-3195 Add querying example to the geode...

Github user metatype commented on a diff in the pull request: https://github.com/apache/geode-examples/pull/8#discussion_r127073236 --- Diff: replicated/scripts/start.gfsh --- @@ -19,7 +19,9 @@ start locator --name=locator --bind-address=127.0.0.1 start server --name=server1

[GitHub] geode-examples pull request #8: GEODE-3195 Add querying example to the geode...

Github user metatype commented on a diff in the pull request: https://github.com/apache/geode-examples/pull/8#discussion_r127074639 --- Diff: queries/src/main/java/org/apache/geode/examples/queries/Example.java --- @@ -0,0 +1,129 @@ +/* + * Licensed to the Apache Software

Re: Requesting edit permissions for Geode tickets

Hi Fred, What's your username on the wiki? -Dan On Wed, Jul 12, 2017 at 11:32 AM, Fred Krone wrote: > Could someone give me karma on the wiki? > > Thanks, > -Fred > > On Thu, Mar 30, 2017 at 1:20 PM, Kirk Lund wrote: > > > I gave you permissions for Jira.

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

We removed error feedback? So how is an application programmer supposed to determine what failed now? Without that information we may have rendered putAll unusable for some cases. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: +1-631-835-4771 On Wed, Jul 12, 2017 at 1:45 PM,

[GitHub] geode pull request #625: GEODE-1988: Increased the readTimeout

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/625 --- 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

Re: Requesting edit permissions for Geode tickets

I don't think I have access. I thought I did. fkrone ? On Wed, Jul 12, 2017 at 2:36 PM, Dan Smith wrote: > Hi Fred, > > What's your username on the wiki? > > -Dan > > On Wed, Jul 12, 2017 at 11:32 AM, Fred Krone wrote: > > > Could someone give me karma on

Re: Requesting wiki edit privileges

I added you, you should have access now. Thanks! -Dan On Wed, Jul 12, 2017 at 10:12 AM, Diane Hardman wrote: > Will someone please give me edit privileges for Geode wiki pages? > > Thanks! > Diane Hardman >

[GitHub] geode-examples issue #9: GEODE-3199: Make signing with a gpg key optional

Github user upthewaterspout commented on the issue: https://github.com/apache/geode-examples/pull/9 Sure, I'll change it. I was using askpass just because that mirrored what the main geode repo uses, but I think signArtifacts or something like that is clearer. --- If your project

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #614 was SUCCESSFUL (with 1953 tests). Change made by John Blum.

--- Spring Data GemFire > Nightly-ApacheGeode > #614 was successful. --- Scheduled with changes by John Blum. 1955 tests in total.

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60718/#review180371 --- Ship it! Ship It! - Udo Kohlmeyer On July 12, 2017, 6:27

[GitHub] geode-examples issue #9: GEODE-3199: Make signing with a gpg key optional

Github user upthewaterspout commented on the issue: https://github.com/apache/geode-examples/pull/9 @metatype - I updated the PR. Travis is now stuck on find a 1.2 geode distribution, but it got past the signing check. --- If your project is set up for it, you can reply to this

Re: Requesting wiki edit privileges

Thanks! On Wed, Jul 12, 2017 at 10:17 AM, Dan Smith wrote: > I added you, you should have access now. > > Thanks! > -Dan > > On Wed, Jul 12, 2017 at 10:12 AM, Diane Hardman > wrote: > > > Will someone please give me edit privileges for Geode wiki pages?

[GitHub] geode pull request #629: GEODE-2997: New flow getAll/putAll

GitHub user WireBaron opened a pull request: https://github.com/apache/geode/pull/629 GEODE-2997: New flow getAll/putAll Changed get response to indicate if LookupFailure was a missing key or key with null value, added test Added GetAllRequestOperationHandler and unit test