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 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 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 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 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
Github user jacksontj commented on the pull request:
https://github.com/apache/trafficserver/pull/564#issuecomment-209719690
@bgaff @jpeach @zwoop ready for 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 you
Github user jacksontj commented on the pull request:
https://github.com/apache/trafficserver/pull/564#issuecomment-209719516
This PR now includes the patches from
https://github.com/apache/trafficserver/pull/562 and
https://github.com/apache/trafficserver/pull/569 as they are related
Github user jacksontj commented on the pull request:
https://github.com/apache/trafficserver/pull/564#issuecomment-209001031
Well, it seems that this doesn't leak in the queue-- but I did find another
bug (https://issues.apache.org/jira/browse/TS-4343) which we'll probably want
to fix
Github user bgaff commented on the pull request:
https://github.com/apache/trafficserver/pull/564#issuecomment-208754987
:eyes: I'm on it.
---
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 jacksontj commented on the pull request:
https://github.com/apache/trafficserver/pull/564#issuecomment-208718335
This isn't complete yet, still need to add handling for timeouts etc., but
I wanted to get the PR opened to get some eyes on it :)
---
If your project is set u
GitHub user jacksontj opened a pull request:
https://github.com/apache/trafficserver/pull/564
Ts 4341
https://jira01.corp.linkedin.com:8443/browse/TRAFFIC-5688
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jacksontj/trafficserve
12 matches
Mail list logo
