geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From DivineEnder <>
Subject [GitHub] geode pull request #619: Feature/geode 2901
Date Thu, 06 Jul 2017 00:02:02 GMT
GitHub user DivineEnder opened a pull request:

    Feature/geode 2901

    Thank you for submitting a contribution to Apache Geode.
    In order to streamline the review of the contribution we ask you
    to ensure the following steps have been taken:
    ### For all changes:
    - [ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
    - [ ] Has your PR been rebased against the latest commit within the target branch (typically
    - [ ] Is your initial contribution a single, squashed commit?
    - [ ] Does `gradlew build` run cleanly?
    - [ ] Have you written or updated unit tests to verify your changes?
    - [ ] If adding new dependencies to the code, are these dependencies licensed in a way
that is compatible for inclusion under [ASF 2.0](
    ### Note:
    Please ensure that once the PR is submitted, you check travis-ci for build issues and
    submit an update to your PR as soon as possible. If you need help, please send an
    email to

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

    $ git pull feature/GEODE-2901

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

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #619
commit 5d5ede50a1367f6851b3c9eb5db7a843b84cb4ac
Author: David Anuta <>
Date:   2017-07-05T17:12:01Z

    GEODE-3167: Added commons-validator dependency to modify_war script and Tomcat required

commit e1b5570bac6d56bce6132daa93c84e39938f5f88
Author: David Anuta <>
Date:   2017-07-03T23:55:03Z

    Updating installation and container management to allow more container independence (just
in time for the 4th)

commit 2350d37f294bf8a03a45cc8626522dcd1b610440
Author: David Anuta <>
Date:   2017-07-04T00:12:29Z

    Fixed incorrect local cache enabled property setting for jetty and tomcat
    Setting has previously been set as a cache property as enableLocalCache. Jetty needs the
cache property enable_local_cache so moved the setting of this property within each container
(Tomcat and Jetty) with their respective variable names

commit 1ad5395befdf330b35b931624a0bf79155dafa0f
Author: David Anuta <>
Date:   2017-07-05T23:21:27Z

    Fixed server.xml so that it was no longer installation specific
    Previously, each container edited the same file, which led to some concurrency issues.
Now the configuration holds the server xml configuration and only writes it to the configuration
folder when a container is started.


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

View raw message