[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-19 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-220376308 . --- 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 feat

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-13 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-219194918 Can one of the admins verify this patch? Only approve PRs which have been reviewed. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-13 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-219191871 Can one of the admins verify this patch? Only approve PRs which have been reviewed. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-13 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-219190145 Can one of the admins verify this patch? Only approve PRs which have been reviewed. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-13 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-219186495 Can one of the admins verify this patch? Only approve PRs which have been reviewed. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-12 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-218782581 Can one of the admins verify this patch? Only approve PRs which have been reviewed. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-218578787 Linux (CentOS7) build finished successfully. Details on https://ci.trafficserver.apache.org/job/Github-Linux/12/ --- If your project is set up for it, you c

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-11 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-218576777 [approve ci] --- 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 feat

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-06 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-217457973 @mingzym I think you are OK to make this change, but unless I missed it, I think you should also email dev@ and users@ saying that you plan on eliminating traffic_

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-02 Thread masaori335
Github user masaori335 commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-216432563 Here is the docs : [Internationalize Documentation - Update PO files](https://cwiki.apache.org/confluence/display/TS/Internationalize+Documentation#Internationa

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-01 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-216085057 @masaori335 ``doc/locale/ja/LC_MESSAGES/admin/working-log-files.en.po`` needs to be regenerated after this change. Can you document how to do that? --- If your pr

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-04-24 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-214041764 Could you please remove these remaining references: ``` $ git grep traffic_sac .gitignore:proxy/traffic_sac doc/locale/ja/LC_MESSAGES/admin/working-l

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-04-21 Thread mingzym
GitHub user mingzym opened a pull request: https://github.com/apache/trafficserver/pull/593 TS-4156: remove the traffic_sac, stand alone log collation server TS-4156: remove the traffic_sac, stand alone log collation server https://issues.apache.org/jira/browse/TS-4156 You