[GitHub] trafficserver pull request: Multi site origin feature

2015-08-27 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/283 Multi site origin feature We have a use case where our linear video and video on demand origin servers are deployed to geographically dispersed data centers and we would like to

[GitHub] trafficserver pull request: Modified the plugin to add back the Ra...

2015-09-18 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/297 Modified the plugin to add back the Range request header at the TS_HT… …TP_SEND_RESPONSE_HDR_HOOK so that custom logging of range requests works properly. You can merge this

[GitHub] trafficserver pull request: Modified the plugin to add back the Ra...

2015-09-18 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/297#issuecomment-141509647 Sudheer, I ran clang format and pushed it. Thanks -- John J. Rushford IPCDN Engineering 1400 Wewatta Street, Denver Colorado

[GitHub] trafficserver pull request: Modified the plugin to add back the Ra...

2015-09-18 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/297#issuecomment-141510195 thanks Thanks -- John J. Rushford IPCDN Engineering 1400 Wewatta Street, Denver Colorado 80202 john_rushf...@cable.comcast.com

[GitHub] trafficserver pull request: Modified the plugin to add back the Ra...

2015-09-18 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/297#issuecomment-141522795 You’re right, I should have. Will do next time ;) Thanks -- John J. Rushford IPCDN Engineering 1400 Wewatta Street, Denver Colorado

[GitHub] trafficserver pull request: Modified the plugin to add back the Ra...

2015-09-18 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/297#issuecomment-141536125 That’s why I added it back as ‘Range’, so that it would be seamless for those using custom logs. Otherwise they’d have to read the source code and

[GitHub] trafficserver pull request: Cache range requests changes

2015-09-21 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/298 Cache range requests changes logging changes suggested by jpeach. You can merge this pull request into a Git repository by running: $ git pull https://github.com/jrushf1239k

[GitHub] trafficserver pull request: Multi site origin feature

2015-10-29 Thread jrushf1239k
Github user jrushf1239k closed the pull request at: https://github.com/apache/trafficserver/pull/283 --- 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: Multi site origin feature

2015-10-29 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/283#issuecomment-152298104 I want to close this pull request for now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] trafficserver pull request: Parent selection 2.0

2015-11-04 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/321 Parent selection 2.0 This is a refactor of Parent Selection to add the following features: 1) multi-site origin - using parent selection, origin servers may be selected using

[GitHub] trafficserver pull request: Parent selection 2.0

2015-11-06 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/321#issuecomment-154527197 I fixed that, ParentSelectionStrategy now has a virtual destructor. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver pull request: TS-3418: Second hash ring for consiste...

2015-12-02 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/359 TS-3418: Second hash ring for consistently hashed parent selection This pull request is to add an optional secondary hash ring to parent selection and is based upon feedback from James

[GitHub] trafficserver pull request: TS-3418: Second hash ring for consiste...

2015-12-07 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/359#issuecomment-162602947 Hi James, I've squashed several commits and cleaned up the history, I'll be more descriptive in the future. Also, I found that I had le

[GitHub] trafficserver pull request: TS-3418: Second hash ring for consiste...

2015-12-09 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/359#issuecomment-163386805 Done, Thanks for the git help! Thanks -- John J. Rushford IPCDN Engineering 1400 Wewatta Street, Denver Colorado 80202 john_rushf

[GitHub] trafficserver pull request: TS-4030: Fixed a regression test failu...

2015-12-15 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/377 TS-4030: Fixed a regression test failure. Fix a bug found with regression tests. ParentResult::start_parent was not initialized in FindParent() when using strict round robin in a

[GitHub] trafficserver pull request: TS-4030: Fixed a regression test failu...

2015-12-15 Thread jrushf1239k
Github user jrushf1239k closed the pull request at: https://github.com/apache/trafficserver/pull/377 --- 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: TS-3418: Second hash ring for consiste...

2015-12-17 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/359#issuecomment-165575394 James, I've made changes based upon your suggestions. Please review --- If your project is set up for it, you can reply to this email and

[GitHub] trafficserver pull request: TS-3418: Second hash ring for consiste...

2015-12-17 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/359#issuecomment-165605094 Fixed function declarations and removed url_len argument from the consistent hash lookup functions. --- If your project is set up for it, you can reply to

[GitHub] trafficserver pull request: TS-4089: Fixed coverity issues in pare...

2015-12-19 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/390 TS-4089: Fixed coverity issues in parent selection. Updated the ParentConfigParams destructor to free the memory pointed at by policy and corrected the null pointer issue in

[GitHub] trafficserver pull request: Cache range requests changes

2016-01-04 Thread jrushf1239k
Github user jrushf1239k closed the pull request at: https://github.com/apache/trafficserver/pull/298 --- 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: Cache range requests changes

2016-01-04 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/298#issuecomment-168814793 Closed. Was merged by jpeach on Oct 22, 2015. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] trafficserver pull request: TS-4115: add multi origin feature to p...

2016-01-06 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/407 TS-4115: add multi origin feature to parent selection. Updates to parent selection for multi origin features. You can merge this pull request into a Git repository by running

[GitHub] trafficserver pull request: TS-4133: Update the url_sig plugin so ...

