Build failed in Jenkins: Geode-release #55

See Changes: [dbarnes] GEODE-1672: When amount of overflowed persisted data exceeds heap size [dbarnes] GEODE-1672: When amount of overflowed persisted data exceeds heap size

Re: Review Request 59866: GEODE-2555: Region Management docs page refers to the wrong field (id= should be refid=)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59866/#review177130 --- Ship it! Ship It! - Karen Miller On June 6, 2017, 10:50 p.m.

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

Github user masaki-yamakawa commented on the issue: https://github.com/apache/geode/pull/474 @galen-pivotal Thank you very much. Because there was a conflicting file with the change so far, I merged again, then pushed. Travis-CI worked normally. --- If your project is set up for

Re: Review Request 59852: GEODE-2632: make SecurityService immutable to improve client/server performance

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59852/ --- (Updated June 6, 2017, 11:57 p.m.) Review request for geode, Emily Yeh, Jinmei

Re: Review Request 59852: GEODE-2632: make SecurityService immutable to improve client/server performance

> On June 6, 2017, 10:23 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/web/controllers/support/LoginHandlerInterceptor.java > > Lines 79 (patched) > > > > > > this seems li

Re: Review Request 59852: GEODE-2632: make SecurityService immutable to improve client/server performance

> On June 6, 2017, 10:23 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/IndexCommands.java > > Lines 84 (patched) > > > > > > this seems to be added by some mer

[GitHub] geode issue #563: Add a bunch of .proto definitions to the new protocol.

Github user mattk-pio commented on the issue: https://github.com/apache/geode/pull/563 @kohlmu-pivotal Please merge this when possible. Trying not to switch between multiple dev branches while working against the new protocol. Thanks! --- If your project is set up for it, you can rep

Re: Do you want to Tweet for @ApacheGeode?

Hey Greg, I could tweet now and then. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: +1-631-835-4771 On Tue, Jun 6, 2017 at 12:28 PM, Greg Chase wrote: > Greetings Apache Geode community, > Tweeting for the Apache Geode project in the past has been a community > effort. > >

Re: Review Request 59850: GEODE-3023: TcpServer thread can be blocked in processRequest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59850/#review177121 --- Ship it! Ship It! - Bruce Schuchardt On June 6, 2017, 11:21

Re: Review Request 59850: GEODE-3023: TcpServer thread can be blocked in processRequest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59850/ --- (Updated June 6, 2017, 11:21 p.m.) Review request for geode, Bruce Schuchardt,

Review Request 59866: GEODE-2555: Region Management docs page refers to the wrong field (id= should be refid=)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59866/ --- Review request for geode and Galen O'Sullivan. Repository: geode Description

Re: Review Request 59811: GEODE-2420: add file-size-limit param to the ExportLogsController

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59811/#review177114 --- Ship it! Ship It! - Patrick Rhomberg On June 5, 2017, 6:09 p

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

--- Spring Data GemFire > Nightly-ApacheGeode > #577 was successful. --- Scheduled 1870 tests in total. https://build.spring.io/browse/SGF-NAG-577/ -- This

Re: Review Request 59852: GEODE-2632: make SecurityService immutable to improve client/server performance

> On June 6, 2017, 10:23 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/web/controllers/support/LoginHandlerInterceptor.java > > Lines 79 (patched) > > > > > > this seems li

Re: Review Request 59852: GEODE-2632: make SecurityService immutable to improve client/server performance

> On June 6, 2017, 10:23 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/IndexCommands.java > > Lines 84 (patched) > > > > > > this seems to be added by some mer

Re: Review Request 59852: GEODE-2632: make SecurityService immutable to improve client/server performance

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59852/ --- (Updated June 6, 2017, 10:26 p.m.) Review request for geode, Emily Yeh, Jinmei

Re: Review Request 59852: GEODE-2632: make SecurityService immutable to improve client/server performance

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59852/#review177108 --- geode-core/src/main/java/org/apache/geode/management/internal/cli

Review Request 59863: Removing obsolete SSL handling in `AcceptorImpl.accept` catch block

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59863/ --- Review request for geode, Bruce Schuchardt, Galen O'Sullivan, Hitesh Khamesra, a

[GitHub] geode pull request #566: GEODE-3044: User Manual: Update Swagger example and...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/566 --- 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 issue #508: GEODE-2908: Adding the new tag as per swagger notification...

Github user jaredjstewart commented on the issue: https://github.com/apache/geode/pull/508 Merged as `ac404ad`. Thank you for your contribution! --- 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 ha

