Re: Review Request 59905: add GfshParserRule to fascilidate command unit test

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59905/#review177280 --- Ship it! Ship It! - Jinmei Liao On June 8, 2017, 1:48 a.m.,

Build failed in Jenkins: Geode-nightly #860

See Changes: [bschuchardt] GEODE-3024 race condition between server locator preparing membership [boglesby] GEODE-3040: Compare gateway profile order policy against 9.0 instead of [dbarnes] GEODE-3044: User

Review Request 59905: add GfshParserRule to fascilidate command unit test

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

[DISCUSS] Using Pull Requests over Review Board

All, I would like to discuss transitioning all code reviews to pull requests over using review board. For non-committer community members we ask them to make pull requests against the mirrored GitHub repo. Some committers use pull requests for their own work reviews while others use review board.

Re: [DISCUSS] easy string based partitioning

Actually it appears that parameterizing the delimeter would be quite problematic for client side single hop. -- Mike Stolz Principal Engineer - Gemfire Product Manager Mobile: 631-835-4771 On Jun 7, 2017 1:56 PM, "Galen M O'Sullivan" wrote: > I don't think it would be

Re: 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/ --- (Updated June 8, 2017, 12:40 a.m.) Review request for geode, Bruce Schuchardt,

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 8, 2017, 12:20 a.m.) Review request for geode, Alexander

[GitHub] geode pull request #567: GEODE-3051: Removing obsolete SSL handling in `Acce...

GitHub user WireBaron opened a pull request: https://github.com/apache/geode/pull/567 GEODE-3051: Removing obsolete SSL handling in `AcceptorImpl.accept` c… …atch block Thank you for submitting a contribution to Apache Geode. In order to streamline the review

Review Request 59904: GEODE-3050: prevent empty dat files

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59904/ --- Review request for geode and Bruce Schuchardt. Bugs: GEODE-3050

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/#review177256 --- Fix it, then Ship it! Ship It!

Re: 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/#review177253 ---

[GitHub] geode issue #325: merge new version

Github user kohlmu-pivotal commented on the issue: https://github.com/apache/geode/pull/325 Is this PR still open or relevant? If not, could we please close 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

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

Github user kohlmu-pivotal commented on the issue: https://github.com/apache/geode/pull/474 @masaki-yamakawa I'm running precheckin. If it passes I'll merge this 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

Re: Review Request 59893: GEODE-3032: Fix CI failure of CommandOverHttpDUnitTest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59893/#review177251 --- Ship it! Ship It! - Jinmei Liao On June 7, 2017, 11:13

Re: 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/#review177250 --- Ship it! Please attach the correct GEODE JIRA number to this.

Re: Review Request 59893: GEODE-3032: Fix CI failure of CommandOverHttpDUnitTest

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

Re: Review Request 59896: GEODE-3043 surprise member added when the member is already in the cluster

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59896/#review177249 --- Ship it! Ship It!

Re: Review Request 59893: GEODE-3032: Fix CI failure of CommandOverHttpDUnitTest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59893/#review177248 --- Ship it! Ship It! - Jinmei Liao On June 7, 2017, 9:08 p.m.,

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

Github user davebarnes97 commented on the issue: https://github.com/apache/geode/pull/508 Will add my comment to GEODE-3035 Rest: Swagger UI has lot of references to GemFire, instead they should be changed to geode --- If your project is set up for it, you can reply to this email

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

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

