You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@trafficserver.apache.org by dmorilha <gi...@git.apache.org> on 2015/10/08 22:54:54 UTC
[GitHub] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...
GitHub user dmorilha opened a pull request:
https://github.com/apache/trafficserver/pull/302
(TS-3961) open source Yahoo's ats-multiplexer plug-in
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dmorilha/trafficserver multiplexer
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/trafficserver/pull/302.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 #302
----
commit b885432e85d15a1a7ce9e6097aab7798e30b24f3
Author: Daniel Vitor Morilha <dm...@yahoo-inc.com>
Date: 2015-10-07T17:51:34Z
integrating ats multiplexer into source tree
commit 033a8505ce7ba6e987c2339cb49513de36bba139
Author: Daniel Vitor Morilha <dm...@yahoo-inc.com>
Date: 2015-10-07T18:25:14Z
fixing multiplexer Makefile
commit 8dc64d191df750b58b1aac4f701dff0021c5811b
Author: Daniel Vitor Morilha <dm...@yahoo-inc.com>
Date: 2015-10-08T03:38:20Z
removing __STDC_FORMAT_MACROS defines
commit 141419c202412c5a0526e4383f4b5ec8c0ff3a1a
Author: Daniel Vitor Morilha <dm...@yahoo-inc.com>
Date: 2015-10-08T03:59:17Z
adding other items into enumeration
commit 8a0fbca81712e8756ad47242e4308c02b27a8297
Author: Daniel Vitor Morilha <dm...@yahoo-inc.com>
Date: 2015-10-08T04:01:44Z
casting to uint64_t
commit 1bc5796d169bc583376348fc4c85067d10ad4772
Author: Daniel Vitor Morilha <dm...@yahoo-inc.com>
Date: 2015-10-08T04:05:03Z
casting to unsigned int
commit a1a3fcd2dd5af6d2efafac8c24b468cb8b50d2c1
Author: Daniel Vitor Morilha <dm...@yahoo-inc.com>
Date: 2015-10-08T04:05:40Z
casting to unsigned int
commit 93dc633175450696df9c95fb34574219c8911380
Author: Daniel Vitor Morilha <dm...@yahoo-inc.com>
Date: 2015-10-08T04:07:37Z
moving timeout to compilation unit
commit 3af53bbaa5c04d3c8cde7eacd21aaadc6ebddf23
Author: Daniel Vitor Morilha <dm...@yahoo-inc.com>
Date: 2015-10-08T04:08:12Z
getting rid of unused variable
commit 0494c1aaa4fd25a27d7d3705df081acbdf8a9174
Author: Daniel Vitor Morilha <dm...@yahoo-inc.com>
Date: 2015-10-08T20:38:28Z
transitioning to TSHttpTxnIsInternal
----
---
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] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...
Posted by PSUdaemon <gi...@git.apache.org>.
Github user PSUdaemon commented on the pull request:
https://github.com/apache/trafficserver/pull/302#issuecomment-146682956
This belongs under the experimental directory.
---
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] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...
Posted by dmorilha <gi...@git.apache.org>.
Github user dmorilha commented on the pull request:
https://github.com/apache/trafficserver/pull/302#issuecomment-149658514
All comments addressed so far, please let me know if there is anything you guys want me to work on before merging. Thanks
---
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] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/trafficserver/pull/302
---
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] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...
Posted by zwoop <gi...@git.apache.org>.
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/302#issuecomment-146685122
Great, thanks!
---
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] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...
Posted by dmorilha <gi...@git.apache.org>.
Github user dmorilha commented on the pull request:
https://github.com/apache/trafficserver/pull/302#issuecomment-146684662
@zwoop will work on these items later, thanks
---
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] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...
Posted by PSUdaemon <gi...@git.apache.org>.
Github user PSUdaemon commented on the pull request:
https://github.com/apache/trafficserver/pull/302#issuecomment-150611112
@dmorilha, this looks pretty good. I am almost ready to commit it. Can you add one last change? Please update the NOTICES file. Then I will merge.
Thanks.
---
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] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...
Posted by bgaff <gi...@git.apache.org>.
Github user bgaff commented on the pull request:
https://github.com/apache/trafficserver/pull/302#issuecomment-146719893
@dmorilha , thanks!
---
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] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...
Posted by PSUdaemon <gi...@git.apache.org>.
Github user PSUdaemon commented on the pull request:
https://github.com/apache/trafficserver/pull/302#issuecomment-146683942
I should also add this looks very interesting and thanks for contributing it!
---
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] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...
Posted by zwoop <gi...@git.apache.org>.
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/302#issuecomment-146684009
This also must be run through clang-format first.
---
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.
---