diff options
| author |  Miek Gieben <miek@miek.nl>
| 2019-10-04 09:44:58 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2019-10-04 09:44:58 +0100 |
| commit | 03ea2ae955823612da2e3b2ebf21da7ab8eea1bd (patch) | |
| tree | 8b140512f8000f4d2042ad43a3dd5e8e301a3328 /plugin/cache/cache_test.go | |
| parent | f8551df27284b93e9f95e064ec65a9df60f4a528 (diff) | |
| download | coredns-03ea2ae955823612da2e3b2ebf21da7ab8eea1bd.tar.gz coredns-03ea2ae955823612da2e3b2ebf21da7ab8eea1bd.tar.zst coredns-03ea2ae955823612da2e3b2ebf21da7ab8eea1bd.zip | |
Make request.Request smaller (#3351)
* Make request.Request smaller
This makes the request struct smaller and removes the pointer to the do
boolean (tri-bool) as size == 0 will indicate if we have cached it.
Family can be a int8 because it only carries 3 values, Size itself is
just a uint16 under the covers.
This is a more comprehensive fix than #3292
Closes #3292
Signed-off-by: Miek Gieben <miek@miek.nl>
* cache: fix test
this now needs a valid response writter
Signed-off-by: Miek Gieben <miek@miek.nl>
Diffstat (limited to 'plugin/cache/cache_test.go')
| -rw-r--r-- | plugin/cache/cache_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/plugin/cache/cache_test.go b/plugin/cache/cache_test.go index 4afaf73c4..b32353372 100644 --- a/plugin/cache/cache_test.go +++ b/plugin/cache/cache_test.go @@ -191,7 +191,7 @@ func TestCache(t *testing.T) { m := tc.in.Msg() m = cacheMsg(m, tc) - state := request.Request{W: nil, Req: m} + state := request.Request{W: &test.ResponseWriter{}, Req: m} mt, _ := response.Typify(m, utc) valid, k := key(state.Name(), m, mt, state.Do()) |