[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 This PR has already been merged in, but feel free to open a new ticket for that. We unfortunately still have a lot of other "gemfire" references scattered around. --- If your project is set

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/ --- (Updated June 7, 2017, 10:34 p.m.) Review request for geode, Emily Yeh, Jinmei

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

> On June 7, 2017, 9:40 p.m., Jared Stewart wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExportLogsCommand.java > > Line 122 (original), 123 (patched) > > > > > > I really

Re: Review Request 59893: GEODE-3032: Fix CI failure of CommandOverHttpDUnitTest

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59893/#review177237 --- Ship it! Ship It! - Emily Yeh On June 7, 2017, 9:08 p.m.,

Review Request 59896: GEODE-3043 surprise member added when the member is already in the cluster

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

Review Request 59894: GEODE-3041 CI failure: DistributedMemberDUnitTest.testGroupsInAllVMs fails intermittently

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

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/#review177230 ---

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/#review177228 --- Ship it! Ship It! - Jared Stewart On June 7, 2017, 6:35

[GitHub] geode pull request #558: GEODE-194: Remove spark connector

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/558 --- 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: Need authorization to tweet for Apache Geode

Never too many. I've added you to the TweetDeck team. On Wed, Jun 7, 2017 at 2:08 PM, Kirk Lund wrote: > I'll volunteer to help too unless that's too many folks. My twitter is > @kirk_lund > > On Wed, Jun 7, 2017 at 1:51 PM, Greg Chase wrote: > > >

Re: Need authorization to tweet for Apache Geode

I'll volunteer to help too unless that's too many folks. My twitter is @kirk_lund On Wed, Jun 7, 2017 at 1:51 PM, Greg Chase wrote: > Added you Mike. > > On Wed, Jun 7, 2017 at 1:41 PM, Michael Stolz wrote: > > > Hi Greg, > > > > I didn't give you my

Review Request 59893: GEODE-3032: Fix CI failure of CommandOverHttpDUnitTest

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

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/#review177224 --- Code looks good. I haven't taken a particularly thorough look

Re: [DISCUSS] easy string based partitioning

I don't think it would be that much harder to add an option for a user-specified delimiter than it would to hardcode ':' or '|'. I think the utility of that would be much higher, and that it would be a much better candidate for inclusion in geode-core. Otherwise, I think it should live in an

[GitHub] geode pull request #563: Add a bunch of .proto definitions to the new protoc...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/563 --- 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: Need authorization to tweet for Apache Geode

Added you Mike. On Wed, Jun 7, 2017 at 1:41 PM, Michael Stolz wrote: > Hi Greg, > > I didn't give you my Twitter handle: @MikeStolz1 > > > -- > Mike Stolz > Principal Engineer, GemFire Product Manager > Mobile: +1-631-835-4771 > > On Wed, Jun 7, 2017 at 1:23 PM, Gregory Chase

Re: Need authorization to tweet for Apache Geode

Hi Greg, I didn't give you my Twitter handle: @MikeStolz1 -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: +1-631-835-4771 On Wed, Jun 7, 2017 at 1:23 PM, Gregory Chase wrote: > Hi Jag, > Thanks for volunteering to help communicate the goodness of

Re: Need authorization to tweet for Apache Geode

Hi Jag, Thanks for volunteering to help communicate the goodness of @ApacheGeode! I'll add you to the authorized Twitter team in TweetDeck! -Greg On Wed, Jun 7, 2017 at 1:19 PM, Jagdish Mirani wrote: > Hello: > I would like to be authorized to tweet for Apache Geode - I

[GitHub] geode-native pull request #101: GEODE-2740: Modified CMAKELists.txt to suppo...

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

Need authorization to tweet for Apache Geode

Hello: I would like to be authorized to tweet for Apache Geode - I would like to Tweet from the project handle on behalf of the project. About me. *Twitter handle: *@jagmirani *Role: *I am the Product Marketing Manager for Pivotal GemFire (which is based on Apache Geode). Also, I work closely

[GitHub] geode-native pull request #101: GEODE-2740: Modified CMAKELists.txt to suppo...

GitHub user mhansonp opened a pull request: https://github.com/apache/geode-native/pull/101 GEODE-2740: Modified CMAKELists.txt to support 64bit linking You can merge this pull request into a Git repository by running: $ git pull https://github.com/mhansonp/geode-native

Re: [DISCUSS] easy string based partitioning

In regard to sending the configuration state to the clients for the partition resolver maybe the config should be a domain object independent of the business object rather than the serialized form of the partition resolver itself. The rational for this would be that non-Java clients could get

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/#review177204 ---

Re: [DISCUSS] easy string based partitioning

Thanks for all the feedback. Since this is such a simple resolver with a fixed delimiter the following changes were made to the original proposal: 1. The StringPrefixPartitionResolver will be in the "org.apache.geode.cache.util" package. This was done to keep it from being viewed as part of

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/ --- (Updated June 7, 2017, 6:35 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/#review177199 ---

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/#review177198 --- Things that are identifiable by inspections on Changed Files:

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/#review177196 --- Ship it! Ship It! - Galen O'Sullivan On June 6, 2017, 10:50

Re: Do you want to Tweet for @ApacheGeode?

Thanks both Michael and Swapnil. If you could please send me a private response with your Twitter handle, I will authorize you to Tweet from @ApacheGeode. If anyone else would also like to help, please respond publicly to the dev group first about your interest. Thanks! -Greg On Wed, Jun 7,

Re: Do you want to Tweet for @ApacheGeode?

So can I. On Tue, Jun 6, 2017 at 4:40 PM Michael Stolz wrote: > Hey Greg, > > I could tweet now and then. > > -- > Mike Stolz > Principal Engineer, GemFire Product Manager > Mobile: +1-631-835-4771 <(631)%20835-4771> > > On Tue, Jun 6, 2017 at 12:28 PM, Greg Chase

Build failed in Jenkins: Geode-release #56

See Changes: [jiliao] GEODE-2981: Fix the line feed code of the test expected value [jiliao] GEODE-3033: Fixing NPE when jarFileNames is null in [dbarnes] GEODE-3044: User Manual: Update Swagger example and screen

Build failed in Jenkins: Geode-nightly #859

See Changes: [jiliao] GEODE-3033: Fixing NPE when jarFileNames is null in [jstewart] GEODE-2908: Adding the new tag as per swagger notification. [dbarnes] GEODE-3044: User Manual: Update Swagger example and screen

Need information about FunctionStatistics

Hi Team, Currently I can see Function stats are getting generated . functionExecutionsCompleted operations/sec: samples=1955 min=0 max=0 average=0 stddev=0 last=0 functionExecutionsCompletedProcessingTime nanoseconds/sec: samples=1955 min=0 max=0 average=0 stddev=0 last=0