Re: Review Request 59852: GEODE-2632: make SecurityService immutable to improve client/server performance

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59852/ --- (Updated June 6, 2017, 9:29 p.m.) Review request for geode, Emily Yeh, Jinmei L

[GitHub] geode issue #566: GEODE-3044: User Manual: Update Swagger example and screen...

Github user jaredjstewart commented on the issue: https://github.com/apache/geode/pull/566 (I also need to file a bug since that command appeared to hang.. But it did end up starting the server despite gfsh hanging with an endless ".") --- If your project is set up for it, you ca

[GitHub] geode issue #566: GEODE-3044: User Manual: Update Swagger example and screen...

Github user jaredjstewart commented on the issue: https://github.com/apache/geode/pull/566 Some more feedback after walking through the example.. Now that we start a locator (which starts an http server of its own on 7070) we need to specify a different http port for the server to us

Re: Review Request 59852: GEODE-2632: make SecurityService immutable to improve client/server performance

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59852/ --- (Updated June 6, 2017, 9:18 p.m.) Review request for geode, Emily Yeh, Jinmei L

[GitHub] geode-native pull request #100: GEODE-2741: Fixes compilation on Mac

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

Review Request 59852: GEODE-2632: make SecurityService immutable to improve client/server performance

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59852/ --- Review request for geode, Emily Yeh, Jinmei Liao, Jared Stewart, Ken Howe, and P

Re: Review Request 59850: GEODE-3023: TcpServer thread can be blocked in processRequest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59850/#review177094 --- geode-core/src/test/java/org/apache/geode/distributed/internal/tc

[GitHub] geode issue #566: GEODE-3044: User Manual: Update Swagger example and screen...

Github user jaredjstewart commented on the issue: https://github.com/apache/geode/pull/566 I think it would probably be best to omit `--J=-Dgemfire.http-service-bind-address=localhost` from the `start server` command for the sake of simplicity. You also might want to consider changin

Re: Review Request 59757: GEODE-3024 race condition between server and restarted locator preparing membership views

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59757/#review177093 --- Ship it! Ship It! - Hitesh Khamesra On June 2, 2017, 7:18 p.

Re: Review Request 59819: GEODE-3034 java.lang.ArrayIndexOutOfBoundsException: 0 on auto-reconnect attempt with multicast enabled

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59819/#review177091 --- Ship it! Ship It! - Hitesh Khamesra On June 5, 2017, 10:14 p

[GitHub] geode-native pull request #100: GEODE-2741: Fixes compilation on Mac

GitHub user mhansonp opened a pull request: https://github.com/apache/geode-native/pull/100 GEODE-2741: Fixes compilation on Mac You can merge this pull request into a Git repository by running: $ git pull https://github.com/mhansonp/geode-native feature/GEODE-2741 Alternativ

[GitHub] geode issue #508: GEODE-2908: Adding the new tag as per swagger notification...

Github user jaredjstewart commented on the issue: https://github.com/apache/geode/pull/508 Looks good to me. I'll merge it in shortly and cherry pick onto the release branch. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] geode issue #564: GEODE-3023 TcpServer thread can be blocked in processReque...

Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/564 👍 good catch! --- 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,

Re: [DISCUSS] easy string based partitioning

I have to agree. Something this trivial an limiting is better suited for a blog post or examples somewhere and not a part of the core codebase. -Jake On Mon, Jun 5, 2017 at 1:27 PM Udo Kohlmeyer wrote: > My concern with this approach is that we provide an implementation that > might be a white

Re: Review Request 59819: GEODE-3034 java.lang.ArrayIndexOutOfBoundsException: 0 on auto-reconnect attempt with multicast enabled

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59819/#review177085 --- Ship it! Ship It! - Udo Kohlmeyer On June 5, 2017, 10:14 p.m

Re: Review Request 59757: GEODE-3024 race condition between server and restarted locator preparing membership views

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59757/#review177084 --- Ship it! Ship It! - Udo Kohlmeyer On June 2, 2017, 7:18 p.m.

Re: Review Request 59850: GEODE-3023: TcpServer thread can be blocked in processRequest

> On June 6, 2017, 6:54 p.m., Anthony Baker wrote: > > geode-core/src/main/java/org/apache/geode/distributed/internal/tcpserver/TcpServer.java > > Line 354 (original), 356 (patched) > > > > > > Did you mean to leave

Broken: apache/geode-native#319 (develop - 11467dd)

