You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by nabarunnag <gi...@git.apache.org> on 2016/05/03 19:59:31 UTC

[GitHub] incubator-geode pull request: GEODE-1121: Increased the max memory...

GitHub user nabarunnag opened a pull request:

    https://github.com/apache/incubator-geode/pull/139

    GEODE-1121: Increased the max memory in the sender

    * Increased the max memory in the sender to be 100MB
    * Reduced the puts to 120MB
    * Made the test identical to the one in SerialWANPropogationsFeatureDUnitTest

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

    $ git pull https://github.com/nabarunnag/incubator-geode feature/GEODE-1121

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

    https://github.com/apache/incubator-geode/pull/139.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 #139
    
----
commit f956214238683bc8da58f4556cfba1b9016ffd5a
Author: nabarun <nn...@pivotal.io>
Date:   2016-05-02T21:30:37Z

    GEODE-1121: Increased the max memory in the sender
    
    * Increased the max memory in the sender to be 100MB
    * Reduced the puts to 120MB
    * Made the test identical to the one in SerialWANPropogationsFeatureDUnitTest

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode pull request: GEODE-1121: Increased the max memory...

Posted by upthewaterspout <gi...@git.apache.org>.
Github user upthewaterspout commented on the pull request:

    https://github.com/apache/incubator-geode/pull/139#issuecomment-216614483
  
    +1 looks good, but maybe we should scale this test way down. Why does it need to put this much data in order to test this feature? Could we bring the max-mem down to the minimum 1 MB or something like that and still test the same thing?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode pull request: GEODE-1121: Increased the max memory...

Posted by nabarunnag <gi...@git.apache.org>.
Github user nabarunnag closed the pull request at:

    https://github.com/apache/incubator-geode/pull/139


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode pull request: GEODE-1121: Increased the max memory...

Posted by nabarunnag <gi...@git.apache.org>.
Github user nabarunnag closed the pull request at:

    https://github.com/apache/incubator-geode/pull/139


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-geode pull request: GEODE-1121: Increased the max memory...

Posted by nabarunnag <gi...@git.apache.org>.
GitHub user nabarunnag reopened a pull request:

    https://github.com/apache/incubator-geode/pull/139

    GEODE-1121: Increased the max memory in the sender

    * Increased the max memory in the sender to be 100MB
    * Reduced the puts to 120MB
    * Made the test identical to the one in SerialWANPropogationsFeatureDUnitTest

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

    $ git pull https://github.com/nabarunnag/incubator-geode feature/GEODE-1121

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

    https://github.com/apache/incubator-geode/pull/139.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 #139
    
----
commit adc2161dcced23bf99abeefa48fe112774872eb3
Author: nabarun <nn...@pivotal.io>
Date:   2016-05-05T18:00:32Z

    GEODE-1121: Reduced the memory configurations in the overflow tests.
    
    * Reduced the sender's max memory to 1MB.
    * Reduced the amount of puts to 5MB.
    * Introduced a listener to sleep on create event in the receiver to slow down the sender.
    * Modified the addListenerToSleepOnCreate function signature to take region name as a parameter.

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---