> On Mar 21, 2016, at 6:57 PM, masa...@apache.org wrote:
>
> Repository: trafficserver
> Updated Branches:
> refs/heads/master 1e9c9484c -> 0e6e5c151
>
>
> TS-4087: Reduce SETTINGS_MAX_CONCURRENT_STREAMS when too many streams
>
> Add below variables in records.config
> - proxy.config.http2.mi
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/210#discussion_r57108812
--- Diff: plugins/experimental/ats_pagespeed/Makefile.am ---
@@ -0,0 +1,72 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+#
Hi Miles,
Next time, would you mind splitting this into separate commits? That makes it
easier to see what was fixed in the commit history ...
> On Mar 22, 2016, at 2:41 PM, mlib...@apache.org wrote:
>
> Repository: trafficserver
> Updated Branches:
> refs/heads/master 41441da8e -> 015b77176
>
GitHub user zwoop opened a pull request:
https://github.com/apache/trafficserver/pull/535
TS-4284 Modernize the geoip_acl plugin
There are two commits here:
1) Modernizes / refactors the plugin
and
2) Adds IPv6 support
I kept them as two separate
GitHub user zwoop opened a pull request:
https://github.com/apache/trafficserver/pull/534
TS-4280 Refactors the gzip plugin to improve hooks and memory usage
This accomplishes three things:
1) Fix memory leak on config reload (ref-counting)
2) Reduces the number of
GitHub user zwoop opened a pull request:
https://github.com/apache/trafficserver/pull/533
TS-3639 Implement GeoIP information into header_rewrite
This adds a set of new GeoIP based conditions to the header_rewrite plugin,
see the Examples/Geo file for an example. They can also be us
Another PR probably. This was really in the nature of an API review, I didn't
actually write any of the code yet.
On Tuesday, March 22, 2016 11:01 AM, zwoop wrote:
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/272#issuecomment-199880
GitHub user brkishore opened a pull request:
https://github.com/apache/trafficserver/pull/532
Ts 4266: Added a GC in add module and delete module
Refer Jira Ts 4266 for the issue and solution.
The first 6 commits are garbage. They were just created when I was trying
to dele
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 ha
Github user oschaaf commented on the pull request:
https://github.com/apache/trafficserver/pull/210#issuecomment-23777
@zwoop thanks!
---
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
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/210#issuecomment-19165
Easiest is to look at the Jira, which has instructions how to merge and
push. Make sure you "git pull --rebase" before you push upstream to ASF,
otherwise the "Comm
Github user oschaaf commented on the pull request:
https://github.com/apache/trafficserver/pull/210#issuecomment-11263
Change looks good to me! What is the flow with regard to committing a pull
request?
---
If your project is set up for it, you can reply to this email and have yo
Github user maskit commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/460#discussion_r57056410
--- Diff: proxy/http2/HPACK.cc ---
@@ -828,9 +828,111 @@ update_dynamic_table_size(const uint8_t *buf_start,
const uint8_t *buf_end, Http
if (len
Github user shaygalon commented on the pull request:
https://github.com/apache/trafficserver/pull/502#issuecomment-199982414
For x86 you actually have to sign extend the lower bit otherwise on AMD
systems you would be addressing the wrong address... The meaning of the bits
>48 is arch
GitHub user zwoop opened a pull request:
https://github.com/apache/trafficserver/pull/531
TS-4285 Makes proxy.config.http.attach_server_session_to_client overridable
EOM.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zwoop/traf
Please submit your proposals for the ATS Spring summit, before April 1st (it’s
no joke!). Send all proposals to
summ...@trafficserver.apache.org
Remember that this is a DevOps event, we encourage both developers, users and
anyone interested in ATS and ATS related technologies to submit
Github user bryancall commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/460#discussion_r57029193
--- Diff: proxy/http2/HPACK.cc ---
@@ -828,9 +828,111 @@ update_dynamic_table_size(const uint8_t *buf_start,
const uint8_t *buf_end, Http
if (
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 fea
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 GitH
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 p
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/123#issuecomment-199880892
@acaciocenteno Any chance you will work on this, and generalize this
concept?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/272#issuecomment-199880458
@SolidWallOfCode You going to land this in this incarnation? Or another PR?
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user SolidWallOfCode closed the pull request at:
https://github.com/apache/trafficserver/pull/272
---
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 user SolidWallOfCode closed the pull request at:
https://github.com/apache/trafficserver/pull/343
---
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 user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/351#issuecomment-199878590
ping on API review?
---
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 th
Github user jacksontj closed the pull request at:
https://github.com/apache/trafficserver/pull/526
---
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 featu
Github user jacksontj commented on the pull request:
https://github.com/apache/trafficserver/pull/526#issuecomment-199878278
I don't want to merge this PR in, its a definite hack. I'm working on a
better fix, it'll just take some time.
---
If your project is set up for it, you can re
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/502#issuecomment-199877909
@PSUdaemon to review, the thought it as that at least the freelist bit
mask is a clearer solution for all platforms.
---
If your project is set up for it, you can
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/503#issuecomment-199875223
@shukitchan Can you address Jame's comments, and then commit.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/493#issuecomment-199874748
@amc @scw00 It sounds like we should not commit this? If so, please close
this.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user PSUdaemon commented on the pull request:
https://github.com/apache/trafficserver/pull/525#issuecomment-199873028
Is it possible to make the priority queue stuff more generic so it can be
reusable?
---
If your project is set up for it, you can reply to this email and have
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/525#issuecomment-199871552
@bryancall to review, if we land this with off by default it seems
harmless. We do want to avoid STL when possible, but it is what it is :).
---
If your project is
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/526#issuecomment-199871069
@SolidWallOfCode to review.
---
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 no
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/402#issuecomment-199870599
@biilmann Can you email something to the dev@ mailing list with the API
proposal, for a review? See the link above for the process.
---
If your project is set up f
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/210#issuecomment-199859557
@oschaaf Ping!
---
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 f
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/199#issuecomment-199859179
Ping @acaciocenteno Are you still working / interested in this? We all want
this, just needs a little tweaking, and generalize the UUID functionality to be
a core c
36 matches
Mail list logo
