Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/788
---
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
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/770
---
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
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/783
---
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
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/783
[approve ci]
---
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
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/783
ð
---
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
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/789
---
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
GitHub user jpeach opened a pull request:
https://github.com/apache/trafficserver/pull/789
TS-4100: Remove remaining stats processor code.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jpeach/trafficserver fix/4100-2
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/786#discussion_r69782549
--- Diff: proxy/http/HttpSM.cc ---
@@ -1954,9 +1954,7 @@ HttpSM::state_read_server_response_header(int event,
void *data
GitHub user jpeach opened a pull request:
https://github.com/apache/trafficserver/pull/788
TS-4635: Fix RefCountCacheSerializer file descriptor leaks.
Fix various file descriptor leaks on the RefCountCacheSerializer error
path. Move RefCountCacheSerializer to a separate header so
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/783#discussion_r69640431
--- Diff: proxy/StatPages.cc ---
@@ -83,7 +83,7 @@ StatPagesManager::handle_http(Continuation *cont, HTTPHdr
*header)
host_len
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/787
Is is possible to break the addditional fixes out into separate tickets and
PRs?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/783#discussion_r69635990
--- Diff: proxy/StatPages.cc ---
@@ -83,7 +83,7 @@ StatPagesManager::handle_http(Continuation *cont, HTTPHdr
*header)
host_len
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/783#discussion_r69613350
--- Diff: proxy/StatPages.cc ---
@@ -83,7 +83,7 @@ StatPagesManager::handle_http(Continuation *cont, HTTPHdr
*header)
host_len
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/783#discussion_r69611157
--- Diff: proxy/http/HttpTransact.cc ---
@@ -762,8 +762,8 @@ HttpTransact::StartRemapRequest(State *s)
const char syntxt[] = "syntheti
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/786#discussion_r69609382
--- Diff: proxy/http/HttpSM.cc ---
@@ -1954,9 +1954,7 @@ HttpSM::state_read_server_response_header(int event,
void *data
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/786#discussion_r69606800
--- Diff: proxy/http/HttpSM.cc ---
@@ -7618,6 +7616,8 @@ HttpSM::do_redirect()
return;
}
+ redirection_tries
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/779#discussion_r69363166
--- Diff: plugins/experimental/ts_lua/ts_lua.c ---
@@ -116,6 +116,69 @@ TSRemapDeleteInstance(void *ih)
return;
}
+void
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/779#discussion_r69363133
--- Diff: plugins/experimental/ts_lua/ts_lua.c ---
@@ -116,6 +116,69 @@ TSRemapDeleteInstance(void *ih)
return;
}
+void
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/779#discussion_r69362522
--- Diff: plugins/experimental/ts_lua/ts_lua_http.c ---
@@ -29,9 +29,33 @@ typedef enum {
TS_LUA_CACHE_LOOKUP_SKIPPED
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/779#discussion_r69362504
--- Diff: plugins/experimental/ts_lua/ts_lua_http.c ---
@@ -29,9 +29,33 @@ typedef enum {
TS_LUA_CACHE_LOOKUP_SKIPPED
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/784
---
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
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/783#discussion_r69356010
--- Diff: proxy/http/HttpTransact.cc ---
@@ -762,8 +762,8 @@ HttpTransact::StartRemapRequest(State *s)
const char syntxt[] = "syntheti
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/783#discussion_r69355867
--- Diff: proxy/http/HttpTransact.cc ---
@@ -249,7 +249,7 @@ find_server_and_update_current_info(HttpTransact::State
*s)
int host_len
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/783#discussion_r69355758
--- Diff: proxy/StatPages.cc ---
@@ -83,7 +83,7 @@ StatPagesManager::handle_http(Continuation *cont, HTTPHdr
*header)
host_len
GitHub user jpeach opened a pull request:
https://github.com/apache/trafficserver/pull/784
Temporarily disable ParentProxy API tests.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jpeach/trafficserver fix/disable-proxy-tests
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/784
@zwoop
---
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
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/777
ð
---
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
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/777#discussion_r69330732
--- Diff: tools/pre-commit ---
@@ -0,0 +1,66 @@
+#! /usr/bin/env bash
+#
+# Simple pre-commit hook script, enforcing clang-format on our
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/740
---
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
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/741
---
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
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/775
s/NetworkUtilisRemote/NetworkUtilsRemote/ ;)
---
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 jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/773
@jacksontj Maybe the right place to do this is when
``HttpSM::set_next_state()
`` handles ``SM_ACTION_DNS_LOOKUP``?
---
If your project is set up for it, you can reply to this email and
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69190550
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -143,18 +144,17 @@ on_http_cache_lookup_complete(TSHttpTxn txnp, TSCont
contp
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69190382
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -202,34 +202,38 @@ handle_purge(TSHttpTxn txnp, PurgeInstance *purge)
bool
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/773
-1, this breaks ``TSHttpTxnServerAddrSet``.
---
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 jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69163526
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69163433
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69163229
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69163031
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69162592
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69162190
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69160469
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69160033
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69159927
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69158965
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69158585
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69158201
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69157502
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69157359
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69157179
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/770
@zwoop @PSUdaemon
Please take this for a spin, but let's not merge yet. It looks like
``clang-tidy`` still get various build errors which cause it to eat code in
unfortunate
GitHub user jpeach opened a pull request:
https://github.com/apache/trafficserver/pull/770
Integrate clang-tidy into the build.
Add a recursive tidy rule to run clang-tidy over all the source.
The mor recommended way to do this is to build a compilation database,
however the
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/740
@SolidWallOfCode I filed
[TS-4574](https://issues.apache.org/jira/browse/TS-4574) to clear the explicit
address. However now that I know a bit more about how the DNS resolutions
works, the
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/757
---
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
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/741
Abandoning this PR for now. Will go back and look into getting
``clang-tidy`` support into the build system.
---
If your project is set up for it, you can reply to this email and have your
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/757
Yup
---
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 jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/757
What is the output of ``python ./doc/checkvers.py --check-version``?
---
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
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/757
Do you have the ``sphinx-doc`` package installed?
---
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
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/745#discussion_r68974323
--- Diff: plugins/experimental/xdebug/xdebug.cc ---
@@ -234,6 +237,20 @@ InjectMilestonesHeader(TSHttpTxn txn, TSMBuffer
buffer, TSMLoc hdr
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/745#discussion_r68973675
--- Diff: plugins/experimental/xdebug/xdebug.cc ---
@@ -234,6 +237,20 @@ InjectMilestonesHeader(TSHttpTxn txn, TSMBuffer
buffer, TSMLoc hdr
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/757
@zwoop I'm going to merge this as is for now, ok?
---
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
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/748#discussion_r68971045
--- Diff: plugins/healthchecks/healthchecks.c ---
@@ -301,8 +301,22 @@ parse_configs(const char *fname)
char buf[2 * 1024];
HCFileInfo
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/748#discussion_r68970820
--- Diff: plugins/healthchecks/healthchecks.c ---
@@ -301,8 +301,22 @@ parse_configs(const char *fname)
char buf[2 * 1024];
HCFileInfo
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/764
---
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
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/764
[approve ci]
---
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
GitHub user jpeach opened a pull request:
https://github.com/apache/trafficserver/pull/764
Fix the CPP API plugins to link against libatscppapi.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jpeach/trafficserver fix/cppapi
GitHub user jpeach opened a pull request:
https://github.com/apache/trafficserver/pull/759
TS-4609: Fix uninitialized read in SDK_API_HttpTxnTransform.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jpeach/trafficserver fix
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/759
[approve ci]
---
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
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/757
For a suboption, I think the syntax would be:
:option:`traffic_ctl config set` NAME VALUE
But I'd have to render the docs locally to see exactly what happens.
@persi
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/756
---
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
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/756#discussion_r68795473
--- Diff: doc/admin-guide/configuring-traffic-server.en.rst ---
@@ -27,14 +27,14 @@ Traffic Server provides several options for configuring
the system
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/756#discussion_r68793583
--- Diff: doc/admin-guide/configuring-traffic-server.en.rst ---
@@ -43,13 +43,13 @@ To view a configuration setting, enter the following
command
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/756#discussion_r68793447
--- Diff: doc/admin-guide/configuring-traffic-server.en.rst ---
@@ -27,13 +27,13 @@ Traffic Server provides several options for configuring
the system
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/755
ð
---
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
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/750#discussion_r68694184
--- Diff: mgmt/RecordsConfig.cc ---
@@ -1212,6 +1212,8 @@ static const RecordElement RecordsConfig
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/750#discussion_r68693861
--- Diff: mgmt/RecordsConfig.cc ---
@@ -1212,6 +1212,8 @@ static const RecordElement RecordsConfig
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/653
ð
---
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
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/736
Looks good ð
---
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
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/736#discussion_r68610169
--- Diff: proxy/api/ts/ts.h ---
@@ -2389,6 +2392,19 @@ tsapi const char *TSHttpHookNameLookup(TSHttpHookID
hook);
*/
tsapi const char
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/736#discussion_r68609628
--- Diff: doc/developer-guide/api/types/TSUuid.en.rst ---
@@ -0,0 +1,48 @@
+.. Licensed to the Apache Software Foundation (ASF) under one or more
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/736#discussion_r68609258
--- Diff: doc/developer-guide/api/types/TSUuid.en.rst ---
@@ -0,0 +1,48 @@
+.. Licensed to the Apache Software Foundation (ASF) under one or more
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/736#discussion_r68609126
--- Diff: doc/developer-guide/api/types/TSUuid.en.rst ---
@@ -0,0 +1,48 @@
+.. Licensed to the Apache Software Foundation (ASF) under one or more
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/736#discussion_r68608955
--- Diff: doc/developer-guide/api/functions/TSUuidCreate.en.rst ---
@@ -0,0 +1,138 @@
+.. Licensed to the Apache Software Foundation (ASF) under one
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/736#discussion_r68608958
--- Diff: doc/developer-guide/api/functions/TSUuidCreate.en.rst ---
@@ -0,0 +1,138 @@
+.. Licensed to the Apache Software Foundation (ASF) under one
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68605778
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/742
---
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
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68496042
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68496039
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68496026
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68496007
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68495996
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68495991
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68495986
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68495979
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68495976
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68495955
--- Diff: iocore/hostdb/HostDB.cc ---
@@ -1234,8 +1036,8 @@ HostDBContinuation::removeEvent(int /* event
ATS_UNUSED */, Event *e)
if (cont
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68495939
--- Diff: iocore/hostdb/HostDB.cc ---
@@ -300,131 +237,77 @@ HostDBProcessor::cache()
return &hostDB;
}
-struct HostDBTe
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68495917
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68495915
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/653#discussion_r68495906
--- Diff: iocore/hostdb/P_RefCountCache.h ---
@@ -0,0 +1,896 @@
+/** @file
+
+ A cache (with map-esque interface) for RefCountObjs
1 - 100 of 680 matches
Mail list logo
