Github user maskit commented on the pull request:
https://github.com/apache/trafficserver/pull/570#issuecomment-210199731
Reverting TS-4321 sounds reasonable if we can modify
``stream->response_header`` without any side effects.
The response headers in FetchSM must not be modi
Github user sudheerv closed the pull request at:
https://github.com/apache/trafficserver/pull/567
---
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 featur
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/567#issuecomment-210174314
+1
---
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
enable
> On Apr 14, 2016, at 1:44 PM, bri...@apache.org wrote:
>
> This is an automated email from the ASF dual-hosted git repository.
>
> briang pushed a commit to branch master
> in repository https://git-dual.apache.org/repos/asf/trafficserver.git
>
> The following commit(s) were added to refs/head
Github user jpeach commented on the pull request:
https://github.com/apache/trafficserver/pull/572#issuecomment-210154373
It would be great to capture some of the information from TS-4323 in the
commit message.
---
If your project is set up for it, you can reply to this email and hav
GitHub user zwoop opened a pull request:
https://github.com/apache/trafficserver/pull/572
TS-4323 Change default DATA frame size to 16k
EOM.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zwoop/trafficserver TS-4323
Alternativel
GitHub user pbchou opened a pull request:
https://github.com/apache/trafficserver/pull/571
TS-3245 : Changes optind = 1 to optind = 0 for all plugins to ensure
that multiples plugins in the global plugin.config can co-exist
together. Note that all plugins were
Github user jacksontj commented on the pull request:
https://github.com/apache/trafficserver/pull/562#issuecomment-210075450
This was rolled into https://github.com/apache/trafficserver/pull/564
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user jacksontj closed the pull request at:
https://github.com/apache/trafficserver/pull/562
---
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 closed the pull request at:
https://github.com/apache/trafficserver/pull/564
---
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 bgaff closed the pull request at:
https://github.com/apache/trafficserver/pull/569
---
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 feature i
Github user bgaff commented on the pull request:
https://github.com/apache/trafficserver/pull/564#issuecomment-210074340
@jacksontj I'm going to merge my pull request, can you please update yours
once my code lands? Thanks
---
If your project is set up for it, you can reply to this e
Github user jrushford closed the pull request at:
https://github.com/apache/trafficserver/pull/530
---
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 jrushford commented on the pull request:
https://github.com/apache/trafficserver/pull/530#issuecomment-210039889
Changes have been committed.
---
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 doe
Github user shinrich commented on the pull request:
https://github.com/apache/trafficserver/pull/570#issuecomment-210037794
I figured there would be some more updates via the review process so I
delayed further squashes. Will definitely squash before the final commit.
Since w
Github user jacksontj commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/564#discussion_r59743447
--- Diff: lib/ts/ink_inet.cc ---
@@ -307,6 +307,26 @@ ats_ip_hash(sockaddr const *addr)
return zret.i;
}
+uint64_t
+ats_ip_p
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/564#discussion_r59742958
--- Diff: lib/ts/ink_inet.cc ---
@@ -307,6 +307,26 @@ ats_ip_hash(sockaddr const *addr)
return zret.i;
}
+uint64_t
+ats_ip_port
Github user jpeach commented on the pull request:
https://github.com/apache/trafficserver/pull/570#issuecomment-210011350
Please squash the clang-format commit. Can you elaborate on why this
reverts TS-4321?
---
If your project is set up for it, you can reply to this email and have y
Github user jacksontj commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/564#discussion_r59741127
--- Diff: lib/ts/ink_inet.cc ---
@@ -307,6 +307,26 @@ ats_ip_hash(sockaddr const *addr)
return zret.i;
}
+uint64_t
+ats_ip_p
GitHub user shinrich opened a pull request:
https://github.com/apache/trafficserver/pull/570
Ts 3612 - Restructuring client session and transaction processing
This has been running on a 5.3 base in our production in a few machines for
about a month. We are starting to roll it out m
Github user jacksontj commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/564#discussion_r59737621
--- Diff: lib/ts/ink_inet.cc ---
@@ -307,6 +307,26 @@ ats_ip_hash(sockaddr const *addr)
return zret.i;
}
+uint64_t
+ats_ip_p
On Thu, Apr 14, 2016 at 1:50 PM, James Peach wrote:
>
>> On Apr 13, 2016, at 9:45 PM, Masakazu Kitajo wrote:
>>
>> I merged a pull request on GitHub with "squash and merge" button.
>> https://github.com/blog/2141-squash-your-commits
>>
>> This makes only one commits on master like we used to. It
22 matches
Mail list logo
