You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by shenzhang920 <gi...@git.apache.org> on 2017/02/09 03:17:49 UTC

[GitHub] trafficserver pull request #1430: backport f71b75e and 734aa31 from master t...

GitHub user shenzhang920 opened a pull request:

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

    backport f71b75e and 734aa31 from master to 6.2.x

    

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

    $ git pull https://github.com/shenzhang920/trafficserver 6.2.x

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

    https://github.com/apache/trafficserver/pull/1430.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 #1430
    
----
commit 0a9c2878b7ff01527bf05c0e9dc88c8609291592
Author: Shen Zhang <se...@linkedin.com>
Date:   2017-02-08T22:39:01Z

    isuue #1399 add back partial code of tcp_congestion_control erased by TS-3612
    
    (cherry picked from commit f71b75ef6f32abd168ac646a50be238e4375cadf)

commit 306a8b2f1cfad12758beee09b3c0c0fc8fc58c68
Author: Masakazu Kitajo <ma...@apache.org>
Date:   2017-02-09T01:13:50Z

    Fix a build issue that use of undeclared variables
    
    PR-1426 causes build error on some platforms which doesn't support congestion control.
    
    (cherry picked from commit 734aa31e0e6b4b200f960eb2be0282ce4826b969)

----


---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    Linux build *successful*! See https://ci.trafficserver.apache.org/job/linux-github/1383/ for details.
     



---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by PSUdaemon <gi...@git.apache.org>.
Github user PSUdaemon commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    [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 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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by maskit <gi...@git.apache.org>.
Github user maskit commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    #1399


---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    clang-analyzer build *failed*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/54/ for details.
     



---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    Linux build *successful*! See https://ci.trafficserver.apache.org/job/linux-github/1389/ for details.
     



---
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 #1430: backport f71b75e and 734aa31 from master t...

Posted by shenzhang920 <gi...@git.apache.org>.
Github user shenzhang920 closed the pull request at:

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


---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by PSUdaemon <gi...@git.apache.org>.
Github user PSUdaemon commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    @shenzhang920 you need to rebase, not merge. I don't want to back port a merge commit. I think your mistake is that you did this directly on the 6.2.x branch. You really should use a feature branch off of 6.2.x.


---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by shenzhang920 <gi...@git.apache.org>.
Github user shenzhang920 commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    @PSUdaemon I reset my origin to the initial state (without my change), and do it all over again.


---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    Linux build *successful*! See https://ci.trafficserver.apache.org/job/linux-github/1402/ for details.
     



---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    clang-analyzer build *failed*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/60/ for details.
     



---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by shenzhang920 <gi...@git.apache.org>.
Github user shenzhang920 commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    I merge upstream/6.2.x to my origin/6.2.x, please let me know if there's anything needed.


---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/freebsd-github/1490/ for details.
     



---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by PSUdaemon <gi...@git.apache.org>.
Github user PSUdaemon commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    @shenzhang920 can you rebase this?


---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/freebsd-github/1509/ for details.
     



---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by maskit <gi...@git.apache.org>.
Github user maskit commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    [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 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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/freebsd-github/1496/ for details.
     



---
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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by PSUdaemon <gi...@git.apache.org>.
Github user PSUdaemon commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    [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 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 issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1430
  
    clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/73/ for details.
     



---
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.
---