Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • Knot Resolver Knot Resolver
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 144
    • Issues 144
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 14
    • Merge requests 14
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Knot projects
  • Knot ResolverKnot Resolver
  • Merge requests
  • !1323

TTL tweaks

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Vladimír Čunát requested to merge cache-ttl into master Jul 14, 2022
  • Overview 6
  • Commits 4
  • Pipelines 4
  • Changes 18

I've been reluctant to do (some of) these changes in a patch-version update, so this has been hibernating, but let me open a MR already.

Fixes #127 (closed); fixes #736 (closed). (and other TTL-related changes)

The earlier choice was applying TTL bounds only when caching. That makes some sense from implementation point of view, but I can't see any motivation to behave that way and treat the first answer differently.

Edited Jul 18, 2022 by Vladimír Čunát
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: cache-ttl