Review Request 48899: GEODE-1565: disable GemfireDataCommandsDUnitTest over HTTP

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48899/ --- Review request for geode, Anthony Baker, Grace Meilen, and Jinmei Liao. Bugs: G

Re: Non-flaky test failures

Looks like I may be responsible for breaking CommandOverHttpDUnitTest :( I'll fix this one as soon as possible (probably tomorrow morning). Sorry! -Kirk On Friday, June 17, 2016, Kirk Lund wrote: > CommandOverHttpDUnitTest is new and runs GFSH commands over HTTP. I think > we need to mark that

Re: M3 status?

Yep, huge thanks to Grace Meilen for finishing up GEODE-835! -Kirk On Friday, June 17, 2016, Swapnil Bawaskar wrote: > GEODE-835 has been fixed. Do we have any progress on GEODE-1493? > > On Wed, Jun 15, 2016 at 10:20 AM, Sai Boorlagadda < > sai.boorlaga...@gmail.com > > wrote: > > > i will gi

Re: Non-flaky test failures

CommandOverHttpDUnitTest is new and runs GFSH commands over HTTP. I think we need to mark that whole class with FlakyTest for now. The tests using runRepeatingIfNecessary are prone to timing issues which *may* have been impacted by me removing the retry loop in VM invoke. That removal was correct

Re: M3 status?

GEODE-1493 is resolved. On Fri, Jun 17, 2016 at 11:48 AM, Swapnil Bawaskar wrote: > GEODE-835 has been fixed. Do we have any progress on GEODE-1493? > > On Wed, Jun 15, 2016 at 10:20 AM, Sai Boorlagadda < > sai.boorlaga...@gmail.com > > wrote: > > > i will give a try fixing GEODE-1493. > > > > S

Re: Review Request 48432: GEODE-1494: Allowing stats to be measured with callbacks

> On June 16, 2016, 6:52 p.m., Kirk Lund wrote: > > geode-core/src/main/java/com/gemstone/gemfire/Statistics.java, line 473 > > > > > > This is a public interface. For backwards compatibility, maybe consider > > pro

Re: Review Request 48865: GEODE-11: Added changes to Lucene AEQ, to propagate expiration destroy events (by setting the flag forwardExpirationDestroy()).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48865/#review138361 --- Ship it! Ship It! - Dan Smith On June 17, 2016, 6:53 p.m., a

Re: Review Request 48889: GEODE-1493: config/gemfire.properties that is shipped with the geode distribution contains user specific info

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48889/#review138360 --- Ship it! Ship It! - Dan Smith On June 17, 2016, 11:48 p.m.,

Re: Review Request 48889: GEODE-1493: config/gemfire.properties that is shipped with the geode distribution contains user specific info

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48889/ --- (Updated June 17, 2016, 11:48 p.m.) Review request for geode and Dan Smith. S

Review Request 48889: Removed default culter config dir

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48889/ --- Review request for geode and Dan Smith. Bugs: GEODE-1493 https://issues.apa

[GitHub] incubator-geode pull request #167: GEODE-985: rebrand GFSH help for Apache G...

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

Re: update website for WAN, CQ and native client

+1 to "multi-site" (maybe even augmented with, dare I say "replication"), though I personally don't have a problem with WAN. Of course, that might seem dated, but then I am getting older, ;-) Perhaps WAN with a short description.. "Multi-Site Replication for Active/Active, Active/Passive (DR), Su

Non-flaky test failures

The most recent jenkins run [1] had 3 failures from non-flaky tests. The geode-core tests seem to reproduce pretty easily. Have there been any recent changes in this area? :geode-core:distributedTest com.gemstone.gemfire.cache30. PreloadedRegionTestCase > testTXNonblockingGetInitialImage F

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #343 has FAILED

--- Spring Data GemFire > Nightly-ApacheGeode > #343 failed. --- Scheduled No failed tests found, a possible compilation error. https://build.spring.io/browse/SG

Re: post processing for integrated security

Oh, yes. In the new model we are working on, users can just configure the security with no custom implementation. But if they want to do something crazy, they can still grab these interfaces and implement their own way. On Fri, Jun 17, 2016 at 1:57 PM, Michael Stolz wrote: > But the new model is

Re: update website for WAN, CQ and native client

I like multi-site. It means physical isolation. -- Mike Stolz Principal Engineer - Gemfire Product Manager Mobile: 631-835-4771 On Jun 17, 2016 5:02 PM, "Gregory Chase" wrote: > On Fri, Jun 17, 2016 at 11:56 AM, Gregory Chase wrote: > > > > > > > Boom! "Multi-site' or "Multi-Cluster" > > > > >

Re: update website for WAN, CQ and native client

On Fri, Jun 17, 2016 at 11:56 AM, Gregory Chase wrote: > > > Boom! "Multi-site' or "Multi-Cluster" > > > At the risk of sounding like I'm "+1"-ing myself, consider this bullet comes on the next row after "clustering". So if it says "multi-cluster" or "multi-clustering" with that cool cloud to c

[GitHub] incubator-geode issue #167: GEODE-985: rebrand GFSH help for Apache Geode

Github user metatype commented on the issue: https://github.com/apache/incubator-geode/pull/167 IIRC, the requirement is to mention (incubating) on the first reference. Here are the branding requirements: http://incubator.apache.org/guides/branding.html Anthony

Re: post processing for integrated security

But the new model is supposed to be configuration only no user code, so I don't think it applies -- Mike Stolz Principal Engineer - Gemfire Product Manager Mobile: 631-835-4771 On Jun 17, 2016 4:48 PM, "Jinmei Liao" wrote: > If we are only doing post-processing for get and query, then it's not t

Re: post processing for integrated security

If we are only doing post-processing for get and query, then it's not too much work adding it into the new model. On Fri, Jun 17, 2016 at 12:00 PM, Michael Stolz wrote: > Post processing is there to filter the data returned by a get or a query. > But it requires that they write code, so it is pr

Re: update website for WAN, CQ and native client

I agree with Greg that we shouldn't use the Redis logo. On Fri, Jun 17, 2016 at 11:56 AM, Gregory Chase wrote: > On Fri, Jun 17, 2016 at 11:49 AM, Anilkumar Gingade > wrote: > > > John is Right...WAN is more than just a replication of data...In order to > > highlight and meaningful we have desc

Re: Review Request 48868: GEODE-1349: prevent 'java.lang.InternalError: MemoryPool not found'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48868/#review138297 --- Ship it! Ship It! - Grace Meilen On June 17, 2016, 7:06 p.m.

Re: Review Request 48869: GEODE-1559: add javadocs to OptionJFormatter

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48869/#review138296 --- Ship it! Ship It! - Grace Meilen On June 17, 2016, 7:08 p.m.

Re: Review Request 48870: GEODE-835: remove developer name from comments

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48870/#review138295 --- Ship it! Ship It! - Grace Meilen On June 17, 2016, 7:10 p.m.

Re: Review Request 48871: GEODE-1563: apply FlakyTest category to testExecuteFunctionOnGroups

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48871/#review138294 --- Fix it, then Ship it! Fix it and then ship it geode-core/src/

Re: Review Request 48872: GEODE-1551: apply FlakyTest category to testSimulateForEntireDS

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48872/#review138292 --- Ship it! Ship It! - Grace Meilen On June 17, 2016, 7:12 p.m.

Re: com.gemstone.gemfire.ForcedDisconnectException: Member isn't responding to heartbeat requests

Overflow in Geode is really intended to be used as a way of dealing with a TEMPORARY situation where there is too much data to fit in memory...NOT as a steady state design pattern. Making Geode disk based is an anti-pattern in my opinion. -- Mike Stolz Principal Engineer, GemFire Product Manager

Review Request 48872: GEODE-1551: apply FlakyTest category to testSimulateForEntireDS

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48872/ --- Review request for geode and Grace Meilen. Bugs: GEODE-1551 https://issues.

Review Request 48871: GEODE-1563: apply FlakyTest category to testExecuteFunctionOnGroups

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48871/ --- Review request for geode and Grace Meilen. Bugs: GEODE-1563 https://issues.

[GitHub] incubator-geode issue #167: GEODE-985: rebrand GFSH help for Apache Geode

Github user scmbuildguy commented on the issue: https://github.com/apache/incubator-geode/pull/167 I did mention in a couple of places, however, in generalI think when referencing the project as 'Apache Geode', I think we should be using 'Apache Geode (incubating)' until Geode is a to

Review Request 48870: GEODE-835: remove developer name from comments

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48870/ --- Review request for geode and Grace Meilen. Bugs: GEODE-835 https://issues.a

[GitHub] incubator-geode pull request #167: GEODE-985: rebrand GFSH help for Apache G...

Github user scmbuildguy commented on a diff in the pull request: https://github.com/apache/incubator-geode/pull/167#discussion_r67560662 --- Diff: geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/i18n/CliStrings.java --- @@ -63,65 +59,67 @@ * * 2.

[GitHub] incubator-geode pull request #167: GEODE-985: rebrand GFSH help for Apache G...

Github user scmbuildguy commented on a diff in the pull request: https://github.com/apache/incubator-geode/pull/167#discussion_r67560641 --- Diff: geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/i18n/CliStrings.java --- @@ -63,65 +59,67 @@ * * 2.

Review Request 48869: GEODE-1559: add javadocs to OptionJFormatter

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48869/ --- Review request for geode and Grace Meilen. Bugs: GEODE-1559 https://issues.

Review Request 48868: GEODE-1349: prevent 'java.lang.InternalError: MemoryPool not found'

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48868/ --- Review request for geode, Grace Meilen, Jens Deppe, Jinmei Liao, and Kevin Dulin

[GitHub] incubator-geode issue #168: GEODE-1557: Not logging BucketNotFoundException

Github user nabarunnag commented on the issue: https://github.com/apache/incubator-geode/pull/168 I was just curious as to know why manager was asked to throw a BucketNotFoundException but in the verification we are checking that logger was never called with an error method conta

Re: post processing for integrated security

Post processing is there to filter the data returned by a get or a query. But it requires that they write code, so it is probably only pertinent for the old security model not for the new fully configurable End-to-end model that we have begun discussing. So MAYBE we're actually doing too much work

Re: update website for WAN, CQ and native client

On Fri, Jun 17, 2016 at 11:49 AM, Anilkumar Gingade wrote: > John is Right...WAN is more than just a replication of data...In order to > highlight and meaningful we have described this as Muti Site (WAN) Topology > in GemFire docs... > > -Anil. > Boom! "Multi-site' or "Multi-Cluster" -- Greg

[GitHub] incubator-geode issue #167: GEODE-985: rebrand GFSH help for Apache Geode

Github user davebarnes97 commented on the issue: https://github.com/apache/incubator-geode/pull/167 Nice work, Grace - thanks for handling it. One suggested correction: the name "geode" should be given an initial cap ('Geode') in the string EXPORT_OFFLINE_DISK_STORE__HELP. Than

Re: Review Request 48865: GEODE-11: Added changes to Lucene AEQ, to propagate expiration destroy events (by setting the flag forwardExpirationDestroy()).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48865/ --- (Updated June 17, 2016, 6:53 p.m.) Review request for geode, anilkumar gingade,

Re: update website for WAN, CQ and native client

John is Right...WAN is more than just a replication of data...In order to highlight and meaningful we have described this as Muti Site (WAN) Topology in GemFire docs... -Anil. On Fri, Jun 17, 2016 at 11:15 AM, Swapnil Bawaskar wrote: > Well, you can can replicate clusters over WAN, but as John

Re: M3 status?

GEODE-835 has been fixed. Do we have any progress on GEODE-1493? On Wed, Jun 15, 2016 at 10:20 AM, Sai Boorlagadda wrote: > i will give a try fixing GEODE-1493. > > Sai > > On Wed, Jun 15, 2016 at 10:17 AM, Dan Smith wrote: > > > I think we should fix GEODE-1493 for M3 - we shouldn't be shippin

Review Request 48865: GEODE-11: Added changes to Lucene AEQ, to propagate expiration destroy events (by setting the flag forwardExpirationDestroy()).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48865/ --- Review request for geode, anilkumar gingade, Barry Oglesby, Jason Huynh, William

[GitHub] incubator-geode pull request #168: GEODE-1557: Not logging BucketNotFoundExc...

GitHub user aparnard opened a pull request: https://github.com/apache/incubator-geode/pull/168 GEODE-1557: Not logging BucketNotFoundException Not logging BucketNotFoundException as an error in LuceneEventLister and added a test shouldIgnoreBucketNotFoundException to verify it

[GitHub] incubator-geode pull request #167: GEODE-985: rebrand GFSH help for Apache G...

GitHub user gracemeilen opened a pull request: https://github.com/apache/incubator-geode/pull/167 GEODE-985: rebrand GFSH help for Apache Geode You can merge this pull request into a Git repository by running: $ git pull https://github.com/gracemeilen/incubator-geode feature/G

Re: update website for WAN, CQ and native client

Well, you can can replicate clusters over WAN, but as John was saying and as per Mike's clubhouse presentation on WAN , there are lots of different use cases that WAN opens up like Business unit Active/Passive, Realm manager, follow the

[GitHub] incubator-geode pull request #166: GEODE-117 - gfsh put ignores --skip-if-ex...

GitHub user kjduling opened a pull request: https://github.com/apache/incubator-geode/pull/166 GEODE-117 - gfsh put ignores --skip-if-exists flag Fixed test, which erroneously reported that the --skip-if-exists flag is honored. Implemented check for flag within the put operation

Re: com.gemstone.gemfire.ForcedDisconnectException: Member isn't responding to heartbeat requests

+ user@geode.i.a.o On Fri, Jun 17, 2016 at 8:49 AM, Anthony Baker wrote: > I would start by turning on GC logging and see if you have any long > collections. A stop-the-world collection that takes longer than the member > timeout could cause the behavior you are observing. > > You may want to c

Re: update website for WAN, CQ and native client

On Fri, Jun 17, 2016 at 10:53 AM, John Blum wrote: > Hmmm, I disagree. > > Second, WAN is much more than Replication. It is a topology used in > Active-Active or DR architectures, or used perhaps for sending a select > subset, "filtered" (possibly transformed), information, based on locality, >

Re: update website for WAN, CQ and native client

Hmmm, I disagree. First, I think Replication (and Partitioning, for that matter) imply distribution (as does WAN). Given Geode is a IMDG, the keyword being "grid", then it is obvious distribution occurs in the scale-out (shared-nothing) architecture to ensure availability and reliability/resilien

Re: update website for WAN, CQ and native client

Hmmok a few suggestions... 1. On first token: "Replication and Partitioning", change the name to "Distribution and Partitioning". Because WAN is really cluster to cluster replication. The first feature is addressing distribution of data around a cluster, so we should reserve the word "replic

Re: update website for WAN, CQ and native client

I want to say "like a WAN". Just subjective, what do you think? On Thu, Jun 16, 2016 at 11:44 PM, Joey McAllister wrote: > +1 > > Looks great, Swapnil. > On Thu, Jun 16, 2016 at 11:16 PM Swapnil Bawaskar > wrote: > > > Thanks for the feedback. Here is take 2: http://i.imgur.com/gmsHomO.png > >

post processing for integrated security

While working on the integrated security for gemfire9/geode1, we are frequently baffled by this post-processing functionality that's been lumped into security. Here are a few observations we've come up with and want to run with the dev community for comments: 1. Post processing is not authorizatio

Re: com.gemstone.gemfire.ForcedDisconnectException: Member isn't responding to heartbeat requests

I would start by turning on GC logging and see if you have any long collections. A stop-the-world collection that takes longer than the member timeout could cause the behavior you are observing. You may want to consider using an off-heap region. Also check out the other eviction algorithms (e

Re: com.gemstone.gemfire.ForcedDisconnectException: Member isn't responding to heartbeat requests

Thanks Anthony * At the CRITICAL threshold further writes are blocked. * Does client get any kind of warning/error or does put or putAll fails. My client is just putting KVs and not aware the server State. and I am getting com.gemstone.gemfire.ForcedDisconnectException: Member isn't respon

Build failed in Jenkins: Geode-nightly #502

See Changes: [jchen] GEODE-1544: Fail to initiate final check when network partition occurs [klund] GEODE-835: replace geode-joptsimple with jopt-simple dependency [klund] GEODE-1521 - APP_FETCH_SIZE in GFSH should not be applied to COUN

Re: com.gemstone.gemfire.ForcedDisconnectException: Member isn't responding to heartbeat requests

You need enough heap to retain all your keys in memory. Values begin to be evicted at the EVICTION threshold. At the CRITICAL threshold further writes are blocked. Your GC settings should be tuned to match these thresholds (search the mailing list archives and/or wiki for some advice on GC tu

Re: com.gemstone.gemfire.ForcedDisconnectException: Member isn't responding to heartbeat requests

Thanks Anthony, I know that my heap is not sufficient for the data I want to put. but My Region is configured as PARTITION_OVERFLOW, so I was hoping that once Geode reaches to critical heap , it will start overflowing the data. Is this assumption correct ? thanks Avinash On Thu, Jun 16, 2016 a