Build Update for apache/geode-native - Build: #319 Status: Broken Duration: 7 minutes and 41 seconds Commit: 11467dd (develop) Author: Jacob Barrett Message: GEODE-2741: Code cleanup to move to std::shared_ptr - Removed the following custom objects: - HashMap

Re: Review Request 59849: add StringPrefixPartitionResolver

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59849/#review177074 --- Ship it! Ship It! - Eric Shu On June 6, 2017, 6:54 p.m., Dar

Do you want to Tweet for @ApacheGeode?

Greetings Apache Geode community, Tweeting for the Apache Geode project in the past has been a community effort. However, our most prolific tweeters are currently not spending much time with the project. As a result @ApacheGeode has fallen silent. Per ASF recommendations, we manage access to the

[GitHub] geode pull request #566: GEODE-3044: User Manual: Update Swagger example and...

GitHub user davebarnes97 opened a pull request: https://github.com/apache/geode/pull/566 GEODE-3044: User Manual: Update Swagger example and screen shots Updates Swagger screen shots and the example command lines that create the screens. Request reviews from @karensmolermil

Re: Review Request 59849: add StringPrefixPartitionResolver

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59849/ --- (Updated June 6, 2017, 11:54 a.m.) Review request for geode, Eric Shu and Lynn

Re: Review Request 59850: GEODE-3023: TcpServer thread can be blocked in processRequest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59850/#review177068 --- geode-core/src/main/java/org/apache/geode/distributed/internal/tc

[GitHub] geode issue #508: GEODE-2908: Adding the new tag as per swagger notification...

Github user metatype commented on the issue: https://github.com/apache/geode/pull/508 @jaredjstewart any thoughts on this change? If it's good I'd like to merge it since we need it for 1.2.0. --- If your project is set up for it, you can reply to this email and have your reply appea

Review Request 59850: GEODE-3023: TcpServer thread can be blocked in processRequest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59850/ --- Review request for geode, Bruce Schuchardt, Galen O'Sullivan, and Hitesh Khamesr

Re: Review Request 59849: add StringPrefixPartitionResolver

> On June 6, 2017, 11:17 a.m., Eric Shu wrote: > > geode-core/src/main/java/org/apache/geode/cache/partition/StringPrefixPartitionResolver.java > > Lines 60 (patched) > > > > > > Does getName() needs the Delimiter bei

Re: Review Request 59849: add StringPrefixPartitionResolver

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59849/#review177064 --- geode-core/src/main/java/org/apache/geode/cache/partition/StringP

Review Request 59849: add StringPrefixPartitionResolver

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59849/ --- Review request for geode, Eric Shu and Lynn Gallinat. Bugs: GEODE-3027 http

[GitHub] geode pull request #565: GEODE-3021: Any call after the first to setPdxStrin...

GitHub user jhuynh1 opened a pull request: https://github.com/apache/geode/pull/565 GEODE-3021: Any call after the first to setPdxStringFlag should no-op * The flag isIndexedPdxKeysFlagSet is now checked before setting pdx string flag @nabarunnag @ladyVader Than

Re: Geode versions

I would expect it to be under a month (hopefully much less), based on the remaining JIRA issues [1]. The community hasn’t begun discussing a release scope for 1.3.0 yet. Your participation in this process is encouraged! Anthony [1] https://issues.apache.org/jira/browse/GEODE-2908?jql=project

[GitHub] geode pull request #561: GEODE-3033: Fixing NPE when jarFileNames is null in...

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

Thanks Jake :-) -Original Message- From: Jacob Barrett [mailto:jbarr...@pivotal.io] Sent: Monday, June 05, 2017 4:07 PM To: dev@geode.apache.org Subject: Re: GEODE and GEODE-native compatibility You asked the same question on the user list on 5/21. In case you didn't get my reply I'll s

RE: Geode versions

Thanks Anthony, Can you estimate, whether this will be a week/month/3 months for the release of 1.2? What about 1.3? -Roi -Original Message- From: Anthony Baker [mailto:aba...@pivotal.io] Sent: Monday, June 05, 2017 7:34 PM To: dev@geode.apache.org Subject: Re: Geode versions Hi Roi,

[GitHub] geode pull request #564: GEODE-3023 TcpServer thread can be blocked in proce...

GitHub user vahrama opened a pull request: https://github.com/apache/geode/pull/564 GEODE-3023 TcpServer thread can be blocked in processRequest Move socket timeout setting before configureServerSSLSocket to ensure that SSL handshake will return in the allotted time and will not