[GitHub] trafficserver pull request: TS-3535: Experimental Support of Strea...

2016-03-15 Thread masaori335
GitHub user masaori335 opened a pull request: https://github.com/apache/trafficserver/pull/525 TS-3535: Experimental Support of Stream Priority Feature in HTTP/2 [TS-3535](https://issues.apache.org/jira/browse/TS-3535) Experimental Support of Stream Priority Feature. ## App

RE: Intercept Plugin

2016-03-15 Thread Luca Rea
Hi, The " Not Found on Accelerator " and "Multi-Hop Cycle" logs seem indicate that ATS itself is acting as parent/origin for some urls, maybe you should take a deeper look at your config files. -Original Message- From: Dk Jack [mailto:dnj0...@gmail.com] Sent: lunedì 14 marzo 2016 20:

[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_rushf...@cable.

[GitHub] trafficserver pull request: TS-3535: Experimental Support of Strea...

2016-03-15 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/525#issuecomment-196868807 Seeing that there are some pretty serious issues, should we leave it disabled for now, with a records.config option to enable it? --- If your project is set up for

[GitHub] trafficserver pull request: TS-3535: Experimental Support of Strea...

2016-03-15 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/525#issuecomment-196875583 @zwoop see ``proxy.config.http2.stream_priority_enabled``? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

[GitHub] trafficserver pull request: Fix for TS-4276

2016-03-15 Thread jacksontj
GitHub user jacksontj opened a pull request: https://github.com/apache/trafficserver/pull/526 Fix for TS-4276 In the event that `lookup_done` returns a NULL, we'll reschedule the lookup in the future instead of dumping core. You can merge this pull request into a Git repository by

[GitHub] trafficserver pull request: Fix for TS-4276

2016-03-15 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/526#issuecomment-196904843 I'm not sure this is the correct fix, in this case where we have no space we probably should just get lookup_done to return a non-persistant version of the reco

[GitHub] trafficserver pull request: Fix for TS-4276

2016-03-15 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/526#issuecomment-196907474 This is a different patch to the same issue. Instead of scheduling it for a retry, we can just return the non-mmaped HostDBInfo we have, so everything else can

[GitHub] trafficserver pull request: Fix for TS-4276

2016-03-15 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/526#issuecomment-196921878 As I'm thinking through this some more, I'm not certain this is a great solution either. The particular case that I reproduced was that hostdb was full when ret

Re: Intercept Plugin

2016-03-15 Thread Dk Jack
Thanks. I think after cleaning up the config and reloading everything, things seem to be working fine with the custom plugin. Thanks everyone for the help. On Tue, Mar 15, 2016 at 1:18 AM, Luca Rea wrote: > Hi, > > The " Not Found on Accelerator " and "Multi-Hop Cycle" logs seem indicate > that

[GitHub] trafficserver pull request: TS-3535: Experimental Support of Strea...

2016-03-15 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/525#issuecomment-196971215 Perfect! --- 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

[GitHub] trafficserver pull request: Fix for TS-4276

2016-03-15 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/526#issuecomment-196975636 Maybe it needs to evict things more aggressively, such that it always has ~10% head room for these sort of events ? --- If your project is set up for it, you can r

[GitHub] trafficserver pull request: Fix for TS-4276

2016-03-15 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/526#issuecomment-197011767 I don't think we can ever gaurantee that there will always be space available, and since we already evict when we need space, we should just keep doing that IMO

[GitHub] trafficserver pull request: Fix for TS-4276

2016-03-15 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/526#issuecomment-197047822 Not sure I understand. I'm saying, evict before it gets full, so normal eviction happens when there's 10% of less available, before inserting new stuff. That gives

[GitHub] trafficserver pull request: Fix for TS-4276

2016-03-15 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/526#issuecomment-197055859 From what I'm seeing 1) should be completely doable-- I'll take a crack at it :) --- If your project is set up for it, you can reply to this email and have you