2016-01-14 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/424 TS-4133: Update the url_sig plugin so that application query paramete… See TS-4133. This update to url_sig will allow application query parameters destined for an origin server to

[GitHub] trafficserver pull request: fixed a bug where a parent is not mark...

2016-02-12 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/473 fixed a bug where a parent is not marked down due to a race updating … …the parent record failCount. Added a fix to only mark up retryable parents in HttpTransact when the retry is

[GitHub] trafficserver pull request: fixed a bug where a parent is not mark...

2016-02-12 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/473#issuecomment-183422438 Hi Leif, Should be good now, I clang formatted it. John --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: fixed a bug where a parent is not mark...

2016-02-12 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/473#issuecomment-183425465 Yes, it’s being done in HttpTransact::handle_response_from_parent() when the parent is marked for retry, “parent_result->retry == true”. There itâ

[GitHub] trafficserver pull request: Fix a potential null reference found b...

2016-02-16 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/482 Fix a potential null reference found by clang-analyzer. I reopened TS-4199 for this pull request. This pull request fixes a potential null reference found by clang-analyzer. You can

[GitHub] trafficserver pull request: [TS-4095] adding a new example plugin ...

2016-02-22 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/393#issuecomment-187279799 I can't build the master branch since this was added, I think that the plugins/experimental/Makefile.am needs to be changed: - stream_e

[GitHub] trafficserver pull request: TS4095: fix the plugins/experimental/M...

2016-02-22 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/494 TS4095: fix the plugins/experimental/Makefile.am for webp_transform plugin. Could not build the master branch this morning as there were build errors with the webp_transform plugin

[GitHub] trafficserver pull request: TS4095: fix the plugins/experimental/M...

2016-02-22 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/494#issuecomment-187374466 I think this change is good as the configure.ac has the enable-cppapi check. so, configure won't create a makefile unless both options are used. I t

[GitHub] trafficserver pull request: TS-4115: add multi origin feature to p...

2016-02-28 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/407#issuecomment-189985840 Submitting another pull request with only changes for multi-site origin. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: TS-4115: add multi origin feature to p...

2016-02-28 Thread jrushf1239k
Github user jrushf1239k closed the pull request at: https://github.com/apache/trafficserver/pull/407 --- 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: TS4115: Add a Parent selection configu...

2016-02-28 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/504 TS4115: Add a Parent selection configuration for multiple origin servers. I refactored the pull request to add origin server selection using parent selection. You can merge this pull

[GitHub] trafficserver pull request: TS4115: Add a Parent selection configu...

2016-03-09 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/504#issuecomment-194416690 This PR adds multi-origin to parent selection. That is origin servers may be listed as parents for selection using parent selection with round robin or

[GitHub] trafficserver pull request: TS-4133: Update the url_sig plugin so ...

2016-03-14 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/424#issuecomment-196387624 Updated the README and added source code comments on the requirement that signing query parameters must always be appended to the end of any application

[GitHub] trafficserver pull request: TS4115: Add a Parent selection configu...

2016-03-15 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/504#issuecomment-196839635 Yup, it's been tested. Thanks -- John J. Rushford IPCDN Engineering 1400 Wewatta Street, Denver Colorado 80202 john_

[GitHub] trafficserver pull request: TS4115: Add a Parent selection configu...

2016-03-18 Thread jrushf1239k
Github user jrushf1239k closed the pull request at: https://github.com/apache/trafficserver/pull/504 --- 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: TS-4287: Add a simple and dead server ...

2016-03-21 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/530 TS-4287: Add a simple and dead server retry feature to Parent Selection. @zwoop, @jpeach, @PSUdaemon, @bryancall please review and comment. This is the simple retry and dead server

[GitHub] trafficserver pull request: TS-4287: Add a simple and dead server ...

2016-03-22 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/530#issuecomment-199881433 Dead Server retry is confusing and not accurate. Change the configuration variable names and description to use unavailabe server retry. --- If your

[GitHub] trafficserver pull request: TS-4133: Update the url_sig plugin so ...

2016-03-22 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/424#issuecomment-199902610 changes were pushed to master, closing this pull request. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver pull request: TS-4133: Update the url_sig plugin so ...

2016-03-22 Thread jrushf1239k
Github user jrushf1239k closed the pull request at: https://github.com/apache/trafficserver/pull/424 --- 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: TS-4287: Add a simple and dead server ...

2016-03-22 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/530#issuecomment-200011630 Updated this pull request and renamed dead server retry to unavailable server retry. --- If your project is set up for it, you can reply to this email and

[GitHub] trafficserver pull request: TS-4327: Fix bug introduced with the a...

2016-04-05 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/550 TS-4327: Fix bug introduced with the addition of multi-site origin to… Fixe a segmentation fault when checking the parent_result.rec pointer before it has been set by findParent

[GitHub] trafficserver pull request: TS-4327: Fix bug introduced with the a...

2016-04-06 Thread jrushf1239k
Github user jrushf1239k closed the pull request at: https://github.com/apache/trafficserver/pull/550 --- 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: TS-4327: Fix bug introduced with the a...

2016-04-06 Thread jrushf1239k
Github user jrushf1239k commented on the pull request: https://github.com/apache/trafficserver/pull/550#issuecomment-206533431 This has been merged to master. --- 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