[GitHub] trafficserver pull request #1254: TS-5085 `posix_fadvise` is incorrectly use...

2017-01-04 Thread shinrich
Github user shinrich closed the pull request at: https://github.com/apache/trafficserver/pull/1254 --- 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

[GitHub] trafficserver pull request #1254: TS-5085 `posix_fadvise` is incorrectly use...

2016-12-07 Thread danobi
Github user danobi commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1254#discussion_r91414850 --- Diff: proxy/logcat.cc --- @@ -300,6 +307,9 @@ main(int /* argc ATS_UNUSED */, const char *argv[]) } } }

[GitHub] trafficserver pull request #1254: TS-5085 `posix_fadvise` is incorrectly use...

2016-12-07 Thread danobi
GitHub user danobi opened a pull request: https://github.com/apache/trafficserver/pull/1254 TS-5085 `posix_fadvise` is incorrectly used in traffic_logcat `traffic_logcat` was not using `posix_fadvise` correctly. This patch corrects the original poor usage as well as adds some