[GitHub] trafficserver issue #1436: Backport f71b75e and 734aa31 from master to 6.2.x

2017-02-09 Thread shenzhang920
Github user shenzhang920 commented on the issue: https://github.com/apache/trafficserver/pull/1436 @maskit @PSUdaemon This is the new pull request (I did a merge instead of rebase last time, sorry about that). Let me know if this is OK. Thanks! --- If your project is set up

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

2017-02-09 Thread shenzhang920
GitHub user shenzhang920 opened a pull request: https://github.com/apache/trafficserver/pull/1436 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

[GitHub] trafficserver issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

2017-02-09 Thread shenzhang920
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

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

2017-02-09 Thread shenzhang920
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

[GitHub] trafficserver issue #1430: backport f71b75e and 734aa31 from master to 6.2.x

2017-02-09 Thread shenzhang920
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

[GitHub] trafficserver pull request #1429: cherry-pick f71b75e from master to 6.2.x

2017-02-08 Thread shenzhang920
Github user shenzhang920 closed the pull request at: https://github.com/apache/trafficserver/pull/1429 --- 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

[GitHub] trafficserver issue #1429: cherry-pick f71b75e from master to 6.2.x

2017-02-08 Thread shenzhang920
Github user shenzhang920 commented on the issue: https://github.com/apache/trafficserver/pull/1429 @maskit I don't know how to modify the pushed commit message, so I just delete the repo and create again, this time using "git cherry-pick -x". please take a look at https://

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

2017-02-08 Thread shenzhang920
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

[GitHub] trafficserver issue #1415: isuue #1399 add back partial code of tcp_congesti...

2017-02-08 Thread shenzhang920
Github user shenzhang920 commented on the issue: https://github.com/apache/trafficserver/pull/1415 @maskit Thanks for the guidance, just create a new pull request in 6.2.x to cherry-pick the one merged in master --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver pull request #1429: cherry-pick f71b75e from master to 6.2.x

2017-02-08 Thread shenzhang920
GitHub user shenzhang920 opened a pull request: https://github.com/apache/trafficserver/pull/1429 cherry-pick f71b75e from master to 6.2.x this is to port the change from master to 6.2.x Issue #1399 add back partial code of tcp_congestion_control erased by TS-3612 You can

[GitHub] trafficserver pull request #1415: isuue #1399 add back partial code of tcp_c...

2017-02-08 Thread shenzhang920
Github user shenzhang920 closed the pull request at: https://github.com/apache/trafficserver/pull/1415 --- 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

[GitHub] trafficserver issue #1415: isuue #1399 add back partial code of tcp_congesti...

2017-02-08 Thread shenzhang920
Github user shenzhang920 commented on the issue: https://github.com/apache/trafficserver/pull/1415 @maskit @zwoop Got it! I made the change on master and create another pull request. How do I discard this pull request? --- If your project is set up for it, you can reply

[GitHub] trafficserver pull request #1426: issue #1399 add code of tcp_congestion_con...

2017-02-08 Thread shenzhang920
GitHub user shenzhang920 opened a pull request: https://github.com/apache/trafficserver/pull/1426 issue #1399 add code of tcp_congestion_control erased by TS-3612 this pull request change is on master branch You can merge this pull request into a Git repository by running

[GitHub] trafficserver issue #1415: isuue #1399 add back partial code of tcp_congesti...

2017-02-07 Thread shenzhang920
Github user shenzhang920 commented on the issue: https://github.com/apache/trafficserver/pull/1415 @maskit I made another commit to fix the clang failure, can you trigger the build again? Thanks! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver issue #1415: isuue #1399 add back partial code of tcp_congesti...

2017-02-02 Thread shenzhang920
Github user shenzhang920 commented on the issue: https://github.com/apache/trafficserver/pull/1415 @maskit Right, since there're 3 caller now, I moved some of the repeated code inside set_tcp_congestion_control() --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver pull request #1415: isuue #1399 add back partial code of tcp_c...

2017-02-02 Thread shenzhang920
GitHub user shenzhang920 opened a pull request: https://github.com/apache/trafficserver/pull/1415 isuue #1399 add back partial code of tcp_congestion_control erased by… Add the missing code of tcp_congestion_control erased by push of TS-3612 You can merge this pull request

[GitHub] trafficserver issue #1399: Paft of tcp_congestion_control code was erased by...

2017-01-31 Thread shenzhang920
GitHub user shenzhang920 opened an issue: https://github.com/apache/trafficserver/issues/1399 Paft of tcp_congestion_control code was erased by push TS-3612 ATS can change the tcp_congestion_control algorithm on both client side and server side, this functionality was added in 5.x