[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

[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-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-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: 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

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

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

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