GitHub user jablko opened a pull request:

    https://github.com/apache/trafficserver/pull/1371

    Calling setrlimit() twice is redundant

    Set .rlim_cur to either .rlim_max or RLIM_INFINITY.
    Doing both is redundant.
    
    This is a follow-up to #1362. The first commit belongs to that PR.

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

    $ git pull https://github.com/jablko/trafficserver max_it

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

    https://github.com/apache/trafficserver/pull/1371.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 #1371
    
----
commit 0d72c7a07446b1b4f931c4df53bed8323ebb2ea8
Author: Jack Bates <[email protected]>
Date:   2017-01-23T23:01:37Z

    If setrlimit() fails, log the details
    
    Fixes #1354

commit a6af4acd449ec04932ef65b1bf0016b0f444c255
Author: Jack Bates <[email protected]>
Date:   2017-01-24T16:54:28Z

    Calling setrlimit() twice is redundant
    
    Set .rlim_cur to either .rlim_max or RLIM_INFINITY.
    Doing both is redundant.

----


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

Reply via email to