You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tinkerpop.apache.org by jeromatron <gi...@git.apache.org> on 2015/12/17 00:07:19 UTC
[GitHub] incubator-tinkerpop pull request: TINKERPOP-1051: Add note in best...
GitHub user jeromatron opened a pull request:
https://github.com/apache/incubator-tinkerpop/pull/183
TINKERPOP-1051: Add note in best practice docs about gremlin server heap setting
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jeromatron/incubator-tinkerpop master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-tinkerpop/pull/183.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 #183
----
commit 3fedaf658a9a358c25bc9dd1d28087453c5eb854
Author: Jeremy Hanna <jh...@datastax.com>
Date: 2015-12-16T23:06:01Z
TINKERPOP-1051: Added best practice to increase the gremlin server heap size in non-trivial use cases.
----
---
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-tinkerpop pull request: TINKERPOP-1051 Add note in best ...
Posted by jeromatron <gi...@git.apache.org>.
Github user jeromatron commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/183#issuecomment-165859202
Changed max to maximum as well.
---
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-tinkerpop pull request: TINKERPOP-1051 Add note in best ...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-tinkerpop/pull/183
---
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-tinkerpop pull request: TINKERPOP-1051 Add note in best ...
Posted by spmallette <gi...@git.apache.org>.
Github user spmallette commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/183#issuecomment-165469993
Hate to nit/pick but "-xmx" should be "-Xmx" and could you change uses of "max" to "maximum" to avoid the abbreviation.
Hopefully, travis will be happy with his next build of this PR - seemed like the failure has nothing to do with the content of your 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 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-tinkerpop pull request: TINKERPOP-1051 Add note in best ...
Posted by jeromatron <gi...@git.apache.org>.
Github user jeromatron commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/183#issuecomment-165857812
Bah you're right. I fixed that typo with this follow-up commit.
---
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.
---